diff options
author | Wei Yongjun <weiyongjun1@huawei.com> | 2018-01-11 14:12:41 +0300 |
---|---|---|
committer | Mark Brown <broonie@kernel.org> | 2018-01-11 15:30:21 +0300 |
commit | d5ff18bcd4b5e66396eab7e8271172157ee1253e (patch) | |
tree | 36761ea19d453ce23bba13d0c578da05b4b199b2 | |
parent | 25140717414c319bcc44b5aac39357a52d0bc8e0 (diff) | |
download | linux-d5ff18bcd4b5e66396eab7e8271172157ee1253e.tar.xz |
IIO: ADC: fix return value check in stm32_dfsdm_adc_probe()
In case of error, the function devm_iio_device_alloc() returns NULL
pointer not ERR_PTR(). The IS_ERR() test in the return value check
should be replaced with NULL test.
Fixes: e2e6771c6462 ("IIO: ADC: add STM32 DFSDM sigma delta ADC support")
Signed-off-by: Wei Yongjun <weiyongjun1@huawei.com>
Signed-off-by: Mark Brown <broonie@kernel.org>
-rw-r--r-- | drivers/iio/adc/stm32-dfsdm-adc.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/iio/adc/stm32-dfsdm-adc.c b/drivers/iio/adc/stm32-dfsdm-adc.c index e628d04d5c77..5e871404f565 100644 --- a/drivers/iio/adc/stm32-dfsdm-adc.c +++ b/drivers/iio/adc/stm32-dfsdm-adc.c @@ -1100,9 +1100,9 @@ static int stm32_dfsdm_adc_probe(struct platform_device *pdev) dev_data = (const struct stm32_dfsdm_dev_data *)of_id->data; iio = devm_iio_device_alloc(dev, sizeof(*adc)); - if (IS_ERR(iio)) { + if (!iio) { dev_err(dev, "%s: Failed to allocate IIO\n", __func__); - return PTR_ERR(iio); + return -ENOMEM; } adc = iio_priv(iio); |