Hello Rafael, On Fri, Feb 05, 2016 at 03:11:54AM +0100, Rafael J. Wysocki wrote: [..snip..] > Index: linux-pm/drivers/cpufreq/cpufreq_performance.c > =================================================================== > --- linux-pm.orig/drivers/cpufreq/cpufreq_performance.c > +++ linux-pm/drivers/cpufreq/cpufreq_performance.c > @@ -33,10 +33,7 @@ static int cpufreq_governor_performance( > return 0; > } > > -#ifdef CONFIG_CPU_FREQ_GOV_PERFORMANCE_MODULE > -static > -#endif > -struct cpufreq_governor cpufreq_gov_performance = { > +static struct cpufreq_governor cpufreq_gov_performance = { > .name = "performance", > .governor = cpufreq_governor_performance, > .owner = THIS_MODULE, > @@ -52,6 +49,19 @@ static void __exit cpufreq_gov_performan > cpufreq_unregister_governor(&cpufreq_gov_performance); > } > > +#ifdef CONFIG_CPU_FREQ_DEFAULT_GOV_PERFORMANCE > +struct cpufreq_governor *cpufreq_default_governor(void) > +{ > + return &cpufreq_gov_performance; > +} > +#endif > +#ifndef CONFIG_CPU_FREQ_GOV_PERFORMANCE_MODULE
Shouldn't this be #ifdef ? > +struct cpufreq_governor *cpufreq_fallback_governor(void) > +{ > + return &cpufreq_gov_performance; > +} > +#endif -- Thanks and Regards gautham.