On Friday 10 August 2007 17:07, Zhang Rui wrote:
> Both ACPI_VIDEO_NOTIFY_SWITCH and ACPI_VIDEO_NOTIFY_PROBE
> are valid for video Display Devices only.
> 
> ACPI video output devices should not handle these kinds of events.

please update this description to clarify that
these event codes are handled by the video bus,
and thus should simply not be here in the video device.

> Signed-off-by: Zhang Rui <[EMAIL PROTECTED]>
> ---
>  drivers/acpi/video.c |    4 ----
>  1 file changed, 4 deletions(-)
> 
> Index: linux-2.6.23-rc2/drivers/acpi/video.c
> ===================================================================
> --- linux-2.6.23-rc2.orig/drivers/acpi/video.c
> +++ linux-2.6.23-rc2/drivers/acpi/video.c
> @@ -1813,10 +1813,6 @@ static void acpi_video_device_notify(acp
>       device = video_device->dev;
>  
>       switch (event) {
> -     case ACPI_VIDEO_NOTIFY_SWITCH:  /* change in status (cycle output 
> device) */
> -     case ACPI_VIDEO_NOTIFY_PROBE:   /* change in status (output device 
> status) */
> -             acpi_bus_generate_event(device, event, 0);
> -             break;
>       case ACPI_VIDEO_NOTIFY_CYCLE_BRIGHTNESS:        /* Cycle brightness */
>       case ACPI_VIDEO_NOTIFY_INC_BRIGHTNESS:  /* Increase brightness */
>       case ACPI_VIDEO_NOTIFY_DEC_BRIGHTNESS:  /* Decrease brightness */
> -
> To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
> the body of a message to [EMAIL PROTECTED]
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 
-
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to