On Thu, Mar 3, 2016 at 12:18 PM, Peter Zijlstra [off-list ref] wrote:

On Wed, Mar 02, 2016 at 03:12:33AM +0100, Rafael J. Wysocki wrote:

quoted

+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).