Re: Subject:ACPI: Send the reserved AC notification event to user space

[Len Brown <lenb@kernel.org>]

On Friday 09 May 2008, Zhao Yakui wrote:
> Subject:ACPI: Send the reserved AC notification event to user space 
> >From : Zhao Yakui <yakui.zhao@intel.com>
> 
> On some laptops the bogus bios sends the reserved AC notification event(0x81)
> when the AC adapter is plugged/unplugged .It can't be sent to 
> user space, which causes that user space application can't 
> receive the AC adapter event. 
> So the reserved AC adapter event(0x81) is still sent to the user space 
> so that user space can receive the notification event.
> 
> Signed-off-by: Zhao Yakui <yakui.zhao@intel.com>
> ---
>  drivers/acpi/ac.c |    4 ++++
>  1 file changed, 4 insertions(+)
> 
> Index: linux-2.6/drivers/acpi/ac.c
> ===================================================================
> --- linux-2.6.orig/drivers/acpi/ac.c
> +++ linux-2.6/drivers/acpi/ac.c
> @@ -42,6 +42,7 @@
>  #define ACPI_AC_DEVICE_NAME		"AC Adapter"
>  #define ACPI_AC_FILE_STATE		"state"
>  #define ACPI_AC_NOTIFY_STATUS		0x80
> +#define ACPI_AC_NOTIFY_RESERVED		0x81
>  #define ACPI_AC_STATUS_OFFLINE		0x00
>  #define ACPI_AC_STATUS_ONLINE		0x01
>  #define ACPI_AC_STATUS_UNKNOWN		0xFF
> @@ -233,6 +234,9 @@ static void acpi_ac_notify(acpi_handle h
>  
>  	device = ac->device;
>  	switch (event) {
> +	case ACPI_AC_NOTIFY_RESERVED:

shouldn't this line above be "default:"
and the default: below this patch be removed?

-Len

> +		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:
> 
> 
> --
> To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>