On 04/25/2014 02:07 PM, Viresh Kumar wrote: > On 25 April 2014 13:48, Srivatsa S. Bhat > <srivatsa.bhat@xxxxxxxxxxxxxxxxxx> wrote: > >> diff --git a/drivers/cpufreq/longhaul.c b/drivers/cpufreq/longhaul.c > >> @@ -269,8 +269,6 @@ static void longhaul_setstate(struct cpufreq_policy *policy, > > This routine has this code as well: > > mult = mults[mults_index & 0x1f]; > if (mult == -1) > return; > speed = calc_speed(mult); > if ((speed > highest_speed) || (speed < lowest_speed)) > return; > > And so it might return without changing frequency and that's why I > left this driver earlier for this change.. So, please return -EINVAL > from here.. > Ok, I'll address this in the next version. Thanks for pointing this out! Regards, Srivatsa S. Bhat -- 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
