aboutsummaryrefslogtreecommitdiffstats
path: root/drivers/cpufreq/spear-cpufreq.c
diff options
context:
space:
mode:
authorPaul Walmsley <pwalmsley@nvidia.com>2013-11-25 21:04:34 -0500
committerRafael J. Wysocki <rafael.j.wysocki@intel.com>2013-12-06 17:19:19 -0500
commit189c9b8d925691cc58823ffdc3bf75dc8630ccdf (patch)
tree91545c503f2a3e44ce04ca83db745964c8b1ed0d /drivers/cpufreq/spear-cpufreq.c
parent2209b0c964c7c402b07d0df6fcfbecf8ff53db30 (diff)
cpufreq: SPEAr: clk_round_rate() can return a zero upon error
Treat both negative and zero return values from clk_round_rate() as errors. This is needed since subsequent patches will convert clk_round_rate()'s return value to be an unsigned type, rather than a signed type, since some clock sources can generate rates higher than (2^31)-1 Hz. Eventually, when calling clk_round_rate(), only a return value of zero will be considered a error. All other values will be considered valid rates. The comparison against values less than 0 is kept to preserve the correct behavior in the meantime. Signed-off-by: Paul Walmsley <pwalmsley@nvidia.com> Acked-by: Viresh Kumar <viresh.kumar@linaro.org> Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
Diffstat (limited to 'drivers/cpufreq/spear-cpufreq.c')
-rw-r--r--drivers/cpufreq/spear-cpufreq.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/cpufreq/spear-cpufreq.c b/drivers/cpufreq/spear-cpufreq.c
index d02ccd19c9c4..45ea4c094542 100644
--- a/drivers/cpufreq/spear-cpufreq.c
+++ b/drivers/cpufreq/spear-cpufreq.c
@@ -138,7 +138,7 @@ static int spear_cpufreq_target(struct cpufreq_policy *policy,
138 } 138 }
139 139
140 newfreq = clk_round_rate(srcclk, newfreq * mult); 140 newfreq = clk_round_rate(srcclk, newfreq * mult);
141 if (newfreq < 0) { 141 if (newfreq <= 0) {
142 pr_err("clk_round_rate failed for cpu src clock\n"); 142 pr_err("clk_round_rate failed for cpu src clock\n");
143 return newfreq; 143 return newfreq;
144 } 144 }