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

Egbert Eich eich at suse.com
Mon Jul 29 04:49:49 PDT 2013


Chris Wilson writes:
 > 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.)

You are right of course. In the patches we use for openSUSE it is right
at the beginning of the list. I hand't paid attention to this when I copied
it over. Will resend.

Cheers,
	Egbert.


More information about the xorg-devel mailing list