[PATCH] intel_display.c: make acpi_video last in backlight_interfaces

Dave Airlie airlied at gmail.com
Sun Aug 26 04:46:01 PDT 2012


On Sun, Aug 26, 2012 at 9:43 PM, Alon Levy <alevy at redhat.com> wrote:
> ACPI is meant as a fallback, so it should be last.

eh no, acpi is never the fallback, its always the preferred way.

Dave.

>
> This allows my Lenovo T510 to reach the minimal backlight (switched
> off), by using intel_backlight instead of acpi.
>
> Signed-off-by: Alon Levy <alevy at redhat.com>
> ---
>  src/intel_display.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/intel_display.c b/src/intel_display.c
> index 6580c8c..7f34172 100644
> --- a/src/intel_display.c
> +++ b/src/intel_display.c
> @@ -136,9 +136,9 @@ static const char *backlight_interfaces[] = {
>         "fujitsu-laptop",
>         "sony",
>         "samsung",
> +       "intel_backlight",
>         "acpi_video1", /* finally fallback to the generic acpi drivers */
>         "acpi_video0",
> -       "intel_backlight",
>         NULL,
>  };
>  /*
> --
> 1.7.12
>
> _______________________________________________
> xorg-devel at lists.x.org: X.Org development
> Archives: http://lists.x.org/archives/xorg-devel
> Info: http://lists.x.org/mailman/listinfo/xorg-devel


More information about the xorg-devel mailing list