diff options
| author | Guenter Roeck <linux@roeck-us.net> | 2016-07-25 14:47:18 -0700 |
|---|---|---|
| committer | Guenter Roeck <linux@roeck-us.net> | 2016-07-31 15:01:06 -0700 |
| commit | be9d6374a76ffbeda783a5bdcb00e219c7bda2d0 (patch) | |
| tree | 75e7a87b1cf772c0bce570b9db773d7b5aafce34 /drivers/hwmon/lm90.c | |
| parent | 991f9fa90360b64e8b79688d8ab0f034983a959d (diff) | |
hwmon: (lm90) Add missing assignment
Coverity reports the following error.
>>> CID 1364474: Error handling issues (CHECKED_RETURN)
>>> Calling "lm90_read_reg" without checking return value (as is done
>>> elsewhere 28 out of 29 times).
532 lm90_read_reg(client, LM90_REG_R_REMOTE_LOWH);
533 if (val < 0)
534 return val;
Fixes: 10bfef47bd259 ("hwmon: (lm90) Read limit registers only once")
Reviewed-by: Jean Delvare <jdelvare@suse.de>
Signed-off-by: Guenter Roeck <linux@roeck-us.net>
Diffstat (limited to 'drivers/hwmon/lm90.c')
| -rw-r--r-- | drivers/hwmon/lm90.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/hwmon/lm90.c b/drivers/hwmon/lm90.c index f51e758ba529..7b3eedf76c62 100644 --- a/drivers/hwmon/lm90.c +++ b/drivers/hwmon/lm90.c @@ -529,7 +529,7 @@ static int lm90_update_limits(struct device *dev) return val; data->temp_hyst = val; - lm90_read_reg(client, LM90_REG_R_REMOTE_LOWH); + val = lm90_read_reg(client, LM90_REG_R_REMOTE_LOWH); if (val < 0) return val; data->temp11[REMOTE_LOW] = val << 8; |