projects
/
linux-2.6-microblaze.git
/ commitdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
| commitdiff |
tree
raw
|
patch
| inline |
side by side
(parent:
3bf639a
)
clk: at91: sam9x60-pll: use logical or for range check
author
Claudiu Beznea
<claudiu.beznea@microchip.com>
Wed, 22 Jul 2020 07:38:12 +0000
(10:38 +0300)
committer
Stephen Boyd
<sboyd@kernel.org>
Fri, 24 Jul 2020 09:18:47 +0000
(
02:18
-0700)
Use logical or for range check. In case bestrate is not in
characteristics->output[0].min..characteristics->output[0].max
range we should return -ERANGE.
Fixes:
a436c2a447e59
("clk: at91: add sam9x60 PLL driver")
Signed-off-by: Claudiu Beznea <claudiu.beznea@microchip.com>
Reviewed-by: Alexandre Belloni <alexandre.belloni@bootlin.com>
Link:
https://lore.kernel.org/r/1595403506-8209-5-git-send-email-claudiu.beznea@microchip.com
Signed-off-by: Stephen Boyd <sboyd@kernel.org>
drivers/clk/at91/clk-sam9x60-pll.c
patch
|
blob
|
history
diff --git
a/drivers/clk/at91/clk-sam9x60-pll.c
b/drivers/clk/at91/clk-sam9x60-pll.c
index
3522eae
..
4b7b6c4
100644
(file)
--- a/
drivers/clk/at91/clk-sam9x60-pll.c
+++ b/
drivers/clk/at91/clk-sam9x60-pll.c
@@
-231,7
+231,7
@@
static long sam9x60_pll_get_best_div_mul(struct sam9x60_pll *pll,
}
/* Check if bestrate is a valid output rate */
- if (bestrate < characteristics->output[0].min
&&
+ if (bestrate < characteristics->output[0].min
||
bestrate > characteristics->output[0].max)
return -ERANGE;