clk: baikal-t1: remove redundant assignment to variable 'divider'
authorColin Ian King <colin.king@canonical.com>
Tue, 2 Jun 2020 17:24:35 +0000 (18:24 +0100)
committerStephen Boyd <sboyd@kernel.org>
Tue, 9 Jun 2020 20:45:37 +0000 (13:45 -0700)
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>
drivers/clk/baikal-t1/ccu-div.c

index bd40f5936f0828c48c041b79df8b9ae25fdc3e31..4062092d67f905a601e1d8d0856175eaeee0240c 100644 (file)
@@ -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);