iio: dac: ds4424: Make use of the helper function dev_err_probe()
authorCai Huoqing <caihuoqing@baidu.com>
Tue, 28 Sep 2021 01:38:57 +0000 (09:38 +0800)
committerJonathan Cameron <Jonathan.Cameron@huawei.com>
Tue, 19 Oct 2021 07:30:43 +0000 (08:30 +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/20210928013902.1341-4-caihuoqing@baidu.com
Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
drivers/iio/dac/ds4424.c

index 79527fb..5a5e967 100644 (file)
@@ -232,12 +232,9 @@ static int ds4424_probe(struct i2c_client *client,
        indio_dev->name = id->name;
 
        data->vcc_reg = devm_regulator_get(&client->dev, "vcc");
-       if (IS_ERR(data->vcc_reg)) {
-               dev_err(&client->dev,
-                       "Failed to get vcc-supply regulator. err: %ld\n",
-                               PTR_ERR(data->vcc_reg));
-               return PTR_ERR(data->vcc_reg);
-       }
+       if (IS_ERR(data->vcc_reg))
+               return dev_err_probe(&client->dev, PTR_ERR(data->vcc_reg),
+                                    "Failed to get vcc-supply regulator.\n");
 
        mutex_init(&data->lock);
        ret = regulator_enable(data->vcc_reg);