diff options
| author |   Dan Carpenter <dan.carpenter@oracle.com> | 2022-01-11 10:25:29 +0300 |
|---|---|---|
| committer |   Stephen Boyd <sboyd@kernel.org> | 2022-01-13 13:21:02 -0800 |
| commit | 818d9150f2b22a0053bf568fa11ad3be804ce5c4 (patch) | |
| tree | 8d723329b6c8a869af800e84aa145444f8b8aff3 /drivers/clk/visconti | |
| parent | clk: si5341: Fix clock HW provider cleanup (diff) | |
| download | wireguard-linux-818d9150f2b22a0053bf568fa11ad3be804ce5c4.tar.xz wireguard-linux-818d9150f2b22a0053bf568fa11ad3be804ce5c4.zip | |
clk: visconti: Fix uninitialized variable in printk
The "pll_clck" variable is uninitialized. The "ret" error code was
supposed to be printed instead.
Fixes: b4cbe606dc36 ("clk: visconti: Add support common clock driver and reset driver")
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Link: https://lore.kernel.org/r/20220111072529.GJ11243@kili
Signed-off-by: Stephen Boyd <sboyd@kernel.org>
Diffstat (limited to 'drivers/clk/visconti')
| -rw-r--r-- | drivers/clk/visconti/pll.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/clk/visconti/pll.c b/drivers/clk/visconti/pll.c index a2398bc6c6e4..a484cb945d67 100644 --- a/drivers/clk/visconti/pll.c +++ b/drivers/clk/visconti/pll.c @@ -246,7 +246,6 @@ static struct clk_hw *visconti_register_pll(struct visconti_pll_provider *ctx, { struct clk_init_data init; struct visconti_pll *pll; - struct clk *pll_clk; struct clk_hw *pll_hw_clk; size_t len; int ret; @@ -277,7 +276,7 @@ static struct clk_hw *visconti_register_pll(struct visconti_pll_provider *ctx, pll_hw_clk = &pll->hw; ret = clk_hw_register(NULL, &pll->hw); if (ret) { - pr_err("failed to register pll clock %s : %ld\n", name, PTR_ERR(pll_clk)); + pr_err("failed to register pll clock %s : %d\n", name, ret); kfree(pll); pll_hw_clk = ERR_PTR(ret); } |