Commit a44908d7 authored by Jean Delvare's avatar Jean Delvare

hwmon: (tmp421) Fix temperature conversions

The low bits of temperature registers are status bits, they must be
masked out before converting the register values to temperatures.
Signed-off-by: default avatarJean Delvare <khali@linux-fr.org>
Tested-by: default avatarAndre Prendel <andre.prendel@gmx.de>
Cc: stable@kernel.org
parent 8d59582a
...@@ -80,14 +80,16 @@ struct tmp421_data { ...@@ -80,14 +80,16 @@ struct tmp421_data {
static int temp_from_s16(s16 reg) static int temp_from_s16(s16 reg)
{ {
int temp = reg; /* Mask out status bits */
int temp = reg & ~0xf;
return (temp * 1000 + 128) / 256; return (temp * 1000 + 128) / 256;
} }
static int temp_from_u16(u16 reg) static int temp_from_u16(u16 reg)
{ {
int temp = reg; /* Mask out status bits */
int temp = reg & ~0xf;
/* Add offset for extended temperature range. */ /* Add offset for extended temperature range. */
temp -= 64 * 256; temp -= 64 * 256;
......
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