[PATCH v5 0/4] Fix Win8 backlight issue

Igor Gnatenko i.gnatenko.brain at gmail.com
Fri Oct 11 23:01:36 CEST 2013


On Fri, 2013-10-11 at 12:42 -0400, Josh Boyer wrote:
> On Fri, Oct 11, 2013 at 9:27 AM, Aaron Lu <aaron.lu at intel.com> wrote:
> > v5:
> > 1 Introduce video.use_native_backlight module parameter and set its
> >   value to false by default as suggested by Rafael. For Win8 systems
> >   which have broken ACPI video backlight control, the parameter can be
> >   set to 1 in kernel cmdline to skip registering ACPI video's backlight
> >   interface. Due to this change, the acpi_video_verify_backlight_support
> >   is moved from video_detect.c to video.c - patch 3/4;
> 
> That's a fairly untenable position for distro kernels to be in.  They
> now have to ask every user that reports an issue with the backlight to
> try setting that option on the command line.  While I appreciate the
> setting breaks things for some people, doesn't the Win8 issue impact
> far more people?  Shouldn't it be defaulted to true?
> 
> If nothing else, can you add a config option for the default so
> distros can use that to decide which way to default it and then work
> on fixing the remaining users that have troubles?
> 
> josh

I think more better to use this unregister as default and give option to
disable it.

-- 
Igor Gnatenko
Fedora release 20 (Heisenbug)
Linux 3.11.4-301.fc20.x86_64



More information about the dri-devel mailing list