* [PATCH] Drop owner assignment from i2c_driver (and platform left-overs) @ 2015-07-10 5:36 Krzysztof Kozlowski 2015-07-10 5:36 ` [PATCH] drm: Drop owner assignment from i2c_driver Krzysztof Kozlowski 0 siblings, 1 reply; 5+ messages in thread From: Krzysztof Kozlowski @ 2015-07-10 5:36 UTC (permalink / raw) To: linux-arm-kernel Hi, The i2c drivers also do not have to set 'owner' field because i2c_register_driver() will do it instead. 'owner' is removed from i2c drivers, which I was able to compile with allyesconfig (arm, arm64, i386, x86_64, ppc64). Only compile-tested. The coccinelle script which generated the patch was sent here: http://www.spinics.net/lists/kernel/msg2029903.html Best regards, Krzysztof Krzysztof Kozlowski (1): drm: Drop owner assignment from i2c_driver drivers/gpu/drm/bridge/ps8622.c | 1 - drivers/gpu/drm/bridge/ptn3460.c | 1 - drivers/gpu/drm/rockchip/rockchip_drm_drv.c | 1 - 3 files changed, 3 deletions(-) -- 1.9.1 ^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH] drm: Drop owner assignment from i2c_driver 2015-07-10 5:36 [PATCH] Drop owner assignment from i2c_driver (and platform left-overs) Krzysztof Kozlowski @ 2015-07-10 5:36 ` Krzysztof Kozlowski 2015-07-10 6:50 ` Mark yao 0 siblings, 1 reply; 5+ messages in thread From: Krzysztof Kozlowski @ 2015-07-10 5:36 UTC (permalink / raw) To: linux-arm-kernel i2c_driver does not need to set an owner because i2c_register_driver() will set it. Signed-off-by: Krzysztof Kozlowski <k.kozlowski@samsung.com> --- The coccinelle script which generated the patch was sent here: http://www.spinics.net/lists/kernel/msg2029903.html --- drivers/gpu/drm/bridge/ps8622.c | 1 - drivers/gpu/drm/bridge/ptn3460.c | 1 - drivers/gpu/drm/rockchip/rockchip_drm_drv.c | 1 - 3 files changed, 3 deletions(-) diff --git a/drivers/gpu/drm/bridge/ps8622.c b/drivers/gpu/drm/bridge/ps8622.c index 1a6607beb29f..be881e9fef8f 100644 --- a/drivers/gpu/drm/bridge/ps8622.c +++ b/drivers/gpu/drm/bridge/ps8622.c @@ -668,7 +668,6 @@ static struct i2c_driver ps8622_driver = { .remove = ps8622_remove, .driver = { .name = "ps8622", - .owner = THIS_MODULE, .of_match_table = ps8622_devices, }, }; diff --git a/drivers/gpu/drm/bridge/ptn3460.c b/drivers/gpu/drm/bridge/ptn3460.c index 1b1bf2384815..0ffa3a6a206a 100644 --- a/drivers/gpu/drm/bridge/ptn3460.c +++ b/drivers/gpu/drm/bridge/ptn3460.c @@ -400,7 +400,6 @@ static struct i2c_driver ptn3460_driver = { .remove = ptn3460_remove, .driver = { .name = "nxp,ptn3460", - .owner = THIS_MODULE, .of_match_table = ptn3460_match, }, }; diff --git a/drivers/gpu/drm/rockchip/rockchip_drm_drv.c b/drivers/gpu/drm/rockchip/rockchip_drm_drv.c index 01b558fe3695..9a0c2911272a 100644 --- a/drivers/gpu/drm/rockchip/rockchip_drm_drv.c +++ b/drivers/gpu/drm/rockchip/rockchip_drm_drv.c @@ -555,7 +555,6 @@ static struct platform_driver rockchip_drm_platform_driver = { .probe = rockchip_drm_platform_probe, .remove = rockchip_drm_platform_remove, .driver = { - .owner = THIS_MODULE, .name = "rockchip-drm", .of_match_table = rockchip_drm_dt_ids, .pm = &rockchip_drm_pm_ops, -- 1.9.1 ^ permalink raw reply related [flat|nested] 5+ messages in thread
* [PATCH] drm: Drop owner assignment from i2c_driver 2015-07-10 5:36 ` [PATCH] drm: Drop owner assignment from i2c_driver Krzysztof Kozlowski @ 2015-07-10 6:50 ` Mark yao 2015-07-10 7:01 ` Krzysztof Kozlowski 0 siblings, 1 reply; 5+ messages in thread From: Mark yao @ 2015-07-10 6:50 UTC (permalink / raw) To: linux-arm-kernel On 2015?07?10? 13:36, Krzysztof Kozlowski wrote: > i2c_driver does not need to set an owner because i2c_register_driver() > will set it. > > Signed-off-by: Krzysztof Kozlowski <k.kozlowski@samsung.com> > > --- > > The coccinelle script which generated the patch was sent here: > http://www.spinics.net/lists/kernel/msg2029903.html > --- > drivers/gpu/drm/bridge/ps8622.c | 1 - > drivers/gpu/drm/bridge/ptn3460.c | 1 - > drivers/gpu/drm/rockchip/rockchip_drm_drv.c | 1 - > 3 files changed, 3 deletions(-) > > diff --git a/drivers/gpu/drm/bridge/ps8622.c b/drivers/gpu/drm/bridge/ps8622.c > index 1a6607beb29f..be881e9fef8f 100644 > --- a/drivers/gpu/drm/bridge/ps8622.c > +++ b/drivers/gpu/drm/bridge/ps8622.c > @@ -668,7 +668,6 @@ static struct i2c_driver ps8622_driver = { > .remove = ps8622_remove, > .driver = { > .name = "ps8622", > - .owner = THIS_MODULE, > .of_match_table = ps8622_devices, > }, > }; > diff --git a/drivers/gpu/drm/bridge/ptn3460.c b/drivers/gpu/drm/bridge/ptn3460.c > index 1b1bf2384815..0ffa3a6a206a 100644 > --- a/drivers/gpu/drm/bridge/ptn3460.c > +++ b/drivers/gpu/drm/bridge/ptn3460.c > @@ -400,7 +400,6 @@ static struct i2c_driver ptn3460_driver = { > .remove = ptn3460_remove, > .driver = { > .name = "nxp,ptn3460", > - .owner = THIS_MODULE, > .of_match_table = ptn3460_match, > }, > }; > diff --git a/drivers/gpu/drm/rockchip/rockchip_drm_drv.c b/drivers/gpu/drm/rockchip/rockchip_drm_drv.c > index 01b558fe3695..9a0c2911272a 100644 > --- a/drivers/gpu/drm/rockchip/rockchip_drm_drv.c > +++ b/drivers/gpu/drm/rockchip/rockchip_drm_drv.c > @@ -555,7 +555,6 @@ static struct platform_driver rockchip_drm_platform_driver = { > .probe = rockchip_drm_platform_probe, > .remove = rockchip_drm_platform_remove, > .driver = { > - .owner = THIS_MODULE, But rockchip drm is platform driver not i2c_driver, why remove its .owner ? -Mark > .name = "rockchip-drm", > .of_match_table = rockchip_drm_dt_ids, > .pm = &rockchip_drm_pm_ops, ^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH] drm: Drop owner assignment from i2c_driver 2015-07-10 6:50 ` Mark yao @ 2015-07-10 7:01 ` Krzysztof Kozlowski 2015-07-10 7:11 ` Mark yao 0 siblings, 1 reply; 5+ messages in thread From: Krzysztof Kozlowski @ 2015-07-10 7:01 UTC (permalink / raw) To: linux-arm-kernel On 10.07.2015 15:50, Mark yao wrote: > On 2015?07?10? 13:36, Krzysztof Kozlowski wrote: >> i2c_driver does not need to set an owner because i2c_register_driver() >> will set it. >> >> Signed-off-by: Krzysztof Kozlowski <k.kozlowski@samsung.com> >> >> --- >> >> The coccinelle script which generated the patch was sent here: >> http://www.spinics.net/lists/kernel/msg2029903.html >> --- >> drivers/gpu/drm/bridge/ps8622.c | 1 - >> drivers/gpu/drm/bridge/ptn3460.c | 1 - >> drivers/gpu/drm/rockchip/rockchip_drm_drv.c | 1 - >> 3 files changed, 3 deletions(-) >> >> diff --git a/drivers/gpu/drm/bridge/ps8622.c >> b/drivers/gpu/drm/bridge/ps8622.c >> index 1a6607beb29f..be881e9fef8f 100644 >> --- a/drivers/gpu/drm/bridge/ps8622.c >> +++ b/drivers/gpu/drm/bridge/ps8622.c >> @@ -668,7 +668,6 @@ static struct i2c_driver ps8622_driver = { >> .remove = ps8622_remove, >> .driver = { >> .name = "ps8622", >> - .owner = THIS_MODULE, >> .of_match_table = ps8622_devices, >> }, >> }; >> diff --git a/drivers/gpu/drm/bridge/ptn3460.c >> b/drivers/gpu/drm/bridge/ptn3460.c >> index 1b1bf2384815..0ffa3a6a206a 100644 >> --- a/drivers/gpu/drm/bridge/ptn3460.c >> +++ b/drivers/gpu/drm/bridge/ptn3460.c >> @@ -400,7 +400,6 @@ static struct i2c_driver ptn3460_driver = { >> .remove = ptn3460_remove, >> .driver = { >> .name = "nxp,ptn3460", >> - .owner = THIS_MODULE, >> .of_match_table = ptn3460_match, >> }, >> }; >> diff --git a/drivers/gpu/drm/rockchip/rockchip_drm_drv.c >> b/drivers/gpu/drm/rockchip/rockchip_drm_drv.c >> index 01b558fe3695..9a0c2911272a 100644 >> --- a/drivers/gpu/drm/rockchip/rockchip_drm_drv.c >> +++ b/drivers/gpu/drm/rockchip/rockchip_drm_drv.c >> @@ -555,7 +555,6 @@ static struct platform_driver >> rockchip_drm_platform_driver = { >> .probe = rockchip_drm_platform_probe, >> .remove = rockchip_drm_platform_remove, >> .driver = { >> - .owner = THIS_MODULE, > > But rockchip drm is platform driver not i2c_driver, why remove its .owner ? Oh, indeed. Thanks for spotting this. The 'owner' is set by core for platform drivers as well. Most platform drivers were already converted (I think by Wolfram Sang). I extended existing coccinelle script to fix also i2c_drivers and sometimes did not notice that it was platform_driver. I can split it into two different patches. Would that be ok? Best regards, Krzysztof ^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH] drm: Drop owner assignment from i2c_driver 2015-07-10 7:01 ` Krzysztof Kozlowski @ 2015-07-10 7:11 ` Mark yao 0 siblings, 0 replies; 5+ messages in thread From: Mark yao @ 2015-07-10 7:11 UTC (permalink / raw) To: linux-arm-kernel On 2015?07?10? 15:01, Krzysztof Kozlowski wrote: > On 10.07.2015 15:50, Mark yao wrote: >> On 2015?07?10? 13:36, Krzysztof Kozlowski wrote: >>> i2c_driver does not need to set an owner because i2c_register_driver() >>> will set it. >>> >>> Signed-off-by: Krzysztof Kozlowski <k.kozlowski@samsung.com> >>> >>> --- >>> >>> The coccinelle script which generated the patch was sent here: >>> http://www.spinics.net/lists/kernel/msg2029903.html >>> --- >>> drivers/gpu/drm/bridge/ps8622.c | 1 - >>> drivers/gpu/drm/bridge/ptn3460.c | 1 - >>> drivers/gpu/drm/rockchip/rockchip_drm_drv.c | 1 - >>> 3 files changed, 3 deletions(-) >>> >>> diff --git a/drivers/gpu/drm/bridge/ps8622.c >>> b/drivers/gpu/drm/bridge/ps8622.c >>> index 1a6607beb29f..be881e9fef8f 100644 >>> --- a/drivers/gpu/drm/bridge/ps8622.c >>> +++ b/drivers/gpu/drm/bridge/ps8622.c >>> @@ -668,7 +668,6 @@ static struct i2c_driver ps8622_driver = { >>> .remove = ps8622_remove, >>> .driver = { >>> .name = "ps8622", >>> - .owner = THIS_MODULE, >>> .of_match_table = ps8622_devices, >>> }, >>> }; >>> diff --git a/drivers/gpu/drm/bridge/ptn3460.c >>> b/drivers/gpu/drm/bridge/ptn3460.c >>> index 1b1bf2384815..0ffa3a6a206a 100644 >>> --- a/drivers/gpu/drm/bridge/ptn3460.c >>> +++ b/drivers/gpu/drm/bridge/ptn3460.c >>> @@ -400,7 +400,6 @@ static struct i2c_driver ptn3460_driver = { >>> .remove = ptn3460_remove, >>> .driver = { >>> .name = "nxp,ptn3460", >>> - .owner = THIS_MODULE, >>> .of_match_table = ptn3460_match, >>> }, >>> }; >>> diff --git a/drivers/gpu/drm/rockchip/rockchip_drm_drv.c >>> b/drivers/gpu/drm/rockchip/rockchip_drm_drv.c >>> index 01b558fe3695..9a0c2911272a 100644 >>> --- a/drivers/gpu/drm/rockchip/rockchip_drm_drv.c >>> +++ b/drivers/gpu/drm/rockchip/rockchip_drm_drv.c >>> @@ -555,7 +555,6 @@ static struct platform_driver >>> rockchip_drm_platform_driver = { >>> .probe = rockchip_drm_platform_probe, >>> .remove = rockchip_drm_platform_remove, >>> .driver = { >>> - .owner = THIS_MODULE, >> But rockchip drm is platform driver not i2c_driver, why remove its .owner ? > Oh, indeed. Thanks for spotting this. > > The 'owner' is set by core for platform drivers as well. Most platform > drivers were already converted (I think by Wolfram Sang). I extended > existing coccinelle script to fix also i2c_drivers and sometimes did not > notice that it was platform_driver. > > I can split it into two different patches. Would that be ok? > > Best regards, > Krzysztof > Ok, Thanks for the fix.:-) -- ?ark ^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2015-07-10 7:11 UTC | newest] Thread overview: 5+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2015-07-10 5:36 [PATCH] Drop owner assignment from i2c_driver (and platform left-overs) Krzysztof Kozlowski 2015-07-10 5:36 ` [PATCH] drm: Drop owner assignment from i2c_driver Krzysztof Kozlowski 2015-07-10 6:50 ` Mark yao 2015-07-10 7:01 ` Krzysztof Kozlowski 2015-07-10 7:11 ` Mark yao
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).