On Friday, January 22, 2016 11:41:05 AM Hans de Goede wrote: > The quirk to get "acpi_backlight=vendor" behavior by default on the > Dell Inspiron 5737 was added before we started doing > "acpi_backlight=native" by default on Win8 ready machines. > > Since we now avoid using acpi-video as backlight driver on these machines > by default (using the native driver instead) we no longer need this quirk. > > Moreover the vendor driver does not work after a suspend/resume where > as the native driver does. > > This reverts commit 08a56226d847 ("ACPI / video: Add Dell Inspiron 5737 > to the blacklist"). > > BugLink: https://bugzilla.kernel.org/show_bug.cgi?id=111061 > Cc: stable@xxxxxxxxxxxxxxx OK, but I think we need to limit the range of "stable" kernels this is going to go into. What's the least recent "stable" series one you'd like this to be applied to? > Reported-and-tested-by: erusan@xxxxxxxxx > Signed-off-by: Hans de Goede <hdegoede@xxxxxxxxxx> > --- > drivers/acpi/video_detect.c | 8 -------- > 1 file changed, 8 deletions(-) > > diff --git a/drivers/acpi/video_detect.c b/drivers/acpi/video_detect.c > index 90e2d54..1316ddd 100644 > --- a/drivers/acpi/video_detect.c > +++ b/drivers/acpi/video_detect.c > @@ -135,14 +135,6 @@ static const struct dmi_system_id video_detect_dmi_table[] = { > DMI_MATCH(DMI_PRODUCT_NAME, "UL30A"), > }, > }, > - { > - .callback = video_detect_force_vendor, > - .ident = "Dell Inspiron 5737", > - .matches = { > - DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."), > - DMI_MATCH(DMI_PRODUCT_NAME, "Inspiron 5737"), > - }, > - }, > > /* > * These models have a working acpi_video backlight control, and using > -- I speak only for myself. Rafael J. Wysocki, Intel Open Source Technology Center. -- To unsubscribe from this list: send the line "unsubscribe stable" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html