Re: [PATCH] acpi-video: Add use native backlight quirk for the ThinkPad W530

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

 



Hi,

On 05/12/2014 10:16 AM, Aaron Lu wrote:
> On 05/12/2014 03:57 PM, Hans de Goede wrote:
>> Like all of the other *30 ThinkPad models, the W530 has a broken acpi-video
>> backlight control. Note in order for this to actually fix things on the
>> ThinkPad W530 the commit titled:
>> "nouveau: Don't check acpi_video_backlight_support() before registering backlight"
>> is also needed.
> 
> Note that the backlight_device_registered(raw) may return false as when
> acpi_video_verify_backlight_support is called, the nouveau driver may
> not run yet.
> 
> Previously, we don't know anything about how laptops with nvidia graphics
> card alone control backlight in Win8, so the existing solution doesn't
> consider this case. If nvidia graphics system also should favour native
> backlight control interface in Win8, the current solution needs
> modifications.

Hmm, how is this dealt with in the case of the intel gfx driver ?

Regards,

Hans



> 
> Thanks,
> Aaron
> 
>>
>> https://bugzilla.redhat.com/show_bug.cgi?id=1093171
>>
>> Cc: stable@xxxxxxxxxxxxxxx
>> Signed-off-by: Hans de Goede <hdegoede@xxxxxxxxxx>
>> ---
>>  drivers/acpi/video.c | 8 ++++++++
>>  1 file changed, 8 insertions(+)
>>
>> diff --git a/drivers/acpi/video.c b/drivers/acpi/video.c
>> index 34198b2..6a2099d 100644
>> --- a/drivers/acpi/video.c
>> +++ b/drivers/acpi/video.c
>> @@ -520,6 +520,14 @@ static struct dmi_system_id video_dmi_table[] __initdata = {
>>  		},
>>  	},
>>  	{
>> +	 .callback = video_set_use_native_backlight,
>> +	 .ident = "ThinkPad W530",
>> +	 .matches = {
>> +		DMI_MATCH(DMI_SYS_VENDOR, "LENOVO"),
>> +		DMI_MATCH(DMI_PRODUCT_VERSION, "ThinkPad W530"),
>> +		},
>> +	},
>> +	{
>>  	.callback = video_set_use_native_backlight,
>>  	.ident = "ThinkPad X1 Carbon",
>>  	.matches = {
>>
> 
--
To unsubscribe from this list: send the line "unsubscribe stable" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




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