[PATCH v2] ARM: ux500: send cpufreq notification for all cpus

[PATCH v2] ARM: ux500: send cpufreq notification for all cpus

[Vincent Guittot]

The same clock is used for all cpus so we must notify the frequency change
for each one in order to update the configuration of all twd clockevents.

change since V1:
* use policy->cpus instead of cpu_online_mask

Signed-off-by: Vincent Guittot <vincent.guittot@linaro.org>
---
 drivers/cpufreq/db8500-cpufreq.c |    7 ++++---
 1 files changed, 4 insertions(+), 3 deletions(-)

diff --git a/drivers/cpufreq/db8500-cpufreq.c b/drivers/cpufreq/db8500-cpufreq.c
index 0d8dd1c..b893f9b 100644
--- a/drivers/cpufreq/db8500-cpufreq.c
+++ b/drivers/cpufreq/db8500-cpufreq.c
@@ -76,13 +76,13 @@ static int db8500_cpufreq_target(struct cpufreq_policy *policy,
 
 	freqs.old = policy->cur;
 	freqs.new = freq_table[idx].frequency;
-	freqs.cpu = policy->cpu;
 
 	if (freqs.old == freqs.new)
 		return 0;
 
 	/* pre-change notification */
-	cpufreq_notify_transition(&freqs, CPUFREQ_PRECHANGE);
+	for_each_cpu(freqs.cpu, policy->cpus)
+		cpufreq_notify_transition(&freqs, CPUFREQ_PRECHANGE);
 
 	/* request the PRCM unit for opp change */
 	if (prcmu_set_arm_opp(idx2opp[idx])) {
@@ -91,7 +91,8 @@ static int db8500_cpufreq_target(struct cpufreq_policy *policy,
 	}
 
 	/* post change notification */
-	cpufreq_notify_transition(&freqs, CPUFREQ_POSTCHANGE);
+	for_each_cpu(freqs.cpu, policy->cpus)
+		cpufreq_notify_transition(&freqs, CPUFREQ_POSTCHANGE);
 
 	return 0;
 }
-- 
1.7.4.1

[PATCH v2] ARM: ux500: send cpufreq notification for all cpus

[Linus Walleij]

On Thu, Aug 25, 2011 at 8:31 AM, Vincent Guittot
<vincent.guittot@linaro.org> wrote:

> The same clock is used for all cpus so we must notify the frequency change
> for each one in order to update the configuration of all twd clockevents.
>
> change since V1:
> * use policy->cpus instead of cpu_online_mask
>
> Signed-off-by: Vincent Guittot <vincent.guittot@linaro.org>

Acked-by: Linus Walleij <linus.walleij@linaro.org>

Dave, are you picking this up?

Yours,
Linus Walleij

[PATCH v2] ARM: ux500: send cpufreq notification for all cpus

[Dave Jones]

On Mon, Aug 29, 2011 at 11:44:31AM +0200, Linus Walleij wrote:
 > On Thu, Aug 25, 2011 at 8:31 AM, Vincent Guittot
 > <vincent.guittot@linaro.org> wrote:
 > 
 > > The same clock is used for all cpus so we must notify the frequency change
 > > for each one in order to update the configuration of all twd clockevents.
 > >
 > > change since V1:
 > > * use policy->cpus instead of cpu_online_mask
 > >
 > > Signed-off-by: Vincent Guittot <vincent.guittot@linaro.org>
 > 
 > Acked-by: Linus Walleij <linus.walleij@linaro.org>
 > 
 > Dave, are you picking this up?

yeah, I have it queued. I'll get to it today.

thanks,

	Dave