i8k: Use driver_data field of dmi_system_id to override fan multiplier
authorGuenter Roeck <linux@roeck-us.net>
Sat, 14 Dec 2013 17:30:19 +0000 (09:30 -0800)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 19 Dec 2013 00:47:04 +0000 (16:47 -0800)
At least on Studio 1555 and XPS M140, the fan speed is reported directly,
not with the default speed multiplier of 30. Information on the web
suggests that this may be true for other models as well, though it is
unknown at this time which systems may be affected.

Use the driver_data field of dmi_system_id to override the default fan
multiplier value for the two systems known to use a multiplier of 1.

Signed-off-by: Guenter Roeck <linux@roeck-us.net>
Acked-by: Arnd Bergmann <arnd@arndb.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/char/i8k.c

index 18571b5..889b87b 100644 (file)
@@ -65,6 +65,7 @@ static DEFINE_MUTEX(i8k_mutex);
 static char bios_version[4];
 static struct device *i8k_hwmon_dev;
 static u32 i8k_hwmon_flags;
+static int i8k_fan_mult;
 
 #define I8K_HWMON_HAVE_TEMP1   (1 << 0)
 #define I8K_HWMON_HAVE_TEMP2   (1 << 1)
@@ -275,7 +276,7 @@ static int i8k_get_fan_speed(int fan)
        struct smm_regs regs = { .eax = I8K_SMM_GET_SPEED, };
 
        regs.ebx = fan & 0xff;
-       return i8k_smm(&regs) ? : (regs.eax & 0xffff) * fan_mult;
+       return i8k_smm(&regs) ? : (regs.eax & 0xffff) * i8k_fan_mult;
 }
 
 /*
@@ -698,6 +699,7 @@ static struct dmi_system_id i8k_dmi_table[] __initdata = {
                        DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."),
                        DMI_MATCH(DMI_PRODUCT_NAME, "Studio"),
                },
+               .driver_data = (void *)1,       /* fan multiplier override */
        },
        {
                .ident = "Dell XPS M140",
@@ -705,6 +707,7 @@ static struct dmi_system_id i8k_dmi_table[] __initdata = {
                        DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."),
                        DMI_MATCH(DMI_PRODUCT_NAME, "MXC051"),
                },
+               .driver_data = (void *)1,       /* fan multiplier override */
        },
        { }
 };
@@ -716,6 +719,7 @@ static int __init i8k_probe(void)
 {
        char buff[4];
        int version;
+       const struct dmi_system_id *id;
 
        /*
         * Get DMI information
@@ -769,6 +773,11 @@ static int __init i8k_probe(void)
                                buff, bios_version);
        }
 
+       i8k_fan_mult = fan_mult;
+       id = dmi_first_match(i8k_dmi_table);
+       if (id && fan_mult == I8K_FAN_MULT && id->driver_data)
+               i8k_fan_mult = (unsigned long)id->driver_data;
+
        return 0;
 }