diff options
| author |   Dan Carpenter <dan.carpenter@linaro.org> | 2024-08-08 22:28:17 +0300 |
|---|---|---|
| committer |   Jonathan Cameron <Jonathan.Cameron@huawei.com> | 2024-08-10 11:34:27 +0100 |
| commit | c5d2291a3086f4b59ae42d5082cfe45ef9103062 (patch) | |
| tree | 0673a87413ccb17b30cad534ff9d8e203db88223 | |
| parent | iio: Fix spelling mistake "avaialable" -> "available" (diff) | |
| download | wireguard-linux-c5d2291a3086f4b59ae42d5082cfe45ef9103062.tar.xz wireguard-linux-c5d2291a3086f4b59ae42d5082cfe45ef9103062.zip | |
iio: adc: pac1921: add missing error return in probe()
This error path was intended to return, and not just print an error. The
current code will lead to an error pointer dereference.
Fixes: 371f778b83cd ("iio: adc: add support for pac1921")
Signed-off-by: Dan Carpenter <dan.carpenter@linaro.org>
Acked-by: Matteo Martelli <matteomartelli3@gmail.com>
Link: https://patch.msgid.link/1fa4ab12-0939-477d-bc92-306fd32e4fd9@stanley.mountain
Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
Diffstat (limited to '')
| -rw-r--r-- | drivers/iio/adc/pac1921.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/iio/adc/pac1921.c b/drivers/iio/adc/pac1921.c index d04c6685d780..8200a47bdf21 100644 --- a/drivers/iio/adc/pac1921.c +++ b/drivers/iio/adc/pac1921.c @@ -1168,8 +1168,8 @@ static int pac1921_probe(struct i2c_client *client) priv->regmap = devm_regmap_init_i2c(client, &pac1921_regmap_config); if (IS_ERR(priv->regmap)) - dev_err_probe(dev, (int)PTR_ERR(priv->regmap), - "Cannot initialize register map\n"); + return dev_err_probe(dev, (int)PTR_ERR(priv->regmap), + "Cannot initialize register map\n"); devm_mutex_init(dev, &priv->lock); |