summaryrefslogtreecommitdiffstats
path: root/drivers
diff options
context:
space:
mode:
authorPaul Mundt <lethal@linux-sh.org>2010-11-08 09:40:23 +0900
committerPaul Mundt <lethal@linux-sh.org>2010-11-08 09:40:23 +0900
commita766b29790b2b6582345624a6e9e686d8015efe1 (patch)
tree9ead3cd78f78ef0eb5b32dc8c332aeff4e91a578 /drivers
parentd0013c9e3bc75b3e1652bd5999a9a8d56a822ce4 (diff)
downloadlinux-a766b29790b2b6582345624a6e9e686d8015efe1.tar.gz
linux-a766b29790b2b6582345624a6e9e686d8015efe1.tar.bz2
linux-a766b29790b2b6582345624a6e9e686d8015efe1.zip
sh: clkfwk: Fix up checkpatch warnings.
The clk_round_parent() change introduced various checkpatch warnings, tidy them up. Signed-off-by: Paul Mundt <lethal@linux-sh.org>
Diffstat (limited to 'drivers')
-rw-r--r--drivers/sh/clk/core.c27
1 files changed, 22 insertions, 5 deletions
diff --git a/drivers/sh/clk/core.c b/drivers/sh/clk/core.c
index b3840597ad6e..09615b51d591 100644
--- a/drivers/sh/clk/core.c
+++ b/drivers/sh/clk/core.c
@@ -561,57 +561,74 @@ long clk_round_parent(struct clk *clk, unsigned long target,
continue;
if (unlikely(freq->frequency / target <= div_min - 1)) {
- unsigned long freq_max = (freq->frequency + div_min / 2) / div_min;
+ unsigned long freq_max;
+
+ freq_max = (freq->frequency + div_min / 2) / div_min;
if (error > target - freq_max) {
error = target - freq_max;
best = freq;
if (best_freq)
*best_freq = freq_max;
}
- pr_debug("too low freq %lu, error %lu\n", freq->frequency, target - freq_max);
+
+ pr_debug("too low freq %lu, error %lu\n", freq->frequency,
+ target - freq_max);
+
if (!error)
break;
+
continue;
}
if (unlikely(freq->frequency / target >= div_max)) {
- unsigned long freq_min = (freq->frequency + div_max / 2) / div_max;
+ unsigned long freq_min;
+
+ freq_min = (freq->frequency + div_max / 2) / div_max;
if (error > freq_min - target) {
error = freq_min - target;
best = freq;
if (best_freq)
*best_freq = freq_min;
}
- pr_debug("too high freq %lu, error %lu\n", freq->frequency, freq_min - target);
+
+ pr_debug("too high freq %lu, error %lu\n", freq->frequency,
+ freq_min - target);
+
if (!error)
break;
+
continue;
}
-
div = freq->frequency / target;
freq_high = freq->frequency / div;
freq_low = freq->frequency / (div + 1);
+
if (freq_high - target < error) {
error = freq_high - target;
best = freq;
if (best_freq)
*best_freq = freq_high;
}
+
if (target - freq_low < error) {
error = target - freq_low;
best = freq;
if (best_freq)
*best_freq = freq_low;
}
+
pr_debug("%u / %lu = %lu, / %lu = %lu, best %lu, parent %u\n",
freq->frequency, div, freq_high, div + 1, freq_low,
*best_freq, best->frequency);
+
if (!error)
break;
}
+
if (parent_freq)
*parent_freq = best->frequency;
+
return error;
}
EXPORT_SYMBOL_GPL(clk_round_parent);