diff options
author | Axel Lin <axel.lin@ingics.com> | 2014-08-05 02:08:31 (GMT) |
---|---|---|
committer | Guenter Roeck <linux@roeck-us.net> | 2014-08-06 00:48:52 (GMT) |
commit | 5b963089161b8fb244889c972edf553b9d737545 (patch) | |
tree | d512add2afcc03879f3cb2581a2f5a32db6a0c93 /drivers/i2c/busses/i2c-i801.c | |
parent | f4d33337eac4007793ca11fd1ab68d91ce7aa762 (diff) | |
download | linux-5b963089161b8fb244889c972edf553b9d737545.tar.xz |
hwmon: (lm92) Prevent overflow problem when writing large limits
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.
The hysteresis temperature range depends on the value of
data->temp[attr->index], since val is subtracted from it.
Use a wider clamp, [-120000, 220000] should do to cover the
possible range. Also add missing TEMP_TO_REG() on writes into
cached hysteresis value.
Also uses clamp_val to simplify the code a bit.
Signed-off-by: Axel Lin <axel.lin@ingics.com>
[Guenter Roeck: Fixed double TEMP_TO_REG on hysteresis updates]
Cc: stable@vger.kernel.org
Signed-off-by: Guenter Roeck <linux@roeck-us.net>
Diffstat (limited to 'drivers/i2c/busses/i2c-i801.c')
0 files changed, 0 insertions, 0 deletions