clk: Si5341/Si5340: remove redundant assignment to n_den
authorColin Ian King <colin.king@canonical.com>
Mon, 1 Jul 2019 16:50:20 +0000 (17:50 +0100)
committerStephen Boyd <sboyd@kernel.org>
Wed, 7 Aug 2019 21:23:24 +0000 (14:23 -0700)
The variable n_den is initialized however that value is never read
as n_den is re-assigned a little later in the two paths of a
following if-statement.  Remove the redundant assignment.

Addresses-Coverity: ("Unused value")
Signed-off-by: Colin Ian King <colin.king@canonical.com>
Link: https://lkml.kernel.org/r/20190701165020.19840-1-colin.king@canonical.com
Acked-by: Mike Looijmans <mike.looijmans@topic.nl>
Signed-off-by: Stephen Boyd <sboyd@kernel.org>
drivers/clk/clk-si5341.c

index 72424eb7e5f8777549721f5a06325f3924175b66..6e780c2a9e6ba4290cf8bce60936c5a943d4e1e4 100644 (file)
@@ -547,7 +547,6 @@ static int si5341_synth_clk_set_rate(struct clk_hw *hw, unsigned long rate,
        bool is_integer;
 
        n_num = synth->data->freq_vco;
-       n_den = rate;
 
        /* see if there's an integer solution */
        r = do_div(n_num, rate);