On 05/05/2014 04:49 PM, Hans de Goede wrote: > Hi Aaron, > > On 05/05/2014 10:32 AM, Aaron Lu wrote: >> On 05/02/2014 04:16 AM, Rafael J. Wysocki wrote: >>> On Thursday, May 01, 2014 12:41:19 PM Hans de Goede wrote: >>>> Acer Aspire V5-171 >>>> https://bugzilla.redhat.com/show_bug.cgi?id=983342 >>>> >>>> Acer Aspire V5-471G >>>> Lenovo Yoga 2 11 >>>> Reported-and-tested-by: Vincent Gerris <vgerris@xxxxxxxxx> >>>> >>>> HP EliteBook 8470p >>>> https://bugzilla.redhat.com/show_bug.cgi?id=1093120 >>>> >>>> Cc: stable@xxxxxxxxxxxxxxx >>>> Signed-off-by: Hans de Goede <hdegoede@xxxxxxxxxx> >>> >>> OK, I'll take this and drop commit 170269a9d3c0 for now. >> >> There are two additional machines that need to go to the quirk table: >> ThinkPad T430 and Acer Aspire 5742G, I didn't find a proper branch >> to base the quirk patch on, is this patch going to be merged soon? >> >> BTW, Hans, it would be great if you can take the two machines into >> your patch too(the below diff is on top of v3.15-rc4): > > I was just preparing a patch adding a quirk for the T530, I'll add > these 2 to the same patch. Thanks a lot for your kind help :-) -Aaron > > Note my new patch applies on top of: > acpi-video: Add 4 new models to the use_native_backlight dmi list > acpi: mv models with win8 brightness problems from win8 blacklist to use_native_backlight > > Thanks & Regards, > > Hans > > >> >> diff --git a/drivers/acpi/video.c b/drivers/acpi/video.c >> index b6ba88ed31ae..ddf9eeb6235b 100644 >> --- a/drivers/acpi/video.c >> +++ b/drivers/acpi/video.c >> @@ -459,10 +459,10 @@ static struct dmi_system_id video_dmi_table[] __initdata = { >> }, >> { >> .callback = video_set_use_native_backlight, >> - .ident = "ThinkPad T430s", >> + .ident = "ThinkPad T430 and T430s", >> .matches = { >> DMI_MATCH(DMI_SYS_VENDOR, "LENOVO"), >> - DMI_MATCH(DMI_PRODUCT_VERSION, "ThinkPad T430s"), >> + DMI_MATCH(DMI_PRODUCT_VERSION, "ThinkPad T430"), >> }, >> }, >> { >> @@ -571,6 +571,14 @@ static struct dmi_system_id video_dmi_table[] __initdata = { >> DMI_MATCH(DMI_PRODUCT_NAME, "HP EliteBook 8780w"), >> }, >> }, >> + { >> + .callback = video_set_use_native_backlight, >> + .ident = "Acer Aspire 5742G", >> + .matches = { >> + DMI_MATCH(DMI_SYS_VENDOR, "Acer"), >> + DMI_MATCH(DMI_PRODUCT_NAME, "Aspire 5742G"), >> + }, >> + }, >> {} >> }; >> >> The T430 is reported-and-tested-by: edm <fuffi.il.fuffo@xxxxxxxxx> >> Buglink: https://bugzilla.kernel.org/show_bug.cgi?id=51231 >> The 5742G is reported-and-tested-by: AnAkkk <anakin.cs@xxxxxxxxx> >> Buglink: https://bugzilla.kernel.org/show_bug.cgi?id=35622 >> >> But if you do not want to do that, feel free to let me know, I'll >> prepare a patch then. >> >> Thanks, >> Aaron >> >>> >>> Thanks! >>> >>>> --- >>>> drivers/acpi/video.c | 32 ++++++++++++++++++++++++++++++++ >>>> 1 file changed, 32 insertions(+) >>>> >>>> diff --git a/drivers/acpi/video.c b/drivers/acpi/video.c >>>> index 8b6990e..48146fc 100644 >>>> --- a/drivers/acpi/video.c >>>> +++ b/drivers/acpi/video.c >>>> @@ -488,6 +488,14 @@ static struct dmi_system_id video_dmi_table[] __initdata = { >>>> }, >>>> }, >>>> { >>>> + .callback = video_set_use_native_backlight, >>>> + .ident = "Lenovo Yoga 2 11", >>>> + .matches = { >>>> + DMI_MATCH(DMI_SYS_VENDOR, "LENOVO"), >>>> + DMI_MATCH(DMI_PRODUCT_VERSION, "Lenovo Yoga 2 11"), >>>> + }, >>>> + }, >>>> + { >>>> .callback = video_set_use_native_backlight, >>>> .ident = "Thinkpad Helix", >>>> .matches = { >>>> @@ -513,6 +521,14 @@ static struct dmi_system_id video_dmi_table[] __initdata = { >>>> }, >>>> { >>>> .callback = video_set_use_native_backlight, >>>> + .ident = "Acer Aspire V5-171", >>>> + .matches = { >>>> + DMI_MATCH(DMI_SYS_VENDOR, "Acer"), >>>> + DMI_MATCH(DMI_PRODUCT_NAME, "V5-171"), >>>> + }, >>>> + }, >>>> + { >>>> + .callback = video_set_use_native_backlight, >>>> .ident = "Acer Aspire V5-431", >>>> .matches = { >>>> DMI_MATCH(DMI_SYS_VENDOR, "Acer"), >>>> @@ -520,6 +536,14 @@ static struct dmi_system_id video_dmi_table[] __initdata = { >>>> }, >>>> }, >>>> { >>>> + .callback = video_set_use_native_backlight, >>>> + .ident = "Acer Aspire V5-471G", >>>> + .matches = { >>>> + DMI_MATCH(DMI_BOARD_VENDOR, "Acer"), >>>> + DMI_MATCH(DMI_PRODUCT_NAME, "Aspire V5-471G"), >>>> + }, >>>> + }, >>>> + { >>>> .callback = video_set_use_native_backlight, >>>> .ident = "HP ProBook 4340s", >>>> .matches = { >>>> @@ -571,6 +595,14 @@ static struct dmi_system_id video_dmi_table[] __initdata = { >>>> }, >>>> { >>>> .callback = video_set_use_native_backlight, >>>> + .ident = "HP EliteBook 8470p", >>>> + .matches = { >>>> + DMI_MATCH(DMI_SYS_VENDOR, "Hewlett-Packard"), >>>> + DMI_MATCH(DMI_PRODUCT_NAME, "HP EliteBook 8470p"), >>>> + }, >>>> + }, >>>> + { >>>> + .callback = video_set_use_native_backlight, >>>> .ident = "HP EliteBook 8780w", >>>> .matches = { >>>> DMI_MATCH(DMI_SYS_VENDOR, "Hewlett-Packard"), >>>> >>> >> > -- > To unsubscribe from this list: send the line "unsubscribe linux-acpi" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html > -- To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
