linux-acpi.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] acpi-video: Add 4 new models to the use_native_backlight dmi list
@ 2014-05-01 10:41 Hans de Goede
  2014-05-01 20:16 ` Rafael J. Wysocki
  0 siblings, 1 reply; 5+ messages in thread
From: Hans de Goede @ 2014-05-01 10:41 UTC (permalink / raw)
  To: Zhang Rui, Rafael J. Wysocki, Len Brown
  Cc: Vincent Gerris, linux-acpi, Hans de Goede, stable

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@gmail.com>

HP EliteBook 8470p
https://bugzilla.redhat.com/show_bug.cgi?id=1093120

Cc: stable@vger.kernel.org
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
---
 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"),
-- 
1.9.0

^ permalink raw reply related	[flat|nested] 5+ messages in thread

* Re: [PATCH] acpi-video: Add 4 new models to the use_native_backlight dmi list
  2014-05-01 10:41 [PATCH] acpi-video: Add 4 new models to the use_native_backlight dmi list Hans de Goede
@ 2014-05-01 20:16 ` Rafael J. Wysocki
  2014-05-05  8:32   ` Aaron Lu
  0 siblings, 1 reply; 5+ messages in thread
From: Rafael J. Wysocki @ 2014-05-01 20:16 UTC (permalink / raw)
  To: Hans de Goede; +Cc: Zhang Rui, Len Brown, Vincent Gerris, linux-acpi, stable

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@gmail.com>
> 
> HP EliteBook 8470p
> https://bugzilla.redhat.com/show_bug.cgi?id=1093120
> 
> Cc: stable@vger.kernel.org
> Signed-off-by: Hans de Goede <hdegoede@redhat.com>

OK, I'll take this and drop commit 170269a9d3c0 for now.

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"),
> 

-- 
I speak only for myself.
Rafael J. Wysocki, Intel Open Source Technology Center.

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] acpi-video: Add 4 new models to the use_native_backlight dmi list
  2014-05-01 20:16 ` Rafael J. Wysocki
@ 2014-05-05  8:32   ` Aaron Lu
  2014-05-05  8:49     ` Hans de Goede
  0 siblings, 1 reply; 5+ messages in thread
From: Aaron Lu @ 2014-05-05  8:32 UTC (permalink / raw)
  To: Rafael J. Wysocki, Hans de Goede
  Cc: Zhang Rui, Len Brown, Vincent Gerris, linux-acpi, stable, edm,
	AnAkkk

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@gmail.com>
>>
>> HP EliteBook 8470p
>> https://bugzilla.redhat.com/show_bug.cgi?id=1093120
>>
>> Cc: stable@vger.kernel.org
>> Signed-off-by: Hans de Goede <hdegoede@redhat.com>
> 
> 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):

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@gmail.com>
Buglink: https://bugzilla.kernel.org/show_bug.cgi?id=51231
The 5742G is reported-and-tested-by: AnAkkk <anakin.cs@gmail.com>
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"),
>>
> 


^ permalink raw reply related	[flat|nested] 5+ messages in thread

* Re: [PATCH] acpi-video: Add 4 new models to the use_native_backlight dmi list
  2014-05-05  8:32   ` Aaron Lu
@ 2014-05-05  8:49     ` Hans de Goede
  2014-05-06  2:02       ` Aaron Lu
  0 siblings, 1 reply; 5+ messages in thread
From: Hans de Goede @ 2014-05-05  8:49 UTC (permalink / raw)
  To: Aaron Lu, Rafael J. Wysocki
  Cc: Zhang Rui, Len Brown, Vincent Gerris, linux-acpi, stable, edm,
	AnAkkk

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@gmail.com>
>>>
>>> HP EliteBook 8470p
>>> https://bugzilla.redhat.com/show_bug.cgi?id=1093120
>>>
>>> Cc: stable@vger.kernel.org
>>> Signed-off-by: Hans de Goede <hdegoede@redhat.com>
>>
>> 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.

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@gmail.com>
> Buglink: https://bugzilla.kernel.org/show_bug.cgi?id=51231
> The 5742G is reported-and-tested-by: AnAkkk <anakin.cs@gmail.com>
> 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"),
>>>
>>
> 

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] acpi-video: Add 4 new models to the use_native_backlight dmi list
  2014-05-05  8:49     ` Hans de Goede
@ 2014-05-06  2:02       ` Aaron Lu
  0 siblings, 0 replies; 5+ messages in thread
From: Aaron Lu @ 2014-05-06  2:02 UTC (permalink / raw)
  To: Hans de Goede, Rafael J. Wysocki
  Cc: Zhang Rui, Len Brown, Vincent Gerris, linux-acpi, stable, edm,
	AnAkkk

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@gmail.com>
>>>>
>>>> HP EliteBook 8470p
>>>> https://bugzilla.redhat.com/show_bug.cgi?id=1093120
>>>>
>>>> Cc: stable@vger.kernel.org
>>>> Signed-off-by: Hans de Goede <hdegoede@redhat.com>
>>>
>>> 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@gmail.com>
>> Buglink: https://bugzilla.kernel.org/show_bug.cgi?id=51231
>> The 5742G is reported-and-tested-by: AnAkkk <anakin.cs@gmail.com>
>> 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@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 


^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2014-05-06  2:01 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-05-01 10:41 [PATCH] acpi-video: Add 4 new models to the use_native_backlight dmi list Hans de Goede
2014-05-01 20:16 ` Rafael J. Wysocki
2014-05-05  8:32   ` Aaron Lu
2014-05-05  8:49     ` Hans de Goede
2014-05-06  2:02       ` Aaron Lu

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).