Commit 48371f90 authored by Guenter Roeck's avatar Guenter Roeck Committed by Ben Hutchings

hwmon: (lm78) Fix overflow problems seen when writing large temperature limits

commit 1074d683 upstream.

On platforms with sizeof(int) < sizeof(long), writing a temperature
limit larger than MAXINT will result in unpredictable limit values
written to the chip. Avoid auto-conversion from long to int to fix
the problem.

Cc: Axel Lin <axel.lin@ingics.com>
Reviewed-by: default avatarAxel Lin <axel.lin@ingics.com>
Signed-off-by: default avatarGuenter Roeck <linux@roeck-us.net>
[bwh: Backported to 3.2: adjust context]
Signed-off-by: default avatarBen Hutchings <ben@decadent.org.uk>
parent d02ae215
...@@ -102,7 +102,7 @@ static inline int FAN_FROM_REG(u8 val, int div) ...@@ -102,7 +102,7 @@ static inline int FAN_FROM_REG(u8 val, int div)
/* TEMP: mC (-128C to +127C) /* TEMP: mC (-128C to +127C)
REG: 1C/bit, two's complement */ REG: 1C/bit, two's complement */
static inline s8 TEMP_TO_REG(int val) static inline s8 TEMP_TO_REG(long val)
{ {
int nval = SENSORS_LIMIT(val, -128000, 127000) ; int nval = SENSORS_LIMIT(val, -128000, 127000) ;
return nval<0 ? (nval-500)/1000 : (nval+500)/1000; return nval<0 ? (nval-500)/1000 : (nval+500)/1000;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment