diff options
author | Jean Delvare <khali@linux-fr.org> | 2007-12-06 17:13:42 -0500 |
---|---|---|
committer | Mark M. Hoffman <mhoffman@lightlink.com> | 2008-02-07 20:39:42 -0500 |
commit | 67b671bceb4a8340a30929e9642620d99ed5ad76 (patch) | |
tree | d302333633bdbd752151933366aaaabfdc60e719 /drivers/hwmon/smsc47m1.c | |
parent | b20ff13a6ad64f07ce78c75e6a335c185270d73c (diff) |
hwmon: Let the user override the detected Super-I/O device ID
While it is possible to force SMBus-based hardware monitoring chip
drivers to drive a not officially supported device, we do not have this
possibility for Super-I/O-based drivers. That's unfortunate because
sometimes newer chips are fully compatible and just forcing the driver
to load would work. Instead of that we have to tell the users to
recompile the kernel driver, which isn't an easy task for everyone.
So, I propose that we add a module parameter to all Super-I/O based
hardware monitoring drivers, letting advanced users force the driver
to load on their machine. The user has to provide the device ID of a
supposedly compatible device. This requires looking at the source code or
a datasheet, so I am confident that users can't randomly force a driver
without knowing what they are doing. Thus this should be relatively safe.
As you can see from the code, the implementation is pretty simple and
unintrusive.
Signed-off-by: Jean Delvare <khali@linux-fr.org>
Acked-by: Hans de Goede <j.w.r.degoede@hhs.nl>
Signed-off-by: Mark M. Hoffman <mhoffman@lightlink.com>
Diffstat (limited to 'drivers/hwmon/smsc47m1.c')
-rw-r--r-- | drivers/hwmon/smsc47m1.c | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/drivers/hwmon/smsc47m1.c b/drivers/hwmon/smsc47m1.c index a10a380868e2..0d7f0c4d06bb 100644 --- a/drivers/hwmon/smsc47m1.c +++ b/drivers/hwmon/smsc47m1.c | |||
@@ -39,6 +39,10 @@ | |||
39 | #include <linux/sysfs.h> | 39 | #include <linux/sysfs.h> |
40 | #include <asm/io.h> | 40 | #include <asm/io.h> |
41 | 41 | ||
42 | static unsigned short force_id; | ||
43 | module_param(force_id, ushort, 0); | ||
44 | MODULE_PARM_DESC(force_id, "Override the detected device ID"); | ||
45 | |||
42 | static struct platform_device *pdev; | 46 | static struct platform_device *pdev; |
43 | 47 | ||
44 | #define DRVNAME "smsc47m1" | 48 | #define DRVNAME "smsc47m1" |
@@ -399,7 +403,7 @@ static int __init smsc47m1_find(unsigned short *addr, | |||
399 | u8 val; | 403 | u8 val; |
400 | 404 | ||
401 | superio_enter(); | 405 | superio_enter(); |
402 | val = superio_inb(SUPERIO_REG_DEVID); | 406 | val = force_id ? force_id : superio_inb(SUPERIO_REG_DEVID); |
403 | 407 | ||
404 | /* | 408 | /* |
405 | * SMSC LPC47M10x/LPC47M112/LPC47M13x (device id 0x59), LPC47M14x | 409 | * SMSC LPC47M10x/LPC47M112/LPC47M13x (device id 0x59), LPC47M14x |