From: Daniel Lezcano <daniel.lezcano@linaro.org> To: Paul Gortmaker <paul.gortmaker@windriver.com>, linux-kernel@vger.kernel.org Cc: "Rafael J. Wysocki" <rjw@rjwysocki.net>, Michal Simek <michal.simek@xilinx.com>, linux-pm@vger.kernel.org, linux-arm-kernel@lists.infradead.org Subject: Re: [PATCH 3/7] drivers/cpuidle: Convert non-modular drivers to use builtin_driver_register Date: Mon, 11 May 2015 19:13:07 +0200 [thread overview] Message-ID: <5550E323.1080800@linaro.org> (raw) In-Reply-To: <1431287385-1526-4-git-send-email-paul.gortmaker@windriver.com> On 05/10/2015 09:49 PM, Paul Gortmaker wrote: > All these drivers are configured with Kconfig options that are > declared as bool. Hence it is not possible for the code > to be built as modular. However the code is currently using the > module_platform_driver() macro for driver registration. > > While this currently works, we really don't want to be including > the module.h header in non-modular code, which we'll be forced > to do, pending some upcoming code relocation from init.h into > module.h. So we fix it now by using the non-modular equivalent. > > Cc: "Rafael J. Wysocki" <rjw@rjwysocki.net> > Cc: Daniel Lezcano <daniel.lezcano@linaro.org> > Cc: Michal Simek <michal.simek@xilinx.com> > Cc: linux-pm@vger.kernel.org > Cc: linux-arm-kernel@lists.infradead.org > Signed-off-by: Paul Gortmaker <paul.gortmaker@windriver.com> Sounds reasonable. -- Daniel > --- > drivers/cpuidle/cpuidle-at91.c | 3 +-- > drivers/cpuidle/cpuidle-calxeda.c | 3 +-- > drivers/cpuidle/cpuidle-zynq.c | 3 +-- > 3 files changed, 3 insertions(+), 6 deletions(-) > > diff --git a/drivers/cpuidle/cpuidle-at91.c b/drivers/cpuidle/cpuidle-at91.c > index f2446c78d87c..9c5853b6ca4a 100644 > --- a/drivers/cpuidle/cpuidle-at91.c > +++ b/drivers/cpuidle/cpuidle-at91.c > @@ -62,5 +62,4 @@ static struct platform_driver at91_cpuidle_driver = { > }, > .probe = at91_cpuidle_probe, > }; > - > -module_platform_driver(at91_cpuidle_driver); > +builtin_platform_driver(at91_cpuidle_driver); > diff --git a/drivers/cpuidle/cpuidle-calxeda.c b/drivers/cpuidle/cpuidle-calxeda.c > index 9445e6cc02be..c13feec89ea1 100644 > --- a/drivers/cpuidle/cpuidle-calxeda.c > +++ b/drivers/cpuidle/cpuidle-calxeda.c > @@ -75,5 +75,4 @@ static struct platform_driver calxeda_cpuidle_plat_driver = { > }, > .probe = calxeda_cpuidle_probe, > }; > - > -module_platform_driver(calxeda_cpuidle_plat_driver); > +builtin_platform_driver(calxeda_cpuidle_plat_driver); > diff --git a/drivers/cpuidle/cpuidle-zynq.c b/drivers/cpuidle/cpuidle-zynq.c > index 543292b1d38e..6f4257fc56e5 100644 > --- a/drivers/cpuidle/cpuidle-zynq.c > +++ b/drivers/cpuidle/cpuidle-zynq.c > @@ -73,5 +73,4 @@ static struct platform_driver zynq_cpuidle_driver = { > }, > .probe = zynq_cpuidle_probe, > }; > - > -module_platform_driver(zynq_cpuidle_driver); > +builtin_platform_driver(zynq_cpuidle_driver); > -- <http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs Follow Linaro: <http://www.facebook.com/pages/Linaro> Facebook | <http://twitter.com/#!/linaroorg> Twitter | <http://www.linaro.org/linaro-blog/> Blog
WARNING: multiple messages have this Message-ID (diff)
From: daniel.lezcano@linaro.org (Daniel Lezcano) To: linux-arm-kernel@lists.infradead.org Subject: [PATCH 3/7] drivers/cpuidle: Convert non-modular drivers to use builtin_driver_register Date: Mon, 11 May 2015 19:13:07 +0200 [thread overview] Message-ID: <5550E323.1080800@linaro.org> (raw) In-Reply-To: <1431287385-1526-4-git-send-email-paul.gortmaker@windriver.com> On 05/10/2015 09:49 PM, Paul Gortmaker wrote: > All these drivers are configured with Kconfig options that are > declared as bool. Hence it is not possible for the code > to be built as modular. However the code is currently using the > module_platform_driver() macro for driver registration. > > While this currently works, we really don't want to be including > the module.h header in non-modular code, which we'll be forced > to do, pending some upcoming code relocation from init.h into > module.h. So we fix it now by using the non-modular equivalent. > > Cc: "Rafael J. Wysocki" <rjw@rjwysocki.net> > Cc: Daniel Lezcano <daniel.lezcano@linaro.org> > Cc: Michal Simek <michal.simek@xilinx.com> > Cc: linux-pm at vger.kernel.org > Cc: linux-arm-kernel at lists.infradead.org > Signed-off-by: Paul Gortmaker <paul.gortmaker@windriver.com> Sounds reasonable. -- Daniel > --- > drivers/cpuidle/cpuidle-at91.c | 3 +-- > drivers/cpuidle/cpuidle-calxeda.c | 3 +-- > drivers/cpuidle/cpuidle-zynq.c | 3 +-- > 3 files changed, 3 insertions(+), 6 deletions(-) > > diff --git a/drivers/cpuidle/cpuidle-at91.c b/drivers/cpuidle/cpuidle-at91.c > index f2446c78d87c..9c5853b6ca4a 100644 > --- a/drivers/cpuidle/cpuidle-at91.c > +++ b/drivers/cpuidle/cpuidle-at91.c > @@ -62,5 +62,4 @@ static struct platform_driver at91_cpuidle_driver = { > }, > .probe = at91_cpuidle_probe, > }; > - > -module_platform_driver(at91_cpuidle_driver); > +builtin_platform_driver(at91_cpuidle_driver); > diff --git a/drivers/cpuidle/cpuidle-calxeda.c b/drivers/cpuidle/cpuidle-calxeda.c > index 9445e6cc02be..c13feec89ea1 100644 > --- a/drivers/cpuidle/cpuidle-calxeda.c > +++ b/drivers/cpuidle/cpuidle-calxeda.c > @@ -75,5 +75,4 @@ static struct platform_driver calxeda_cpuidle_plat_driver = { > }, > .probe = calxeda_cpuidle_probe, > }; > - > -module_platform_driver(calxeda_cpuidle_plat_driver); > +builtin_platform_driver(calxeda_cpuidle_plat_driver); > diff --git a/drivers/cpuidle/cpuidle-zynq.c b/drivers/cpuidle/cpuidle-zynq.c > index 543292b1d38e..6f4257fc56e5 100644 > --- a/drivers/cpuidle/cpuidle-zynq.c > +++ b/drivers/cpuidle/cpuidle-zynq.c > @@ -73,5 +73,4 @@ static struct platform_driver zynq_cpuidle_driver = { > }, > .probe = zynq_cpuidle_probe, > }; > - > -module_platform_driver(zynq_cpuidle_driver); > +builtin_platform_driver(zynq_cpuidle_driver); > -- <http://www.linaro.org/> Linaro.org ? Open source software for ARM SoCs Follow Linaro: <http://www.facebook.com/pages/Linaro> Facebook | <http://twitter.com/#!/linaroorg> Twitter | <http://www.linaro.org/linaro-blog/> Blog
next prev parent reply other threads:[~2015-05-11 17:13 UTC|newest] Thread overview: 49+ messages / expand[flat|nested] mbox.gz Atom feed top 2015-05-10 19:49 [PATCH 0/7] Introduce builtin_driver and use it for non-modular code Paul Gortmaker 2015-05-10 19:49 ` Paul Gortmaker 2015-05-10 19:49 ` Paul Gortmaker 2015-05-10 19:49 ` [PATCH 1/7] platform_device: better support builtin boilerplate avoidance Paul Gortmaker 2015-05-12 11:46 ` Linus Walleij 2015-05-28 0:50 ` Paul Gortmaker 2015-06-02 9:19 ` Linus Walleij 2015-06-03 14:07 ` Paul Gortmaker 2015-06-10 7:35 ` Linus Walleij 2015-05-10 19:49 ` [PATCH 2/7] drivers/platform: Convert non-modular pdev_bus to use builtin_driver_register Paul Gortmaker 2015-05-10 19:49 ` [PATCH 3/7] drivers/cpuidle: Convert non-modular drivers " Paul Gortmaker 2015-05-10 19:49 ` Paul Gortmaker 2015-05-10 19:49 ` Paul Gortmaker 2015-05-11 17:13 ` Daniel Lezcano [this message] 2015-05-11 17:13 ` Daniel Lezcano 2015-05-10 19:49 ` [PATCH 4/7] drivers/cpufreq: " Paul Gortmaker 2015-05-10 19:49 ` Paul Gortmaker 2015-05-10 19:49 ` Paul Gortmaker 2015-05-12 7:04 ` Viresh Kumar 2015-05-12 7:04 ` Viresh Kumar 2015-05-13 3:17 ` Paul Gortmaker 2015-05-13 3:17 ` Paul Gortmaker 2015-05-13 3:17 ` Paul Gortmaker 2015-06-03 20:59 ` Paul Gortmaker 2015-06-03 20:59 ` Paul Gortmaker 2015-06-03 20:59 ` Paul Gortmaker 2015-06-03 21:12 ` [PATCH v2] drivers/cpufreq: Convert non-modular s5pv210-cpufreq.c to use builtin_platform_driver Paul Gortmaker 2015-06-03 21:12 ` Paul Gortmaker 2015-06-03 21:12 ` Paul Gortmaker 2015-06-04 2:28 ` Viresh Kumar 2015-06-04 2:28 ` Viresh Kumar 2015-06-15 23:50 ` Rafael J. Wysocki 2015-06-15 23:50 ` Rafael J. Wysocki 2015-06-16 15:37 ` Paul Gortmaker 2015-06-16 15:37 ` Paul Gortmaker 2015-06-16 15:37 ` Paul Gortmaker 2015-05-10 19:49 ` [PATCH 5/7] drivers/soc: Convert non-modular tegra/pmc to use builtin_driver_register Paul Gortmaker 2015-05-10 19:49 ` Paul Gortmaker 2015-05-10 19:49 ` [PATCH 6/7] drivers/soc: Convert non-modular soc-realview " Paul Gortmaker 2015-05-12 11:39 ` Linus Walleij 2015-05-10 19:49 ` [PATCH 7/7] drivers/power: Convert non-modular syscon-reboot " Paul Gortmaker 2015-05-10 19:49 ` Paul Gortmaker 2015-05-23 17:55 ` Sebastian Reichel 2015-06-03 19:09 ` [PATCH] drivers/clk: convert sunxi/clk-mod0.c to use builtin_platform_driver Paul Gortmaker 2015-06-03 19:09 ` Paul Gortmaker 2015-06-04 16:21 ` Maxime Ripard 2015-06-04 16:21 ` Maxime Ripard 2015-06-04 22:50 ` Stephen Boyd 2015-06-04 22:50 ` Stephen Boyd
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=5550E323.1080800@linaro.org \ --to=daniel.lezcano@linaro.org \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-pm@vger.kernel.org \ --cc=michal.simek@xilinx.com \ --cc=paul.gortmaker@windriver.com \ --cc=rjw@rjwysocki.net \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.