Re: [PATCH] acer-wmi: add video_vendor quirk for Acer Aspire E1-572

[Oleksij Rempel <linux@rempel-privat.de>]

[-- Attachment #1: Type: text/plain, Size: 1799 bytes --]

Am 27.12.2013 05:25, schrieb joeyli:
> Hi Oleksij, 
> 
> 於 五，2013-12-27 於 11:51 +0800，joeyli 提到：
>> Hi Oleksij, 
>>
>> 於 四，2013-12-05 於 14:00 +0100，Oleksij Rempel 提到：
>>> Signed-off-by: Oleksij Rempel <linux@rempel-privat.de>
>>> ---
>>>  drivers/platform/x86/acer-wmi.c | 8 ++++++++
>>>  1 file changed, 8 insertions(+)
>>>
>>> diff --git a/drivers/platform/x86/acer-wmi.c b/drivers/platform/x86/acer-wmi.c
>>> index c9076bd..d5ca1648 100644
>>> --- a/drivers/platform/x86/acer-wmi.c
>>> +++ b/drivers/platform/x86/acer-wmi.c
>>> @@ -572,6 +572,14 @@ static const struct dmi_system_id video_vendor_dmi_table[] = {
>>>  			DMI_MATCH(DMI_PRODUCT_NAME, "Aspire 5750"),
>>>  		},
>>>  	},
>>> +	{
>>> +		.callback = video_set_backlight_video_vendor,
>>> +		.ident = "Acer Aspire E1-572",
>>> +		.matches = {
>>> +			DMI_MATCH(DMI_BOARD_VENDOR, "Acer"),
>>> +			DMI_MATCH(DMI_PRODUCT_NAME, "Aspire E1-572"),
>>> +		},
>>> +	},
>>>  	{}
>>>  };
>>>  
>>
>> I have applied this patch, build and load acer-wmi driver, didn't see
>> problem.
>>
>> Acked-by: Lee, Chun-Yi <jlee@suse.com>
>>
>>
>> Thanks for your patch!
>> Joey Lee
> 
> I just aware Aspire E1-572 shipped with Windows 8. Before merge this
> patch. Could you please try v3.13-rc5 kernel with
> video.use_native_backlight=1 kernel parameter?
> 
> Aaron Lu's "[PATCH v5 0/4] Fix Win8 backlight issue" merged to mainline
> in v3.13-rc1, his patches works to me on a Gateway Windows 8 notebook.
> 
> Please try it, if Aaron's patches works on E1-572, then we don't need
> this patch.

Hi,

suddenly i do not have access to this Laptops any more. It was a
purchase for a friend. I tested it, provided some patch and gave it away :(

-- 
Regards,
Oleksij


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 295 bytes --]