aboutsummaryrefslogtreecommitdiffstats
path: root/drivers/hwmon
diff options
context:
space:
mode:
authorGuenter Roeck <linux@roeck-us.net>2016-07-25 17:57:47 -0400
committerGuenter Roeck <linux@roeck-us.net>2016-07-31 18:02:51 -0400
commit1aa4f0289fd5554a9cd58e5b47a361b14256ea90 (patch)
tree16e50496c19c420d82aa83f612b34757f62f04f7 /drivers/hwmon
parent90e2b545ce78516e727ba641e9f2b0703a17da1a (diff)
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>
Diffstat (limited to 'drivers/hwmon')
-rw-r--r--drivers/hwmon/tmp102.c4
1 files changed, 3 insertions, 1 deletions
diff --git a/drivers/hwmon/tmp102.c b/drivers/hwmon/tmp102.c
index a942a2574a4d..8479ac5eb853 100644
--- a/drivers/hwmon/tmp102.c
+++ b/drivers/hwmon/tmp102.c
@@ -227,7 +227,9 @@ static int tmp102_probe(struct i2c_client *client,
227 227
228 tmp102->config_orig = regval; 228 tmp102->config_orig = regval;
229 229
230 devm_add_action(dev, tmp102_restore_config, tmp102); 230 err = devm_add_action_or_reset(dev, tmp102_restore_config, tmp102);
231 if (err)
232 return err;
231 233
232 regval &= ~TMP102_CONFIG_CLEAR; 234 regval &= ~TMP102_CONFIG_CLEAR;
233 regval |= TMP102_CONFIG_SET; 235 regval |= TMP102_CONFIG_SET;