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 <[email protected]>
>>>>
>>>> HP EliteBook 8470p
>>>> https://bugzilla.redhat.com/show_bug.cgi?id=1093120
>>>>
>>>> Cc: [email protected]
>>>> Signed-off-by: Hans de Goede <[email protected]>
>>>
>>> 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 <[email protected]>
>> Buglink: https://bugzilla.kernel.org/show_bug.cgi?id=51231
>> The 5742G is reported-and-tested-by: AnAkkk <[email protected]>
>> 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 [email protected]
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
--
To unsubscribe from this list: send the line "unsubscribe stable" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
