On Wed, Jul 31, 2013 at 07:49:11PM +0530, Viresh Kumar wrote: > CPUFreq core does following at multiple places: > module_{get|put}(cpufreq_driver->owner)). > > This is done to make sure module doesn't get unloaded if it is currently in > use. > This will work only if the .owner field of cpufreq driver is initialized with > a > valid pointer. > > This field wasn't initialized for this driver, lets initialize it with > THIS_MODULE. > > Cc: Shawn Guo <shawn....@linaro.org> > Signed-off-by: Viresh Kumar <viresh.ku...@linaro.org>
For this and imx6q-cpufreq one, Acked-by: Shawn Guo <shawn....@linaro.org> > --- > drivers/cpufreq/cpufreq-cpu0.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/cpufreq/cpufreq-cpu0.c b/drivers/cpufreq/cpufreq-cpu0.c > index ad1fde2..a087bf1 100644 > --- a/drivers/cpufreq/cpufreq-cpu0.c > +++ b/drivers/cpufreq/cpufreq-cpu0.c > @@ -169,6 +169,7 @@ static struct cpufreq_driver cpu0_cpufreq_driver = { > .init = cpu0_cpufreq_init, > .exit = cpu0_cpufreq_exit, > .name = "generic_cpu0", > + .owner = THIS_MODULE, > .attr = cpu0_cpufreq_attr, > }; > > -- > 1.7.12.rc2.18.g61b472e > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/