hwmon: (adc128d818) simplify final return in probe
authorDavid Lechner <dlechner@baylibre.com>
Mon, 6 May 2024 15:59:16 +0000 (10:59 -0500)
committerMark Brown <broonie@kernel.org>
Tue, 7 May 2024 01:31:48 +0000 (10:31 +0900)
The final return in adc128d818_probe() can be simplified by using
PTR_ERR_OR_ZERO rather than an if statement.

Suggested-by: Guenter Roeck <linux@roeck-us.net>
Signed-off-by: David Lechner <dlechner@baylibre.com>
Acked-by: Guenter Roeck <linux@roeck-us.net>
Link: https://lore.kernel.org/r/20240506-regulator-devm_regulator_get_enable_read_voltage-fixes-v1-2-356cdd152067@baylibre.com
Signed-off-by: Mark Brown <broonie@kernel.org>
drivers/hwmon/adc128d818.c

index 2a35acb011eb578087602bc812daee4b82facdeb..321a61977396f358157796ec2942fc1b436fa10d 100644 (file)
@@ -475,10 +475,8 @@ static int adc128_probe(struct i2c_client *client)
 
        hwmon_dev = devm_hwmon_device_register_with_groups(dev, client->name,
                                                           data, adc128_groups);
-       if (IS_ERR(hwmon_dev))
-               return PTR_ERR(hwmon_dev);
 
-       return 0;
+       return PTR_ERR_OR_ZERO(hwmon_dev);
 }
 
 static const struct i2c_device_id adc128_id[] = {