hwmon: (tmp102) Improve error handling

Use devm_add_action_or_reset() instead of devm_add_action(), and check its return code. Signed-off-by: Guenter Roeck <linux@roeck-us.net>
mariux64 · Jul 31, 2016 · 1aa4f02 · 1aa4f02
1 parent 90e2b54
commit 1aa4f02
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/drivers/hwmon/tmp102.c b/drivers/hwmon/tmp102.c
@@ -227,7 +227,9 @@ static int tmp102_probe(struct i2c_client *client,
 
 	tmp102->config_orig = regval;
 
-	devm_add_action(dev, tmp102_restore_config, tmp102);
+	err = devm_add_action_or_reset(dev, tmp102_restore_config, tmp102);
+	if (err)
+		return err;
 
 	regval &= ~TMP102_CONFIG_CLEAR;
 	regval |= TMP102_CONFIG_SET;