Hello, On Tue, 29 Dec 2009 00:34:14 -0800 Dmitry Torokhov <dmitry.torokhov@xxxxxxxxx> wrote: > wmi_install_notify_handler() retruns ACPI error codes instead of standard > Exxxx error codes and they should not be propagated unchanged to the upper > layers. > > Reported-by: Paul Rolland <rol@xxxxxxxxxx> > Signed-off-by: Dmitry Torokhov <dtor@xxxxxxx> Tested-by: Paul Rolland <rol@xxxxxxxxxx> This is causing : [root@tux ~]# modprobe dell-wmi FATAL: Error inserting dell_wmi (/lib/modules/2.6.33-rc2/kernel/drivers/platform/x86/dell-wmi.ko): Input/output error and Dec 29 09:54:54 tux kernel: ACPI: WMI: Mapper loaded Dec 29 09:54:54 tux kernel: input: Dell WMI hotkeys as /devices/virtual/input/input13 Dec 29 09:54:54 tux kernel: dell-wmi: Unable to register notify handler - 6 which looks Ok as my machine doesn't seem to have the corresponding stuff. Anyway, this still needs the changes I sent for wmi.c to avoid the Oops. Paul > --- > > drivers/platform/x86/dell-wmi.c | 11 ++++++----- > 1 files changed, 6 insertions(+), 5 deletions(-) > > diff --git a/drivers/platform/x86/dell-wmi.c > b/drivers/platform/x86/dell-wmi.c index ff67a78..79bb1f1 100644 > --- a/drivers/platform/x86/dell-wmi.c > +++ b/drivers/platform/x86/dell-wmi.c > @@ -265,6 +265,7 @@ static void __init find_hk_type(const struct > dmi_header *dm, void *dummy) > static int __init dell_wmi_init(void) > { > + acpi_status status; > int err; > > if (!wmi_has_guid(DELL_EVENT_GUID)) { > @@ -279,12 +280,12 @@ static int __init dell_wmi_init(void) > if (err) > return err; > > - err = wmi_install_notify_handler(DELL_EVENT_GUID, > - dell_wmi_notify, NULL); > - if (err) { > + status = wmi_install_notify_handler(DELL_EVENT_GUID, > + dell_wmi_notify, NULL); > + if (!ACPI_SUCCESS(status)) { > dell_wmi_input_destroy(); > - pr_err("Unable to register notify handler - %d\n", err); > - return err; > + pr_err("Unable to register notify handler - %d\n", > status); > + return -EIO; > } > > return 0; > -- Paul Rolland E-Mail : rol(at)witbe.net CTO - Witbe.net SA Tel. +33 (0)1 47 67 77 77 Les Collines de l'Arche Fax. +33 (0)1 47 67 77 99 F-92057 Paris La Defense RIPE : PR12-RIPE Please no HTML, I'm not a browser - Pas d'HTML, je ne suis pas un navigateur "Some people dream of success... while others wake up and work hard at it" "I worry about my child and the Internet all the time, even though she's too young to have logged on yet. Here's what I worry about. I worry that 10 or 15 years from now, she will come to me and say 'Daddy, where were you when they took freedom of the press away from the Internet?'" --Mike Godwin, Electronic Frontier Foundation -- To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html