<div dir="ltr">Backlight can't be modified with this patch set -  neither with function keys nor<div>with the gui. It is a step backward to v3.11-rc1</div><div><br></div><div>Video driver: i915<br></div><div>FUJITSU LIFEBOOK AH532/FJNBB1C, BIOS Version 1.09 05/22/2012<br>
</div><div><br></div><div>acpi_backlight=video works.</div><div><br></div><div>Jörg</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">2013/9/24 Igor Gnatenko <span dir="ltr"><<a href="mailto:i.gnatenko.brain@gmail.com" target="_blank">i.gnatenko.brain@gmail.com</a>></span><br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On Tue, 2013-09-24 at 17:47 +0800, Aaron Lu wrote:<br>
> v3:<br>
> 1 Add a new patch 4/4 to fix some problems in thinkpad-acpi module;<br>
> 2 Remove unnecessary function acpi_video_unregister introduced in<br>
>   patch 2/3 as pointed out by Jani Nikula.<br>
><br>
> v2:<br>
> v1 has the subject of "Rework ACPI video driver" and is posted here:<br>
> <a href="http://lkml.org/lkml/2013/9/9/74" target="_blank">http://lkml.org/lkml/2013/9/9/74</a><br>
> Since the objective is really to fix Win8 backlight issues, I changed<br>
> the subject in this version, sorry about that.<br>
><br>
> This patchset has three patches, the first introduced a new API named<br>
> backlight_device_registered in backlight layer that can be used for<br>
> backlight interface provider module to check if a specific type backlight<br>
> interface has been registered, see changelog for patch 1/3 for details.<br>
> Then patch 2/3 does the cleanup to sepeate the backlight control and<br>
> event delivery functionality in the ACPI video module and patch 3/3<br>
> solves some Win8 backlight control problems by avoiding register ACPI<br>
> video's backlight interface if:<br>
> 1 Kernel cmdline option acpi_backlight=video is not given;<br>
> 2 This is a Win8 system;<br>
> 3 Native backlight control interface exists.<br>
><br>
> Technically, patch 2/3 is not required to fix the issue here. So if you<br>
> think it is not necessary, I can remove it from the series.<br>
><br>
> Aaron Lu (4):<br>
>   backlight: introduce backlight_device_registered<br>
>   ACPI / video: seperate backlight control and event interface<br>
>   ACPI / video: Do not register backlight if win8 and native interface<br>
>     exists<br>
>   thinkpad-acpi: fix handle locate for video and query of _BCL<br>
><br>
>  drivers/acpi/internal.h              |   5 +-<br>
>  drivers/acpi/video.c                 | 442 ++++++++++++++++++++---------------<br>
>  drivers/acpi/video_detect.c          |  14 +-<br>
>  drivers/platform/x86/thinkpad_acpi.c |  31 ++-<br>
>  drivers/video/backlight/backlight.c  |  31 +++<br>
>  include/acpi/video.h                 |   2 +<br>
>  include/linux/backlight.h            |   4 +<br>
>  7 files changed, 324 insertions(+), 205 deletions(-)<br>
><br>
<br>
</div></div>Excellent! I've tested this patch-set on ThinkPad X230 and backlight<br>
issue is exhausted.<br>
<br>
Thanks.<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
Igor Gnatenko<br>
Fedora release 20 (Heisenbug)<br>
Linux 3.11.1-300.fc20.x86_64<br>
<br>
</font></span></blockquote></div><br></div>