On 13-12-17, 14:33, Dave Gerlach wrote: > ti-cpufreq will be responsible for calling dev_pm_opp_set_regulators on > platforms that require AVS and ABB regulator support so we must be > able to defer probe if regulators are not yet available, so change > ti-cpufreq to be a module_platform_driver to allow for probe defer. > > Signed-off-by: Dave Gerlach <d-gerlach@xxxxxx> > --- > drivers/cpufreq/ti-cpufreq.c | 23 +++++++++++++++++++++-- > 1 file changed, 21 insertions(+), 2 deletions(-) > > diff --git a/drivers/cpufreq/ti-cpufreq.c b/drivers/cpufreq/ti-cpufreq.c > index 923317f03b4b..b1c230a1e2aa 100644 > --- a/drivers/cpufreq/ti-cpufreq.c > +++ b/drivers/cpufreq/ti-cpufreq.c > @@ -17,6 +17,7 @@ > #include <linux/cpu.h> > #include <linux/io.h> > #include <linux/mfd/syscon.h> > +#include <linux/module.h> > #include <linux/init.h> > #include <linux/of.h> > #include <linux/of_platform.h> > @@ -195,7 +196,7 @@ static const struct of_device_id ti_cpufreq_of_match[] = { > {}, > }; > > -static int ti_cpufreq_init(void) > +static int ti_cpufreq_probe(struct platform_device *pdev) > { > u32 version[VERSION_COUNT]; > struct device_node *np; > @@ -269,4 +270,22 @@ static int ti_cpufreq_init(void) > > return ret; > } > -device_initcall(ti_cpufreq_init); > + > +static int ti_cpufreq_init(void) > +{ > + platform_device_register_simple("ti-cpufreq", -1, NULL, 0); > + return 0; > +} > +module_init(ti_cpufreq_init); > + > +static struct platform_driver ti_cpufreq_driver = { > + .probe = ti_cpufreq_probe, > + .driver = { > + .name = "ti-cpufreq", > + }, > +}; > +module_platform_driver(ti_cpufreq_driver); > + > +MODULE_DESCRIPTION("TI CPUFreq/OPP hw-supported driver"); > +MODULE_AUTHOR("Dave Gerlach <d-gerlach@xxxxxx>"); > +MODULE_LICENSE("GPL v2"); Acked-by: Viresh Kumar <viresh.kumar@xxxxxxxxxx> -- viresh -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html