summaryrefslogtreecommitdiffstats
path: root/drivers
diff options
context:
space:
mode:
authorGuenter Roeck <linux@roeck-us.net>2013-01-10 21:16:47 -0800
committerGuenter Roeck <linux@roeck-us.net>2013-02-06 09:57:57 -0800
commit91bba688016be0ba87ecb9d80d6490c2ebc63b0d (patch)
treef16bbaa5947f88b590f73ff3fb0719610637c475 /drivers
parentb06367070de3d9f50e4f8f82b92e77ba9a8ebf8f (diff)
downloadlinux-91bba688016be0ba87ecb9d80d6490c2ebc63b0d.tar.gz
linux-91bba688016be0ba87ecb9d80d6490c2ebc63b0d.tar.bz2
linux-91bba688016be0ba87ecb9d80d6490c2ebc63b0d.zip
hwmon: (lm73) Fix lower and upper temperature limits
While the LM73 is only specified for temperatures from -40 to +150 degrees C, its power-up minimum and maximum temperature limits are -256 and +255.75 degrees C. For better consistency and to avoid confusion, clamp limits to the power-up limits and not to -40 / +150 degrees C. Signed-off-by: Guenter Roeck <linux@roeck-us.net>
Diffstat (limited to 'drivers')
-rw-r--r--drivers/hwmon/lm73.c7
1 files changed, 3 insertions, 4 deletions
diff --git a/drivers/hwmon/lm73.c b/drivers/hwmon/lm73.c
index 5166a8e3a061..dad83fea807d 100644
--- a/drivers/hwmon/lm73.c
+++ b/drivers/hwmon/lm73.c
@@ -36,8 +36,8 @@ static const unsigned short normal_i2c[] = { 0x48, 0x49, 0x4a, 0x4c,
#define LM73_ID 0x9001 /* 0x0190, byte-swapped */
#define DRVNAME "lm73"
-#define LM73_TEMP_MIN (-40)
-#define LM73_TEMP_MAX 150
+#define LM73_TEMP_MIN (-256000 / 250)
+#define LM73_TEMP_MAX (255750 / 250)
/*-----------------------------------------------------------------------*/
@@ -56,8 +56,7 @@ static ssize_t set_temp(struct device *dev, struct device_attribute *da,
return status;
/* Write value */
- value = (short) clamp_val(temp / 250, LM73_TEMP_MIN * 4,
- LM73_TEMP_MAX * 4) << 5;
+ value = clamp_val(temp / 250, LM73_TEMP_MIN, LM73_TEMP_MAX) << 5;
err = i2c_smbus_write_word_swapped(client, attr->index, value);
return (err < 0) ? err : count;
}