Comment 307 for bug 1098216

(In reply to Yves-Alexis Perez from comment #180)
> (In reply to Aaron Lu from comment #179)
> > Hi Perez,
> >
> > I believe commit 5f24079b021cd3147c8d24ba65833f7a0df7e80d is meant to fix
> > such problems:
> >
> > commit 5f24079b021cd3147c8d24ba65833f7a0df7e80d
> > Author: Hans de Goede <email address hidden>
> > Date: Thu Aug 28 10:20:46 2014 +0200
> >
> > ACPI / video: Add a disable_native_backlight quirk
> >
> > Some laptops have a working acpi_video backlight control, and using
> > native
> > backlight on these causes a regression where backlight control does not
> > work
> > when userspace is not handling brightness key events. Disable
> > native_backlight
> > on these to fix this.
> >
> > Link: https://bugzilla.kernel.org/show_bug.cgi?id=81691
> > Reported-and-tested-by: Andre Müller <email address hidden>
> > Cc: 3.16+ <email address hidden> # 3.16+
> > Signed-off-by: Hans de Goede <email address hidden>
> > Signed-off-by: Rafael J. Wysocki <email address hidden>
> >
> > Feel free to submit a patch to add your model into the video_dmi_table to
> > disable native backlight control.
>
> Thanks. So the “DMI table” approach was considered bad because not
> sustainable for this bug, but ok for bug #81691 ?

The systems that claim to be Win8 but have a working ACPI video interface are supposed to be not common(hopefully this is true), so should be OK to keep a DMI table for them. While the systems that claim to be Win8 and don't have a working ACPI video interface are supposed to be the common case, so keeping a DMI table for all of them is not feasible.

>
> Anyway, will provide the correct DMI information on that bug.

OK, thanks.