Re: [PATCH] ACPI / video: Use native backlight control interface by default

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

 



On Thu, Dec 5, 2013 at 3:32 AM, Matthew Garrett <mjg59@xxxxxxxxxxxxx> wrote:
> We had problems with the Intel backlight driver the last time we tried
> this, so let's push this through their tree for testing first? Cc:ed
> intel-gfx and Daniel Vetter.

Makes sense imo. Or if I just get a branch with just those patches (it
can rebase, I don't really care) I could  pull it into our
drm-intel-nightly integration tree, which is what everyone is testing
anyway.
-Daniel

>
>
> On Thu, Dec 05, 2013 at 10:03:57AM +0800, AceLan Kao wrote:
>> The native/vendor backlight control interface is hardly fail to control the
>> brightness, and we had encountered many backlight issues result from the broken
>> ACPI backlight interface, so I think we should promote the native backlight
>> control interface and use it by default.
>>
>> Signed-off-by: AceLan Kao <acelan.kao@xxxxxxxxxxxxx>
>> ---
>>  drivers/acpi/video.c | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/drivers/acpi/video.c b/drivers/acpi/video.c
>> index 995e91b..b3032f8 100644
>> --- a/drivers/acpi/video.c
>> +++ b/drivers/acpi/video.c
>> @@ -85,7 +85,7 @@ module_param(allow_duplicates, bool, 0644);
>>   * For Windows 8 systems: if set ture and the GPU driver has
>>   * registered a backlight interface, skip registering ACPI video's.
>>   */
>> -static bool use_native_backlight = false;
>> +static bool use_native_backlight = true;
>>  module_param(use_native_backlight, bool, 0644);
>>
>>  static int register_count;
>> --
>> 1.8.3.2
>>
>>
> --
> Matthew Garrett | mjg59@xxxxxxxxxxxxx



-- 
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux