Re: [PATCH V2 0/3] cpufreq: add support for intermediate (stable)

From: Doug Anderson
Date: Fri May 16 2014 - 11:20:53 EST


On Fri, May 16, 2014 at 3:10 AM, Viresh Kumar <viresh.kumar@xxxxxxxxxx> wrote:
> On 16 May 2014 15:39, Thomas Abraham <ta.omasab@xxxxxxxxx> wrote:
>> In the given example above, the reworked implementation of cpufreq for
>> exynos maintains the transition frequency at 800MHz / 4 = 200MHz by
>> using a clock divider. So the transition frequency is ensured to be
>> less than or equal to the pre-transition cpu clock frequency.
>
> Thanks for the information. But I think this patchset is still required for
> many platforms. Anyway these extra notifications must be sent for exynos
> as well.

Right, so I think on exynos no functionality will be broken once
Thomas's cpufreq-cpu0 change lands (udelay will only run long, never
short). ...but from the purist standpoint we will be transitioning
from 1.6 GHz => 800 MHz => 1.7 GHz without any notification about the
800 MHz. You could imagine someone registering for cpufreq
notifications that would care about the 800MHz transition.

...so it seems like we could wait for Thomas's patches to land as-is
(since they make things better) and then atop that see about adding
support for intermediate frequencies to cpufreq-cpu0.

-Doug
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/