diff options
| author |   Colin Ian King <colin.king@canonical.com> | 2019-07-01 17:50:20 +0100 |
|---|---|---|
| committer |   Stephen Boyd <sboyd@kernel.org> | 2019-08-07 14:23:24 -0700 |
| commit | e2f2402f3be89e4407d94d92d326204f70261754 (patch) | |
| tree | 524e408df21cc50193aaac0d30f6d75f3fa53047 /drivers/clk/clk-si5341.c | |
| parent | clk: qoriq: Fix -Wunused-const-variable (diff) | |
| download | wireguard-linux-e2f2402f3be89e4407d94d92d326204f70261754.tar.xz wireguard-linux-e2f2402f3be89e4407d94d92d326204f70261754.zip | |
clk: Si5341/Si5340: remove redundant assignment to n_den
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>
Diffstat (limited to 'drivers/clk/clk-si5341.c')
| -rw-r--r-- | drivers/clk/clk-si5341.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/drivers/clk/clk-si5341.c b/drivers/clk/clk-si5341.c index 72424eb7e5f8..6e780c2a9e6b 100644 --- a/drivers/clk/clk-si5341.c +++ b/drivers/clk/clk-si5341.c @@ -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); |