[PATCH] abituguru: Fix fan detection
authorHans de Goede <j.w.r.degoede@hhs.nl>
Sun, 4 Jun 2006 18:24:11 +0000 (20:24 +0200)
committerGreg Kroah-Hartman <gregkh@suse.de>
Thu, 22 Jun 2006 18:10:34 +0000 (11:10 -0700)
One of my testers had a problem where the driver only saw 2 of the 4 fan
sensors his uGuru has, this fixes this.
 -accept 0x40 (bit 6) being high as a valid fan sensor setting for all fans
  not just fan 1, I have a feeling this bit indicates whether or not a fan is
  actually connected .

Signed-off-by: Hans de Goede <j.w.r.degoede@hhs.nl>
Signed-off-by: Jean Delvare <khali@linux-fr.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
drivers/hwmon/abituguru.c

index ab80b418bd69757a2279e518a6f8a68445e6dd19..59122cc0a50a1b39740970ab00adfb840a08c01d 100644 (file)
@@ -529,9 +529,8 @@ abituguru_detect_no_bank2_sensors(struct abituguru_data *data)
                   -0x08 enable beep
                   -0x01 enable alarm
                   All other bits should be 0, but on some motherboards
-                  0x40 (bit 6) is also high, at least for fan1 */
-               if ((!i && (data->bank2_settings[i][0] & ~0xC9)) ||
-                    (i && (data->bank2_settings[i][0] & ~0x89))) {
+                  0x40 (bit 6) is also high for some of the fans?? */
+               if (data->bank2_settings[i][0] & ~0xC9) {
                        ABIT_UGURU_DEBUG(2, "  bank2 sensor %d does not seem "
                                "to be a fan sensor: settings[0] = %02X\n",
                                i, (unsigned int)data->bank2_settings[i][0]);