Messages in this thread Patch in this message | | | Date | Tue, 21 Apr 2009 13:01:55 -0500 | From | Mario Limonciello <> | Subject | [PATCH 2/3] Don't load Dell-WMI on non WMI systems |
| |
Hi:
I've noticed that the dell-wmi module will load on systems even if WMI isn't supported. As this is dependent upon BIOS version, dell-wmi should return -ENODEV in these situations.
Regards -- Mario Limonciello *Dell | Linux Engineering* mario_limonciello@dell.com --- a/drivers/platform/x86/dell-wmi.c~ 2009-04-21 12:41:50.000000000 -0500 +++ b/drivers/platform/x86/dell-wmi.c 2009-04-21 12:30:35.000000000 -0500 @@ -255,8 +255,10 @@ return err; } - } else - printk(KERN_WARNING "dell-wmi: No known WMI GUID found\n"); + } else { + printk(KERN_ERR "dell-wmi: No known WMI GUID found\n"); + return -ENODEV; + } return 0; }[unhandled content-type:application/pgp-signature] | |