diff options
author | Guenter Roeck <linux@roeck-us.net> | 2016-07-25 17:57:47 -0400 |
---|---|---|
committer | Guenter Roeck <linux@roeck-us.net> | 2016-07-31 18:02:51 -0400 |
commit | 1aa4f0289fd5554a9cd58e5b47a361b14256ea90 (patch) | |
tree | 16e50496c19c420d82aa83f612b34757f62f04f7 /drivers/hwmon | |
parent | 90e2b545ce78516e727ba641e9f2b0703a17da1a (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.c | 4 |
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; |