On Fri, May 16, 2014 at 3:10 AM, Viresh Kumar viresh.kumar@linaro.org wrote:
On 16 May 2014 15:39, Thomas Abraham ta.omasab@gmail.com 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