diff mbox series

[1/2] drm/bridge: chipone-icn6211: drop driver owner assignment

Message ID 20240330202741.83867-1-krzysztof.kozlowski@linaro.org (mailing list archive)
State New, archived
Headers show
Series [1/2] drm/bridge: chipone-icn6211: drop driver owner assignment | expand

Commit Message

Krzysztof Kozlowski March 30, 2024, 8:27 p.m. UTC
Core in mipi_dsi_driver_register() already sets the .owner, so driver
does not need to.

Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
 drivers/gpu/drm/bridge/chipone-icn6211.c | 1 -
 1 file changed, 1 deletion(-)

Comments

Krzysztof Kozlowski April 24, 2024, 6:39 a.m. UTC | #1
On 30/03/2024 21:27, Krzysztof Kozlowski wrote:
> Core in mipi_dsi_driver_register() already sets the .owner, so driver
> does not need to.
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> ---

It has been almost a month. Any comments on this patchset?

Best regards,
Krzysztof
Dmitry Baryshkov April 24, 2024, 9:12 a.m. UTC | #2
On Sat, Mar 30, 2024 at 09:27:40PM +0100, Krzysztof Kozlowski wrote:
> Core in mipi_dsi_driver_register() already sets the .owner, so driver
> does not need to.
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> ---
>  drivers/gpu/drm/bridge/chipone-icn6211.c | 1 -
>  1 file changed, 1 deletion(-)
> 

Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
Neil Armstrong April 25, 2024, 7:47 a.m. UTC | #3
Hi Dmitry,

On 24/04/2024 11:12, Dmitry Baryshkov wrote:
> On Sat, Mar 30, 2024 at 09:27:40PM +0100, Krzysztof Kozlowski wrote:
>> Core in mipi_dsi_driver_register() already sets the .owner, so driver
>> does not need to.
>>
>> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
>> ---
>>   drivers/gpu/drm/bridge/chipone-icn6211.c | 1 -
>>   1 file changed, 1 deletion(-)
>>
> 
> Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
> 
> 

I tried to apply them but you already applied them... could you make sure to notify the list when applying ?

Thanks,
Neil
Dmitry Baryshkov April 25, 2024, 8:02 a.m. UTC | #4
On Thu, 25 Apr 2024 at 10:47, Neil Armstrong <neil.armstrong@linaro.org> wrote:
>
> Hi Dmitry,
>
> On 24/04/2024 11:12, Dmitry Baryshkov wrote:
> > On Sat, Mar 30, 2024 at 09:27:40PM +0100, Krzysztof Kozlowski wrote:
> >> Core in mipi_dsi_driver_register() already sets the .owner, so driver
> >> does not need to.
> >>
> >> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> >> ---
> >>   drivers/gpu/drm/bridge/chipone-icn6211.c | 1 -
> >>   1 file changed, 1 deletion(-)
> >>
> >
> > Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
> >
> >
>
> I tried to apply them but you already applied them... could you make sure to notify the list when applying ?

Ack, excuse me.
diff mbox series

Patch

diff --git a/drivers/gpu/drm/bridge/chipone-icn6211.c b/drivers/gpu/drm/bridge/chipone-icn6211.c
index 82d23e4df09e..43c1202701ee 100644
--- a/drivers/gpu/drm/bridge/chipone-icn6211.c
+++ b/drivers/gpu/drm/bridge/chipone-icn6211.c
@@ -783,7 +783,6 @@  static struct mipi_dsi_driver chipone_dsi_driver = {
 	.remove = chipone_dsi_remove,
 	.driver = {
 		.name = "chipone-icn6211",
-		.owner = THIS_MODULE,
 		.of_match_table = chipone_of_match,
 	},
 };