lkml.org 
[lkml]   [2013]   [Sep]   [22]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH v2 0/3] Fix Win8 backlight issue
From
Date
On Sun, 2013-09-22 at 17:10 +0800, Aaron Lu wrote:
> On 09/18/2013 08:36 PM, Igor Gnatenko wrote:
> > On Wed, 2013-09-18 at 20:31 +0800, Aaron Lu wrote:
> >> On 09/18/2013 02:30 PM, Igor Gnatenko wrote:
> >>> On Wed, 2013-09-18 at 09:03 +0800, Aaron Lu wrote:
> >>>> On 09/17/2013 09:34 PM, Igor Gnatenko wrote:
> >>>>>
> >>>>> Aaron, how about fix indicator on ThinkPads ?
> >>>>
> >>>> Can you please describe the problem in detail, is it that when you
> >>>> adjust brightness level through hotkey, there is no GUI indication?
> >>>> Thanks.
> >>>>
> >>>> -Aaron
> >>>
> >>> Yes. On my ThinkPad X230 I pressing backlight hotkeys. Actually
> >>> brightnes changing, but have no indicator in GUI.
> >>
> >> Oh, that's still the problem of _BCL not getting executed once for
> >> Lenovo thinkpad laptops. I borrowed a Thinkpad X1 this afternoon and can
> >> reproduce this, I'll take a look at this issue. The thinkpad-acpi module
> >> already has a call to _BCL but somehow that doesn't happen.
> >>
> >> Since it's national holidays here, I'll check this issue when I got back
> >> to work on this Saturday. Thanks for the quick test :-)
> > Thanks. No problem ;-)
>
> Here is a quick fix for thinkpad-acpi.c:
> https://github.com/aaronlu/linux acpi_video_win8
> commit thinkpad-acpi: fix handle locate for video and query of _BCL.
>
> Note that it is a separate issue specifically for thinkpad so I'll
> submit that patch in another thread.
>
> Thanks,
> Aaron
Excellent! I've tested 3 patches from this patchset + 1 latest patch
from you branch and it is works fine. Regulating and indicating works
OK.

Thank you. I think you need to make new patch-set within 4 patches.

--
Igor Gnatenko
Fedora release 20 (Heisenbug)
Linux 3.12.0-0.rc1.git0.1.fc20.x86_64



\
 
 \ /
  Last update: 2013-09-22 12:41    [W:0.112 / U:0.612 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site