diff mbox

[05/28] cpufreq: drop owner assignment from platform_drivers

Message ID 1419196495-9626-6-git-send-email-wsa@the-dreams.de (mailing list archive)
State Accepted, archived
Delegated to: Rafael Wysocki
Headers show

Commit Message

Wolfram Sang Dec. 21, 2014, 9:14 p.m. UTC
This platform_driver does not need to set an owner, it will be populated by the
driver core.

Signed-off-by: Wolfram Sang <wsa@the-dreams.de>
---
Generated with coccinelle. SmPL file is in the introductory msg. The big
cleanup was pulled in this merge window. This series catches the bits fallen
through. The patches shall go in via the subsystem trees.

 drivers/cpufreq/ls1x-cpufreq.c | 1 -
 1 file changed, 1 deletion(-)

Comments

Viresh Kumar Dec. 22, 2014, 1:44 a.m. UTC | #1
On 22 December 2014 at 02:44, Wolfram Sang <wsa@the-dreams.de> wrote:
> This platform_driver does not need to set an owner, it will be populated by the
> driver core.
>
> Signed-off-by: Wolfram Sang <wsa@the-dreams.de>
> ---
> Generated with coccinelle. SmPL file is in the introductory msg. The big
> cleanup was pulled in this merge window. This series catches the bits fallen
> through. The patches shall go in via the subsystem trees.
>
>  drivers/cpufreq/ls1x-cpufreq.c | 1 -
>  1 file changed, 1 deletion(-)
>
> diff --git a/drivers/cpufreq/ls1x-cpufreq.c b/drivers/cpufreq/ls1x-cpufreq.c
> index 25fbd6a1374f..f0913eee2f50 100644
> --- a/drivers/cpufreq/ls1x-cpufreq.c
> +++ b/drivers/cpufreq/ls1x-cpufreq.c
> @@ -210,7 +210,6 @@ out:
>  static struct platform_driver ls1x_cpufreq_platdrv = {
>         .driver = {
>                 .name   = "ls1x-cpufreq",
> -               .owner  = THIS_MODULE,
>         },
>         .probe          = ls1x_cpufreq_probe,
>         .remove         = ls1x_cpufreq_remove,

Acked-by: Viresh Kumar <viresh.kumar@linaro.org>
--
To unsubscribe from this list: send the line "unsubscribe linux-pm" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Rafael J. Wysocki Dec. 22, 2014, 10:39 p.m. UTC | #2
On Monday, December 22, 2014 07:14:56 AM Viresh Kumar wrote:
> On 22 December 2014 at 02:44, Wolfram Sang <wsa@the-dreams.de> wrote:
> > This platform_driver does not need to set an owner, it will be populated by the
> > driver core.
> >
> > Signed-off-by: Wolfram Sang <wsa@the-dreams.de>
> > ---
> > Generated with coccinelle. SmPL file is in the introductory msg. The big
> > cleanup was pulled in this merge window. This series catches the bits fallen
> > through. The patches shall go in via the subsystem trees.
> >
> >  drivers/cpufreq/ls1x-cpufreq.c | 1 -
> >  1 file changed, 1 deletion(-)
> >
> > diff --git a/drivers/cpufreq/ls1x-cpufreq.c b/drivers/cpufreq/ls1x-cpufreq.c
> > index 25fbd6a1374f..f0913eee2f50 100644
> > --- a/drivers/cpufreq/ls1x-cpufreq.c
> > +++ b/drivers/cpufreq/ls1x-cpufreq.c
> > @@ -210,7 +210,6 @@ out:
> >  static struct platform_driver ls1x_cpufreq_platdrv = {
> >         .driver = {
> >                 .name   = "ls1x-cpufreq",
> > -               .owner  = THIS_MODULE,
> >         },
> >         .probe          = ls1x_cpufreq_probe,
> >         .remove         = ls1x_cpufreq_remove,
> 
> Acked-by: Viresh Kumar <viresh.kumar@linaro.org>
> --

OK

Wolfram, please feel free to push this through whatever tree you think is
appropriate.
Wolfram Sang Dec. 31, 2014, 10:23 a.m. UTC | #3
On Mon, Dec 22, 2014 at 11:39:32PM +0100, Rafael J. Wysocki wrote:
> On Monday, December 22, 2014 07:14:56 AM Viresh Kumar wrote:
> > On 22 December 2014 at 02:44, Wolfram Sang <wsa@the-dreams.de> wrote:
> > > This platform_driver does not need to set an owner, it will be populated by the
> > > driver core.
> > >
> > > Signed-off-by: Wolfram Sang <wsa@the-dreams.de>
> > > ---
> > > Generated with coccinelle. SmPL file is in the introductory msg. The big
> > > cleanup was pulled in this merge window. This series catches the bits fallen
> > > through. The patches shall go in via the subsystem trees.
> > >
> > >  drivers/cpufreq/ls1x-cpufreq.c | 1 -
> > >  1 file changed, 1 deletion(-)
> > >
> > > diff --git a/drivers/cpufreq/ls1x-cpufreq.c b/drivers/cpufreq/ls1x-cpufreq.c
> > > index 25fbd6a1374f..f0913eee2f50 100644
> > > --- a/drivers/cpufreq/ls1x-cpufreq.c
> > > +++ b/drivers/cpufreq/ls1x-cpufreq.c
> > > @@ -210,7 +210,6 @@ out:
> > >  static struct platform_driver ls1x_cpufreq_platdrv = {
> > >         .driver = {
> > >                 .name   = "ls1x-cpufreq",
> > > -               .owner  = THIS_MODULE,
> > >         },
> > >         .probe          = ls1x_cpufreq_probe,
> > >         .remove         = ls1x_cpufreq_remove,
> > 
> > Acked-by: Viresh Kumar <viresh.kumar@linaro.org>
> > --
> 
> OK
> 
> Wolfram, please feel free to push this through whatever tree you think is
> appropriate.

I'd prefer if you take it. However, I'll send all leftover ones to Greg,
so you can decide.

--
To unsubscribe from this list: send the line "unsubscribe linux-pm" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Rafael J. Wysocki Jan. 30, 2015, 12:12 a.m. UTC | #4
On Wednesday, December 31, 2014 11:23:26 AM Wolfram Sang wrote:
> On Mon, Dec 22, 2014 at 11:39:32PM +0100, Rafael J. Wysocki wrote:
> > On Monday, December 22, 2014 07:14:56 AM Viresh Kumar wrote:
> > > On 22 December 2014 at 02:44, Wolfram Sang <wsa@the-dreams.de> wrote:
> > > > This platform_driver does not need to set an owner, it will be populated by the
> > > > driver core.
> > > >
> > > > Signed-off-by: Wolfram Sang <wsa@the-dreams.de>
> > > > ---
> > > > Generated with coccinelle. SmPL file is in the introductory msg. The big
> > > > cleanup was pulled in this merge window. This series catches the bits fallen
> > > > through. The patches shall go in via the subsystem trees.
> > > >
> > > >  drivers/cpufreq/ls1x-cpufreq.c | 1 -
> > > >  1 file changed, 1 deletion(-)
> > > >
> > > > diff --git a/drivers/cpufreq/ls1x-cpufreq.c b/drivers/cpufreq/ls1x-cpufreq.c
> > > > index 25fbd6a1374f..f0913eee2f50 100644
> > > > --- a/drivers/cpufreq/ls1x-cpufreq.c
> > > > +++ b/drivers/cpufreq/ls1x-cpufreq.c
> > > > @@ -210,7 +210,6 @@ out:
> > > >  static struct platform_driver ls1x_cpufreq_platdrv = {
> > > >         .driver = {
> > > >                 .name   = "ls1x-cpufreq",
> > > > -               .owner  = THIS_MODULE,
> > > >         },
> > > >         .probe          = ls1x_cpufreq_probe,
> > > >         .remove         = ls1x_cpufreq_remove,
> > > 
> > > Acked-by: Viresh Kumar <viresh.kumar@linaro.org>
> > > --
> > 
> > OK
> > 
> > Wolfram, please feel free to push this through whatever tree you think is
> > appropriate.
> 
> I'd prefer if you take it. However, I'll send all leftover ones to Greg,
> so you can decide.

I've queued it up for 3.20, thanks!
diff mbox

Patch

diff --git a/drivers/cpufreq/ls1x-cpufreq.c b/drivers/cpufreq/ls1x-cpufreq.c
index 25fbd6a1374f..f0913eee2f50 100644
--- a/drivers/cpufreq/ls1x-cpufreq.c
+++ b/drivers/cpufreq/ls1x-cpufreq.c
@@ -210,7 +210,6 @@  out:
 static struct platform_driver ls1x_cpufreq_platdrv = {
 	.driver = {
 		.name	= "ls1x-cpufreq",
-		.owner	= THIS_MODULE,
 	},
 	.probe		= ls1x_cpufreq_probe,
 	.remove		= ls1x_cpufreq_remove,