summaryrefslogtreecommitdiffstats
path: root/drivers/clk
diff options
context:
space:
mode:
authorMaxime Ripard <maxime.ripard@free-electrons.com>2016-05-11 20:57:44 +0200
committerMaxime Ripard <maxime.ripard@free-electrons.com>2016-06-10 11:49:15 +0200
commit4de2d58bc973caa8988b44ddd11787e57051c843 (patch)
treee2568a9df7272d15d3583eb0a820f336065f2bf9 /drivers/clk
parent1a695a905c18548062509178b98bc91e67510864 (diff)
downloadop-kernel-dev-4de2d58bc973caa8988b44ddd11787e57051c843.zip
op-kernel-dev-4de2d58bc973caa8988b44ddd11787e57051c843.tar.gz
clk: sunxi: tcon-ch1: Do not return a negative error in get_parent
get_parent is supposed to return an unsigned 8 bit integer, so returning -EINVAL is a bad idea. Remove it. Reported-by: Dan Carpenter <dan.carpenter@oracle.com> Acked-by: Chen-Yu Tsai <wens@csie.org> Signed-off-by: Maxime Ripard <maxime.ripard@free-electrons.com>
Diffstat (limited to 'drivers/clk')
-rw-r--r--drivers/clk/sunxi/clk-sun4i-tcon-ch1.c3
1 files changed, 0 insertions, 3 deletions
diff --git a/drivers/clk/sunxi/clk-sun4i-tcon-ch1.c b/drivers/clk/sunxi/clk-sun4i-tcon-ch1.c
index 98a4582..2485852 100644
--- a/drivers/clk/sunxi/clk-sun4i-tcon-ch1.c
+++ b/drivers/clk/sunxi/clk-sun4i-tcon-ch1.c
@@ -85,9 +85,6 @@ static u8 tcon_ch1_get_parent(struct clk_hw *hw)
reg = readl(tclk->reg) >> TCON_CH1_SCLK2_MUX_SHIFT;
reg &= reg >> TCON_CH1_SCLK2_MUX_MASK;
- if (reg >= num_parents)
- return -EINVAL;
-
return reg;
}
OpenPOWER on IntegriCloud