On Wed, 2008-05-14 at 19:04 -0400, Len Brown wrote:
> Maybe I'm too tired to parse what you're saying.
> What is the matter with doing it this way:
> (this is now bugzilla 10695, btw.
>
The following patch is OK. Of course the matter is that all the reserved
events will be sent to the user space. For example: 0x82, 0x83. Even the
invalid event will also be sent.
In my patch only the 0x81 reserved event is sent to the user space. For
the invalid events and other reserved events(For example : 0x82, 0x83),
only warning message is printed but they won't be sent to the user
space.
>
> diff --git a/drivers/acpi/ac.c b/drivers/acpi/ac.c
> index 5b73f6a..831883b 100644
> --- a/drivers/acpi/ac.c
> +++ b/drivers/acpi/ac.c
> @@ -233,6 +233,9 @@ static void acpi_ac_notify(acpi_handle handle, u32 event, void *data)
>
> device = ac->device;
> switch (event) {
> + default:
> + ACPI_DEBUG_PRINT((ACPI_DB_INFO,
> + "Unsupported event [0x%x]\n", event));
> case ACPI_AC_NOTIFY_STATUS:
> case ACPI_NOTIFY_BUS_CHECK:
> case ACPI_NOTIFY_DEVICE_CHECK:
> @@ -244,11 +247,6 @@ static void acpi_ac_notify(acpi_handle handle, u32 event, void *data)
> #ifdef CONFIG_ACPI_SYSFS_POWER
> kobject_uevent(&ac->charger.dev->kobj, KOBJ_CHANGE);
> #endif
> - break;
> - default:
> - ACPI_DEBUG_PRINT((ACPI_DB_INFO,
> - "Unsupported event [0x%x]\n", event));
> - break;
> }
>
> return;
>
--
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
