summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorColin Ian King <colin.king@canonical.com>2020-06-02 18:24:35 +0100
committerStephen Boyd <sboyd@kernel.org>2020-06-09 13:45:37 -0700
commit45edc7e27445f4d467d915c0e1ef0a7b41270f47 (patch)
treec2c1a5309512435b2586a71b65655dbbfc737bcd
parent8ae91d36ac1a16debb39e372f2a81278bb5b4830 (diff)
clk: baikal-t1: remove redundant assignment to variable 'divider'
The variable divider is being initialized with a value that is never read and it is being updated later with a new value. The initialization is redundant and can be removed. Addresses-Coverity: ("Unused value") Signed-off-by: Colin Ian King <colin.king@canonical.com> Link: https://lore.kernel.org/r/20200602172435.70282-1-colin.king@canonical.com Reviewed-by: Serge Semin <fancer.lancer@gmail.com> Signed-off-by: Stephen Boyd <sboyd@kernel.org>
-rw-r--r--drivers/clk/baikal-t1/ccu-div.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/clk/baikal-t1/ccu-div.c b/drivers/clk/baikal-t1/ccu-div.c
index bd40f5936f08..4062092d67f9 100644
--- a/drivers/clk/baikal-t1/ccu-div.c
+++ b/drivers/clk/baikal-t1/ccu-div.c
@@ -248,7 +248,7 @@ static int ccu_div_var_set_rate_fast(struct clk_hw *hw, unsigned long rate,
unsigned long parent_rate)
{
struct ccu_div *div = to_ccu_div(hw);
- unsigned long flags, divider = 1;
+ unsigned long flags, divider;
u32 val;
divider = ccu_div_var_calc_divider(rate, parent_rate, div->mask);