Re: [PATCH 2/2] cpufreq: db8500: set CPUFREQ_CONST_LOOPS
From: Stephen Boyd <hidden>
Date: 2012-12-17 18:24:22
Also in:
linux-arm-kernel, lkml
On 12/04/12 02:10, Fabio Baltieri wrote:
quoted hunk ↗ jump to hunk
As ux500 is being converted to timer based delay loops, and the timer used is not depending on CPUs clock frequency, set cpufreq_driver flag CPUFREQ_CONST_LOOPS to prevent cpufreq rescaling loops_for_jiffies. Acked-by: Linus Walleij <redacted> Signed-off-by: Fabio Baltieri <redacted> --- drivers/cpufreq/db8500-cpufreq.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)diff --git a/drivers/cpufreq/db8500-cpufreq.c b/drivers/cpufreq/db8500-cpufreq.c index 74b830b..e758891 100644 --- a/drivers/cpufreq/db8500-cpufreq.c +++ b/drivers/cpufreq/db8500-cpufreq.c@@ -150,7 +150,7 @@ static int __cpuinit db8500_cpufreq_init(struct cpufreq_policy *policy) } static struct cpufreq_driver db8500_cpufreq_driver = { - .flags = CPUFREQ_STICKY, + .flags = CPUFREQ_STICKY | CPUFREQ_CONST_LOOPS, .verify = db8500_cpufreq_verify_speed, .target = db8500_cpufreq_target, .get = db8500_cpufreq_getspeed,
Wouldn't you want to fold this into the previous patch so bisection is not broken? Otherwise you have a place where lpj is all screwed up when cpufreq mucks with it. -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by The Linux Foundation