iio: light: veml6070: use dev_err_probe in probe function
authorJavier Carrasco <javier.carrasco.cruz@gmail.com>
Sun, 29 Sep 2024 20:38:52 +0000 (22:38 +0200)
committerJonathan Cameron <Jonathan.Cameron@huawei.com>
Sat, 5 Oct 2024 11:09:15 +0000 (12:09 +0100)
Drop the common 'dev_err() + return' combination in the probe function
and use 'return dev_err_probe()' instead.

Signed-off-by: Javier Carrasco <javier.carrasco.cruz@gmail.com>
Link: https://patch.msgid.link/20240929-veml6070-cleanup-v1-7-a9350341a646@gmail.com
Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
drivers/iio/light/veml6070.c

index 46eafaa9053d48010ab47aa3a4d71526509bdb92..898e285322d45384c9bddc74c8d5f0ee1f6a2a1b 100644 (file)
@@ -168,10 +168,9 @@ static int veml6070_probe(struct i2c_client *client)
                return ret;
 
        data->client2 = i2c_new_dummy_device(client->adapter, VEML6070_ADDR_DATA_LSB);
-       if (IS_ERR(data->client2)) {
-               dev_err(&client->dev, "i2c device for second chip address failed\n");
-               return PTR_ERR(data->client2);
-       }
+       if (IS_ERR(data->client2))
+               return dev_err_probe(&client->dev, PTR_ERR(data->client2),
+                                    "i2c device for second chip address failed\n");
 
        data->config = VEML6070_IT_10 | VEML6070_COMMAND_RSRVD |
                VEML6070_COMMAND_SD;