diff options
| author | Dan Carpenter <dan.carpenter@linaro.org> | 2023-06-27 10:12:36 +0300 |
|---|---|---|
| committer | Wolfram Sang <wsa@kernel.org> | 2023-08-14 17:52:58 +0200 |
| commit | 0abbf0ac10eeede6e771a1a79342baf5e8466ee3 (patch) | |
| tree | 2ab46b1cf76a2515d7d316d3eee85d34fb917e52 /drivers/i2c | |
| parent | fff67c1b17ee093947bdcbac6f64d072e644159a (diff) | |
| download | lwn-0abbf0ac10eeede6e771a1a79342baf5e8466ee3.tar.gz lwn-0abbf0ac10eeede6e771a1a79342baf5e8466ee3.zip | |
i2c: sun6i-p2wi: Fix an error message in probe()
The "ret" variable is uninitialized. It was the "p2wi->rstc" variable
that was intended. We can also use the %pe string format to print the
error code name instead of just the number.
Fixes: 75ff8a340a81 ("i2c: sun6i-p2wi: Use devm_clk_get_enabled()")
Signed-off-by: Dan Carpenter <dan.carpenter@linaro.org>
Reviewed-by: Andi Shyti <andi.shyti@kernel.org>
Signed-off-by: Wolfram Sang <wsa@kernel.org>
Diffstat (limited to 'drivers/i2c')
| -rw-r--r-- | drivers/i2c/busses/i2c-sun6i-p2wi.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/i2c/busses/i2c-sun6i-p2wi.c b/drivers/i2c/busses/i2c-sun6i-p2wi.c index ad8270cdbd3e..fa6020dced59 100644 --- a/drivers/i2c/busses/i2c-sun6i-p2wi.c +++ b/drivers/i2c/busses/i2c-sun6i-p2wi.c @@ -250,7 +250,8 @@ static int p2wi_probe(struct platform_device *pdev) p2wi->rstc = devm_reset_control_get_exclusive(dev, NULL); if (IS_ERR(p2wi->rstc)) { - dev_err(dev, "failed to retrieve reset controller: %d\n", ret); + dev_err(dev, "failed to retrieve reset controller: %pe\n", + p2wi->rstc); return PTR_ERR(p2wi->rstc); } |