path: root/drivers/clk
diff options
authorGregory CLEMENT <gregory.clement@bootlin.com>2018-07-13 12:27:26 +0200
committerStephen Boyd <sboyd@kernel.org>2018-07-25 09:42:50 -0700
commit616bf80d381da13fbb392ebff06f46f946e3ee84 (patch)
treea15ff5b335b690e203dfb5dbd6bd8302236abaf0 /drivers/clk
parentce397d215ccd07b8ae3f71db689aedb85d56ab40 (diff)
clk: mvebu: armada-37xx-periph: Fix wrong return value in get_parent
The return value of the get_parent operation is a u8, whereas a -EINVAL was returned. This wrong value was return if the value was bigger that the number of parent but this case was already handled by the core. So we can just remove this chunk of code to fix the issue. Reported-by: Dan Carpenter <dan.carpenter@oracle.com> Fixes: 9818a7a4fd10 ("clk: mvebu: armada-37xx-periph: prepare cpu clk to be used with DVFS") Signed-off-by: Gregory CLEMENT <gregory.clement@bootlin.com> Signed-off-by: Stephen Boyd <sboyd@kernel.org>
Diffstat (limited to 'drivers/clk')
1 files changed, 0 insertions, 3 deletions
diff --git a/drivers/clk/mvebu/armada-37xx-periph.c b/drivers/clk/mvebu/armada-37xx-periph.c
index 6860bd5a37c5..5cf0a52b87e8 100644
--- a/drivers/clk/mvebu/armada-37xx-periph.c
+++ b/drivers/clk/mvebu/armada-37xx-periph.c
@@ -428,9 +428,6 @@ static u8 clk_pm_cpu_get_parent(struct clk_hw *hw)
val &= pm_cpu->mask_mux;
- if (val >= num_parents)
- return -EINVAL;
return val;