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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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@xxxxxxxxxxxxxxxx>
>>> ---
>>>  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@xxxxxxxx>
>>
>>
>> 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: signature.asc
Description: OpenPGP digital signature


[Index of Archives]     [Linux Kernel Development]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux