* [PATCH] cpufreq: integrator: Fix module autoload for OF platform driver
@ 2015-09-16 20:24 Luis de Bethencourt
2015-09-17 1:57 ` Viresh Kumar
0 siblings, 1 reply; 3+ messages in thread
From: Luis de Bethencourt @ 2015-09-16 20:24 UTC (permalink / raw)
To: linux-kernel; +Cc: Rafael J. Wysocki, Viresh Kumar, linux-pm
This platform driver has a OF device ID table but the OF module
alias information is not created so module autoloading won't work.
Signed-off-by: Luis de Bethencourt <luisbg@osg.samsung.com>
---
Hello,
This patch adds the missing MODULE_DEVICE_TABLE() for OF to export
that information so modules have the correct aliases built-in and
autoloading works correctly.
A longer explanation by Javier Canillas can be found here:
https://lkml.org/lkml/2015/7/30/519
Thanks,
Luis
drivers/cpufreq/integrator-cpufreq.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/cpufreq/integrator-cpufreq.c b/drivers/cpufreq/integrator-cpufreq.c
index 2faa421..79e3ff2 100644
--- a/drivers/cpufreq/integrator-cpufreq.c
+++ b/drivers/cpufreq/integrator-cpufreq.c
@@ -221,6 +221,8 @@ static const struct of_device_id integrator_cpufreq_match[] = {
{ },
};
+MODULE_DEVICE_TABLE(of, integrator_cpufreq_match);
+
static struct platform_driver integrator_cpufreq_driver = {
.driver = {
.name = "integrator-cpufreq",
--
2.4.6
^ permalink raw reply related [flat|nested] 3+ messages in thread* Re: [PATCH] cpufreq: integrator: Fix module autoload for OF platform driver
2015-09-16 20:24 [PATCH] cpufreq: integrator: Fix module autoload for OF platform driver Luis de Bethencourt
@ 2015-09-17 1:57 ` Viresh Kumar
2015-10-05 20:51 ` Rafael J. Wysocki
0 siblings, 1 reply; 3+ messages in thread
From: Viresh Kumar @ 2015-09-17 1:57 UTC (permalink / raw)
To: Luis de Bethencourt; +Cc: linux-kernel, Rafael J. Wysocki, linux-pm
On 16-09-15, 22:24, Luis de Bethencourt wrote:
> This platform driver has a OF device ID table but the OF module
> alias information is not created so module autoloading won't work.
>
> Signed-off-by: Luis de Bethencourt <luisbg@osg.samsung.com>
> ---
>
> Hello,
>
> This patch adds the missing MODULE_DEVICE_TABLE() for OF to export
> that information so modules have the correct aliases built-in and
> autoloading works correctly.
>
> A longer explanation by Javier Canillas can be found here:
> https://lkml.org/lkml/2015/7/30/519
>
> Thanks,
> Luis
>
> drivers/cpufreq/integrator-cpufreq.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/drivers/cpufreq/integrator-cpufreq.c b/drivers/cpufreq/integrator-cpufreq.c
> index 2faa421..79e3ff2 100644
> --- a/drivers/cpufreq/integrator-cpufreq.c
> +++ b/drivers/cpufreq/integrator-cpufreq.c
> @@ -221,6 +221,8 @@ static const struct of_device_id integrator_cpufreq_match[] = {
> { },
> };
>
> +MODULE_DEVICE_TABLE(of, integrator_cpufreq_match);
> +
> static struct platform_driver integrator_cpufreq_driver = {
> .driver = {
> .name = "integrator-cpufreq",
Acked-by: Viresh Kumar <viresh.kumar@linaro.org>
--
viresh
^ permalink raw reply [flat|nested] 3+ messages in thread* Re: [PATCH] cpufreq: integrator: Fix module autoload for OF platform driver
2015-09-17 1:57 ` Viresh Kumar
@ 2015-10-05 20:51 ` Rafael J. Wysocki
0 siblings, 0 replies; 3+ messages in thread
From: Rafael J. Wysocki @ 2015-10-05 20:51 UTC (permalink / raw)
To: Viresh Kumar; +Cc: Luis de Bethencourt, linux-kernel, linux-pm
On Thursday, September 17, 2015 07:27:56 AM Viresh Kumar wrote:
> On 16-09-15, 22:24, Luis de Bethencourt wrote:
> > This platform driver has a OF device ID table but the OF module
> > alias information is not created so module autoloading won't work.
> >
> > Signed-off-by: Luis de Bethencourt <luisbg@osg.samsung.com>
> > ---
> >
> > Hello,
> >
> > This patch adds the missing MODULE_DEVICE_TABLE() for OF to export
> > that information so modules have the correct aliases built-in and
> > autoloading works correctly.
> >
> > A longer explanation by Javier Canillas can be found here:
> > https://lkml.org/lkml/2015/7/30/519
> >
> > Thanks,
> > Luis
> >
> > drivers/cpufreq/integrator-cpufreq.c | 2 ++
> > 1 file changed, 2 insertions(+)
> >
> > diff --git a/drivers/cpufreq/integrator-cpufreq.c b/drivers/cpufreq/integrator-cpufreq.c
> > index 2faa421..79e3ff2 100644
> > --- a/drivers/cpufreq/integrator-cpufreq.c
> > +++ b/drivers/cpufreq/integrator-cpufreq.c
> > @@ -221,6 +221,8 @@ static const struct of_device_id integrator_cpufreq_match[] = {
> > { },
> > };
> >
> > +MODULE_DEVICE_TABLE(of, integrator_cpufreq_match);
> > +
> > static struct platform_driver integrator_cpufreq_driver = {
> > .driver = {
> > .name = "integrator-cpufreq",
>
> Acked-by: Viresh Kumar <viresh.kumar@linaro.org>
Applied, thanks!
Rafael
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2015-10-05 20:23 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-09-16 20:24 [PATCH] cpufreq: integrator: Fix module autoload for OF platform driver Luis de Bethencourt
2015-09-17 1:57 ` Viresh Kumar
2015-10-05 20:51 ` Rafael J. Wysocki
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).