[PATCH driver/intel] Add support for the dell backlight interface

Chris Wilson chris at chris-wilson.co.uk
Mon Jul 29 04:34:50 PDT 2013


On Mon, Jul 29, 2013 at 01:08:59PM +0200, Egbert Eich wrote:
> Signed-off-by: Egbert Eich <eich at freedesktop.org>
> ---
>  src/uxa/intel_display.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/src/uxa/intel_display.c b/src/uxa/intel_display.c
> index 0acb86d..5e8b4c3 100644
> --- a/src/uxa/intel_display.c
> +++ b/src/uxa/intel_display.c
> @@ -211,6 +211,7 @@ static const char *backlight_interfaces[] = {
>  	"acpi_video1", /* finally fallback to the generic acpi drivers */
>  	"acpi_video0",
>  	"intel_backlight",
> +	"dell_backlight",

Are you certain about the ordering? In the past we've made the firmware
specific interfaces higher priority than the platform (acpi), and had
only used the raw interface as a means of last resort. (Which apparently
thanks to Windows8 needs somehow to be revised.)
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre


More information about the xorg-devel mailing list