hwmon: (adc128d818) simplify final return in probe

dlech · broonie · commit 346fe0ce1fd7 · 2024-05-07T10:31:48.000+09:00
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>
diff --git a/drivers/hwmon/adc128d818.c b/drivers/hwmon/adc128d818.c
@@ -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[] = {

Original file line number	Diff line number	Diff line change
`@@ -475,10 +475,8 @@ static int adc128_probe(struct i2c_client *client)`
`475`	`475`
`476`	`476`	`hwmon_dev = devm_hwmon_device_register_with_groups(dev, client->name,`
`477`	`477`	`data, adc128_groups);`
`478`		`- if (IS_ERR(hwmon_dev))`
`479`		`- return PTR_ERR(hwmon_dev);`
`480`	`478`
`481`		`- return 0;`
	`479`	`+ return PTR_ERR_OR_ZERO(hwmon_dev);`
`482`	`480`	`}`
`483`	`481`
`484`	`482`	`static const struct i2c_device_id adc128_id[] = {`