On Wed, Feb 6, 2013 at 10:32 PM, <dirk.brandewie@xxxxxxxxx> wrote: > From: Dirk Brandewie <dirk.brandewie@xxxxxxxxx> > > Scaling drivers that implement internal governors do not have governor > structures assocaited with them. Only track the name of the governor > associated with the CPU if the driver does not implement > cpufreq_driver.setpolicy() > > Signed-off-by: Dirk Brandewie <dirk.j.brandewie@xxxxxxxxx> > Acked-by: Viresh Kumar <viresh.kumar@xxxxxxxxxx> To fix sparse warning updated this patch as: commit ea241d95152c535c3d261f3a2240c45480b040bc Author: Viresh Kumar <viresh.kumar@xxxxxxxxxx> Date: Thu Feb 7 19:13:43 2013 +0530 fixup! cpufreq: Do not track governor name for scaling drivers with internal governors. --- drivers/cpufreq/cpufreq.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/cpufreq/cpufreq.c b/drivers/cpufreq/cpufreq.c index e140b81..819aa33 100644 --- a/drivers/cpufreq/cpufreq.c +++ b/drivers/cpufreq/cpufreq.c @@ -1039,7 +1039,7 @@ static int __cpufreq_remove_dev(struct device *dev, struct subsys_interface *sif __cpufreq_governor(data, CPUFREQ_GOV_STOP); #ifdef CONFIG_HOTPLUG_CPU - if (!cpufreq_driver->setpolicy) + if (!driver->setpolicy) strncpy(per_cpu(cpufreq_cpu_governor, cpu), data->governor->name, CPUFREQ_NAME_LEN); #endif -- To unsubscribe from this list: send the line "unsubscribe cpufreq" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html