diff options
| author | Jean Delvare <khali@linux-fr.org> | 2009-11-16 12:45:39 +0100 |
|---|---|---|
| committer | Jean Delvare <khali@linux-fr.org> | 2009-11-16 12:45:39 +0100 |
| commit | cf312e077662ec3a07529551ab6e885828ccfb1d (patch) | |
| tree | 447795bf44155537f707e50f6574b62a958ee514 | |
| parent | be4c23c93c4828d36ac9e1a88410618a61676426 (diff) | |
| download | lwn-cf312e077662ec3a07529551ab6e885828ccfb1d.tar.gz lwn-cf312e077662ec3a07529551ab6e885828ccfb1d.zip | |
hwmon: (adt7475) Fix temperature fault flags
The logic of temperature fault flags is wrong, it shows faults when
there are none and vice versa. Fix it.
I can't believe this has been broken since the driver was added, 8
months ago, basically breaking temp1 and temp3, and nobody ever
complained.
Signed-off-by: Jean Delvare <khali@linux-fr.org>
Acked-by: Hans de Goede <hdegoede@redhat.com>
Cc: Jordan Crouse <jordan@cosmicpenguin.net>
| -rw-r--r-- | drivers/hwmon/adt7475.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/hwmon/adt7475.c b/drivers/hwmon/adt7475.c index d39877a7da63..d15819b4fe5f 100644 --- a/drivers/hwmon/adt7475.c +++ b/drivers/hwmon/adt7475.c @@ -350,8 +350,7 @@ static ssize_t show_temp(struct device *dev, struct device_attribute *attr, case FAULT: /* Note - only for remote1 and remote2 */ - out = data->alarms & (sattr->index ? 0x8000 : 0x4000); - out = out ? 0 : 1; + out = !!(data->alarms & (sattr->index ? 0x8000 : 0x4000)); break; default: |