On 27 June 2013 21:25, Lukasz Majewski <l.majewski@xxxxxxxxxxx> wrote: > On Wed, 26 Jun 2013 16:24:32 +0530, Viresh Kumar wrote: >> > + if (boost_enabled != state) { >> > + write_lock_irqsave(&cpufreq_driver_lock, flags); >> > + boost_enabled = state; >> > + if (cpufreq_driver->enable_boost) >> > + ret = cpufreq_driver->enable_boost(state); >> > + else >> > + ret = cpufreq_boost_trigger_state_sw(); > > I will use only one call to cpufreq_driver->enable_boost(state) [*] with > either cpufreq_boost_enable_sw() (function with SW boost handling) or > the one provided by cpufreq driver. > > Only when cpufreq driver doesn't provide [*], it will be filled with > "default" cpufreq_boost_enable_sw(). I didn't get it completely. You are saying you will send a function pointer now? -- 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