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 02.01.2014 13:39, schrieb joeyli:
> 於 三,2014-01-01 於 15:37 +0100,Oleksij Rempel 提到:
>> Am 27.12.2013 05:25, schrieb joeyli:
> [...]
>>>>>  		},
>>>>>  	},
>>>>> +	{
>>>>> +		.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 :(
>>
> 
> I think Aaron's patches can fix it on E1-572. Let's wait until v3.13
> version released then look at maybe someone report issue again.
> 
> Very appreciate for your patch.
> 
> Joey Lee
> 

Hi Joey,

i have access to this hardware for next two weeks. Theoretically this
kernel should have patches you was talking about:
http://kernel.ubuntu.com/~kernel-ppa/mainline/daily/2014-01-06-trusty/

it works with video.use_native_backlight=1.

If you need more testing, you have two weeks ;)

-- 
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