Browse Source

WMI: remove EC region handler when _WDG parsing fails

Driver initialization was forgetting to remove EC address space handler
in cases when parse_wdg() method failed.

Signed-off-by: Dmitry Torokhov <dtor@mail.ru>
Signed-off-by: Matthew Garrett <mjg@redhat.com>
Dmitry Torokhov 15 years ago
parent
commit
5212cd678a
1 changed files with 7 additions and 2 deletions
  1. 7 2
      drivers/platform/x86/wmi.c

+ 7 - 2
drivers/platform/x86/wmi.c

@@ -956,12 +956,17 @@ static int acpi_wmi_add(struct acpi_device *device)
 						    ACPI_ADR_SPACE_EC,
 						    ACPI_ADR_SPACE_EC,
 						    &acpi_wmi_ec_space_handler,
 						    &acpi_wmi_ec_space_handler,
 						    NULL, NULL);
 						    NULL, NULL);
-	if (ACPI_FAILURE(status))
+	if (ACPI_FAILURE(status)) {
+		printk(KERN_ERR PREFIX "Error installing EC region handler\n");
 		return -ENODEV;
 		return -ENODEV;
+	}
 
 
 	status = parse_wdg(device->handle);
 	status = parse_wdg(device->handle);
 	if (ACPI_FAILURE(status)) {
 	if (ACPI_FAILURE(status)) {
-		printk(KERN_ERR PREFIX "Error installing EC region handler\n");
+		acpi_remove_address_space_handler(device->handle,
+						  ACPI_ADR_SPACE_EC,
+						  &acpi_wmi_ec_space_handler);
+		printk(KERN_ERR PREFIX "Failed to parse WDG method\n");
 		return -ENODEV;
 		return -ENODEV;
 	}
 	}