Eduardo Valentin had written, on 03/24/2010 06:02 AM, the following:
From: Eduardo Valentin <eduardo.valentin@xxxxxxxxx>
This patch fix the sequence inside omap_target while setting
a frequency for OMAP3 devices.
Previously any frequency was set, even if out of range. Now
the range set by the governor is also checked for OMAP3.
Signed-off-by: Eduardo Valentin <eduardo.valentin@xxxxxxxxx>
Good catch. thanks.
Acked-by: Nishanth Menon <nm@xxxxxx>
---
arch/arm/plat-omap/cpu-omap.c | 3 ++-
1 files changed, 2 insertions(+), 1 deletions(-)
diff --git a/arch/arm/plat-omap/cpu-omap.c b/arch/arm/plat-omap/cpu-omap.c
index a69b557..0674405 100644
--- a/arch/arm/plat-omap/cpu-omap.c
+++ b/arch/arm/plat-omap/cpu-omap.c
@@ -88,7 +88,7 @@ static int omap_target(struct cpufreq_policy *policy,
struct cpufreq_freqs freqs;
#endif
#if defined(CONFIG_ARCH_OMAP3) && !defined(CONFIG_OMAP_PM_NONE)
- unsigned long freq = target_freq * 1000;
+ unsigned long freq;
#endif
int ret = 0;
@@ -114,6 +114,7 @@ static int omap_target(struct cpufreq_policy *policy,
ret = clk_set_rate(mpu_clk, freqs.new * 1000);
cpufreq_notify_transition(&freqs, CPUFREQ_POSTCHANGE);
#elif defined(CONFIG_ARCH_OMAP3) && !defined(CONFIG_OMAP_PM_NONE)
+ freq = target_freq * 1000;
if (opp_find_freq_ceil(OPP_MPU, &freq))
omap_pm_cpu_set_freq(freq);
#endif
--
Regards,
Nishanth Menon
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
