diff options
author | Axel Lin <axel.lin@ingics.com> | 2014-08-05 20:27:11 -0400 |
---|---|---|
committer | Guenter Roeck <linux@roeck-us.net> | 2014-08-05 22:44:42 -0400 |
commit | 970255b75d0b54e457091182a7ae3a188ac0c346 (patch) | |
tree | b8a1309772eca7b66a1e729c87f3082557ffc085 /drivers/hwmon | |
parent | 5c570b9573ecabf3ffc9af7f03daa088f2466d7b (diff) |
hwmon: (w83627hf) Fix vrm write operation
vrm is an u8, so the written value needs to be limited to [0, 255].
Signed-off-by: Axel Lin <axel.lin@ingics.com>
Signed-off-by: Guenter Roeck <linux@roeck-us.net>
Diffstat (limited to 'drivers/hwmon')
-rw-r--r-- | drivers/hwmon/w83627hf.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/drivers/hwmon/w83627hf.c b/drivers/hwmon/w83627hf.c index c1726be3654c..2f55973a8c4c 100644 --- a/drivers/hwmon/w83627hf.c +++ b/drivers/hwmon/w83627hf.c | |||
@@ -820,6 +820,9 @@ store_vrm_reg(struct device *dev, struct device_attribute *attr, const char *buf | |||
820 | err = kstrtoul(buf, 10, &val); | 820 | err = kstrtoul(buf, 10, &val); |
821 | if (err) | 821 | if (err) |
822 | return err; | 822 | return err; |
823 | |||
824 | if (val > 255) | ||
825 | return -EINVAL; | ||
823 | data->vrm = val; | 826 | data->vrm = val; |
824 | 827 | ||
825 | return count; | 828 | return count; |