iio: adc: max1118: Make use of the helper function dev_err_probe()
authorCai Huoqing <caihuoqing@baidu.com>
Fri, 8 Oct 2021 09:28:52 +0000 (17:28 +0800)
committerJonathan Cameron <Jonathan.Cameron@huawei.com>
Wed, 20 Oct 2021 13:43:52 +0000 (14:43 +0100)
When possible use dev_err_probe help to properly deal with the
PROBE_DEFER error, the benefit is that DEFER issue will be logged
in the devices_deferred debugfs file.
Using dev_err_probe() can reduce code size, and the error value
gets printed.

Signed-off-by: Cai Huoqing <caihuoqing@baidu.com>
Link: https://lore.kernel.org/r/20211008092858.495-4-caihuoqing@baidu.com
Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
drivers/iio/adc/max1118.c

index 8cec9d94908386634fab2dc76008f6dc6d49c0bb..a41bc570be210af7c08fb20591f55b43d9ac4455 100644 (file)
@@ -221,10 +221,9 @@ static int max1118_probe(struct spi_device *spi)
 
        if (id->driver_data == max1118) {
                adc->reg = devm_regulator_get(&spi->dev, "vref");
-               if (IS_ERR(adc->reg)) {
-                       dev_err(&spi->dev, "failed to get vref regulator\n");
-                       return PTR_ERR(adc->reg);
-               }
+               if (IS_ERR(adc->reg))
+                       return dev_err_probe(&spi->dev, PTR_ERR(adc->reg),
+                                            "failed to get vref regulator\n");
                ret = regulator_enable(adc->reg);
                if (ret)
                        return ret;