Commit c0f31403 authored by Juerg Haefliger's avatar Juerg Haefliger Committed by Mark M. Hoffman

hwmon: fix dme1737 temp fault attribute

Fix temp?_fault attribute. The temp was incorrectly compared against
0x0800 rather than 0x8000. Only the upper 8 bits are compared as the
datasheet doesn't specify what happens to the lower bits in case of a
diode fault.

Signed-off-by: Juerg Haefliger <juergh at gmail.com>
Acked-by: default avatarJean Delvare <khali@linux-fr.org>
Signed-off-by: default avatarMark M. Hoffman <mhoffman@lightlink.com>
parent d0546128
...@@ -750,7 +750,7 @@ static ssize_t show_temp(struct device *dev, struct device_attribute *attr, ...@@ -750,7 +750,7 @@ static ssize_t show_temp(struct device *dev, struct device_attribute *attr,
res = (data->alarms >> DME1737_BIT_ALARM_TEMP[ix]) & 0x01; res = (data->alarms >> DME1737_BIT_ALARM_TEMP[ix]) & 0x01;
break; break;
case SYS_TEMP_FAULT: case SYS_TEMP_FAULT:
res = (data->temp[ix] == 0x0800); res = (((u16)data->temp[ix] & 0xff00) == 0x8000);
break; break;
default: default:
res = 0; res = 0;
......
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