[Acked] [PATCH] [trusty] SAUCE: Add use_native_backlight quirk for Dell Inspiron 5721/3521

[Seth Forshee]

On Thu, Sep 04, 2014 at 11:30:01AM +0100, Andy Whitcroft wrote:
> On Thu, Sep 04, 2014 at 12:28:49PM +0800, Edward Lin wrote:
> > BugLink: http://bugs.launchpad.net/bugs/1354313
> > BugLink: http://bugs.launchpad.net/bugs/1354253
> > 
> > Signed-off-by: Edward Lin <yidi.lin at canonical.com>
> > ---
> >  drivers/acpi/video.c |   16 ++++++++++++++++
> >  1 file changed, 16 insertions(+)
> > 
> > diff --git a/drivers/acpi/video.c b/drivers/acpi/video.c
> > index ffdd2c6..e858463 100644
> > --- a/drivers/acpi/video.c
> > +++ b/drivers/acpi/video.c
> > @@ -516,6 +516,22 @@ static struct dmi_system_id video_dmi_table[] __initdata = {
> >  	},
> >  	{
> >  	 .callback = video_set_use_native_backlight,
> > +	 .ident = "Dell Inspiron 5721",
> > +	 .matches = {
> > +		DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."),
> > +		DMI_MATCH(DMI_PRODUCT_NAME, "Inspiron 5721"),
> > +		},
> > +	},
> > +	{
> > +	 .callback = video_set_use_native_backlight,
> > +	 .ident = "Dell Inspiron 3521",
> > +	 .matches = {
> > +		DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."),
> > +		DMI_MATCH(DMI_PRODUCT_NAME, "Inspiron 3521"),
> > +		},
> > +	},
> > +	{
> > +	 .callback = video_set_use_native_backlight,
> >  	 .ident = "Acer Aspire 5733Z",
> >  	 .matches = {
> >  		DMI_MATCH(DMI_SYS_VENDOR, "Acer"),
> 
> These look safe and self contained.  I assume these are on their way
> upstream?

I agree.

Acked-by: Seth Forshee <seth.forshee at canonical.com>

Probably not applicable for upstream though. Upstream switched to make
video.use_native_backlight default to 1, and now seems to be going
through the process of quirking machines which need it to be 0
(https://www.mail-archive.com/stable@vger.kernel.org/msg91591.html).