aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorBjorn Helgaas <bjorn.helgaas@hp.com>2009-09-21 15:29:15 -0400
committerLen Brown <len.brown@intel.com>2009-09-25 14:24:25 -0400
commitccba2a36d74a9da815e597ac727cfd096fa8e750 (patch)
tree2fc51fdcf0cc3e7fefcf35b6d25da2f7edede24c
parentcaaa6efb3d82d0102db9e7094ca5773c46e6780c (diff)
ACPI: use device_type rather than comparing HID
Check the acpi_device device_type rather than the HID. Signed-off-by: Bjorn Helgaas <bjorn.helgaas@hp.com> Signed-off-by: Len Brown <len.brown@intel.com>
-rw-r--r--drivers/acpi/scan.c10
1 files changed, 4 insertions, 6 deletions
diff --git a/drivers/acpi/scan.c b/drivers/acpi/scan.c
index c73681b7e69e..c8e867b4a842 100644
--- a/drivers/acpi/scan.c
+++ b/drivers/acpi/scan.c
@@ -378,15 +378,13 @@ static acpi_status acpi_device_notify_fixed(void *data)
378static int acpi_device_install_notify_handler(struct acpi_device *device) 378static int acpi_device_install_notify_handler(struct acpi_device *device)
379{ 379{
380 acpi_status status; 380 acpi_status status;
381 char *hid;
382 381
383 hid = acpi_device_hid(device); 382 if (device->device_type == ACPI_BUS_TYPE_POWER_BUTTON)
384 if (!strcmp(hid, ACPI_BUTTON_HID_POWERF))
385 status = 383 status =
386 acpi_install_fixed_event_handler(ACPI_EVENT_POWER_BUTTON, 384 acpi_install_fixed_event_handler(ACPI_EVENT_POWER_BUTTON,
387 acpi_device_notify_fixed, 385 acpi_device_notify_fixed,
388 device); 386 device);
389 else if (!strcmp(hid, ACPI_BUTTON_HID_SLEEPF)) 387 else if (device->device_type == ACPI_BUS_TYPE_SLEEP_BUTTON)
390 status = 388 status =
391 acpi_install_fixed_event_handler(ACPI_EVENT_SLEEP_BUTTON, 389 acpi_install_fixed_event_handler(ACPI_EVENT_SLEEP_BUTTON,
392 acpi_device_notify_fixed, 390 acpi_device_notify_fixed,
@@ -404,10 +402,10 @@ static int acpi_device_install_notify_handler(struct acpi_device *device)
404 402
405static void acpi_device_remove_notify_handler(struct acpi_device *device) 403static void acpi_device_remove_notify_handler(struct acpi_device *device)
406{ 404{
407 if (!strcmp(acpi_device_hid(device), ACPI_BUTTON_HID_POWERF)) 405 if (device->device_type == ACPI_BUS_TYPE_POWER_BUTTON)
408 acpi_remove_fixed_event_handler(ACPI_EVENT_POWER_BUTTON, 406 acpi_remove_fixed_event_handler(ACPI_EVENT_POWER_BUTTON,
409 acpi_device_notify_fixed); 407 acpi_device_notify_fixed);
410 else if (!strcmp(acpi_device_hid(device), ACPI_BUTTON_HID_SLEEPF)) 408 else if (device->device_type == ACPI_BUS_TYPE_SLEEP_BUTTON)
411 acpi_remove_fixed_event_handler(ACPI_EVENT_SLEEP_BUTTON, 409 acpi_remove_fixed_event_handler(ACPI_EVENT_SLEEP_BUTTON,
412 acpi_device_notify_fixed); 410 acpi_device_notify_fixed);
413 else 411 else