On Thu, Mar 3, 2016 at 12:18 PM, Peter Zijlstra <pet...@infradead.org> wrote: > On Wed, Mar 02, 2016 at 03:12:33AM +0100, Rafael J. Wysocki wrote: >> +void cpufreq_driver_fast_switch(struct cpufreq_policy *policy, >> + unsigned int target_freq, unsigned int >> relation) >> +{ >> + unsigned int freq; >> + >> + if (target_freq == policy->cur) >> + return; > > But what if relation is different from last time? ;-)
Doh Never mind, I'll drop this check (that said this mistake is present elsewhere too IIRC).