[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