hwmon: (it87) Fix fan tachometer reading in IT8712F rev 0x7 (I)
authorAndrew Paprocki <andrew@ishiboo.com>
Sat, 20 Sep 2008 08:25:19 +0000 (10:25 +0200)
committerJean Delvare <khali@mahadeva.delvare>
Sat, 20 Sep 2008 08:25:19 +0000 (10:25 +0200)
The IT8712F v0.9.1 datasheet applies to revisions >= 0x8 (J).
The driver was incorrectly attempting to enable 16-bit fan
readings on rev 0x7 (I) which led to incorrect RPM values.

Signed-off-by: Andrew Paprocki <andrew@ishiboo.com>
Tested-by: John Gumb <john.gumb@tandberg.com>
Signed-off-by: Jean Delvare <khali@linux-fr.org>
drivers/hwmon/it87.c

index 30cdb0956779646d311476078a9c30ebc6f643ca..f1133081cc42789b642a9e11c76c5efa6749a618 100644 (file)
@@ -273,10 +273,10 @@ struct it87_data {
 static inline int has_16bit_fans(const struct it87_data *data)
 {
        /* IT8705F Datasheet 0.4.1, 3h == Version G.
-          IT8712F Datasheet 0.9.1, section 8.3.5 indicates 7h == Version I.
+          IT8712F Datasheet 0.9.1, section 8.3.5 indicates 8h == Version J.
           These are the first revisions with 16bit tachometer support. */
        return (data->type == it87 && data->revision >= 0x03)
-           || (data->type == it8712 && data->revision >= 0x07)
+           || (data->type == it8712 && data->revision >= 0x08)
            || data->type == it8716
            || data->type == it8718;
 }