[PATCH] UBUNTU: SAUCE: Dell All-In-One: Remove need for Dell module alias

Colin King colin.king at canonical.com
Mon Feb 14 15:04:21 UTC 2011


From: Colin Ian King <colin.king at canonical.com>

The driver originally used a workaround to load the All-In-One
module based on a Dell DMI module alias.  Since the WMI sys interface
supports the WMI GUIDs listed in the driver we can remove this
unneccessary work around.  Also, fix GUIDs to have no lower case hex
characters.

http://bugs.launchpad.net/bugs/718775

Signed-off-by: Colin Ian King <colin.king at canonical.com>
---
 drivers/platform/x86/dell-wmi-aio.c |    6 +-----
 1 files changed, 1 insertions(+), 5 deletions(-)

diff --git a/drivers/platform/x86/dell-wmi-aio.c b/drivers/platform/x86/dell-wmi-aio.c
index 755c6d2..3dccbaa 100644
--- a/drivers/platform/x86/dell-wmi-aio.c
+++ b/drivers/platform/x86/dell-wmi-aio.c
@@ -32,7 +32,7 @@ MODULE_DESCRIPTION("WMI hotkeys driver for Dell All-In-One series");
 MODULE_LICENSE("GPL");
 
 #define EVENT_GUID1 "284A0E6B-380E-472A-921F-E52786257FB4"
-#define EVENT_GUID2 "02314822-307C-4F66-bf0E-48AEAEB26CC8"
+#define EVENT_GUID2 "02314822-307C-4F66-BF0E-48AEAEB26CC8"
 
 static char *dell_wmi_aio_guids[] = {
 	EVENT_GUID1,
@@ -40,10 +40,6 @@ static char *dell_wmi_aio_guids[] = {
 	NULL
 };
 
-/* Temporary workaround until the WMI sysfs interface goes in.
-   Borrowed from acer-wmi */
-MODULE_ALIAS("dmi:*:*Dell*:*:");
-
 MODULE_ALIAS("wmi:"EVENT_GUID1);
 MODULE_ALIAS("wmi:"EVENT_GUID2);
 
-- 
1.7.0.4





More information about the kernel-team mailing list