From: "Pali Rohár" <pali.rohar@gmail.com>
To: Luca Tettamanti <kronos.it@gmail.com>
Cc: Artur Flinta <aflinta@gmail.com>,
linux-acpi@vger.kernel.org, dri-devel@lists.freedesktop.org,
platform-driver-x86@vger.kernel.org, joeyli <jlee@suse.com>,
Alex Deucher <alexdeucher@gmail.com>
Subject: Re: Fwd: Brightness on HP EliteBook 8460p
Date: Tue, 31 Jul 2012 18:03:02 +0200 [thread overview]
Message-ID: <5231555.Ogc4s2l7Ug@pali> (raw)
In-Reply-To: <CAKPcRGrLvsTvY2UUTmnQan4uPWk3Nar4HBn3BY4wOyePfQFXGQ@mail.gmail.com>
[-- Attachment #1: Type: text/plain, Size: 2478 bytes --]
On Tuesday 31 July 2012 17:49:32 Luca Tettamanti wrote:
> I'm putting back the CC and adding Alex.
>
> On Tue, Jul 31, 2012 at 5:07 PM, Pali Rohár
<pali.rohar@gmail.com> wrote:
> > Thanks, now working. When I write to acpi video brightness
> > file it change brightness and in dmesg is:
> >
> > [ 47.200998] [drm:radeon_atif_handler], event, device_class
> > = video, type = 0xd0
> > [ 47.201102] [drm:radeon_atif_get_sbios_requests], SBIOS
> > pending requests: 0x80
> > [ 47.201104] [drm:radeon_atif_handler], ATIF: 1 pending
> > SBIOS requests
> > [ 47.201105] [drm:radeon_atif_handler], Changing brightness
> > to 11
>
> Great! I'll send an updated patch to Alex soon.
>
> > I think that acpi only sent event about brightness key
> > pressed, because nothing happened when I pressed it.
> >
> > Also for windows is needed special HP application (hp hotkey)
> > for brightness keys. Without it brightness keys not working
> > too...
> I've looked at hp-wmi: the hotkey is indeed dispatched via WMI
> (hp-wmi) so you need something in userspace (KDE, Gnome, etc)
> to respond to that key press.
>
No, when I rmmod hp-wmi brightness keys still generate events.
And when I disable acpi then brightness keys do not generate
events but adjust brightness automatically (by BIOS).
I think that hp-wmi on my notebook only handle bluetooth & wifi
rfkills and ALS switch. All button working without hp-wmi too.
> > And there is one problem with /sys/class/backlight/radeon_bl.
> > When I enable Ambient Light Sensor which auto adjust
> > brightness based on sensor data, writing value 0 (min) or
> > 255 (max) to /sys/class/backlight/radeon_bl/brightness turn
> > off display.
> Ok, that's weird. 0 turns off the panel by design, 255 should
> not...
But when ALS is disabled 0 did not turn display off.
> > All
> > other values (1-254) are OK (adjust brightness level). When I
> > turn off Ambient Light Sensor (via hp-wmi kernel module) then
> > values 0 and 255 also set brightness level (min and max). My
> > suggestion is to convert value 0 to 1 and 255 to 254 to
> > prevent this problem.
>
> No idea what's going on here... might be some weird vendor
> magic. The WMI code is rather obscure...
>
> Luca
ACPI/WMI cannot change brightness, see this what doing that ALS
function: http://lists.freedesktop.org/archives/dri-devel/2012-
March/020416.html
--
Pali Rohár
pali.rohar@gmail.com
[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 198 bytes --]
next prev parent reply other threads:[~2012-07-31 16:03 UTC|newest]
Thread overview: 26+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-02-05 3:19 Fwd: Brightness on HP EliteBook 8460p Joey Lee
2012-03-20 22:03 ` Pali Rohár
2012-03-21 4:16 ` joeyli
2012-07-28 14:47 ` Pali Rohár
2012-07-30 8:17 ` Artur Flinta
2012-07-30 9:28 ` joeyli
2012-07-31 10:29 ` Luca Tettamanti
[not found] ` <1539163.XtGOYjTjgt@pali>
[not found] ` <CAKPcRGp+1DBjP7ZTnVUphtY2acdU64gGBi9VO5WG7+83revV9g@mail.gmail.com>
[not found] ` <3502295.b9B3oqyYO6@pali>
2012-07-31 15:49 ` Luca Tettamanti
2012-07-31 15:56 ` Alex Deucher
2012-07-31 16:03 ` Pali Rohár [this message]
2012-07-31 16:17 ` Alex Deucher
2012-08-01 7:38 ` Pali Rohár
-- strict thread matches above, loose matches on Subject: below --
2012-02-05 13:51 Joey Lee
2012-02-05 17:49 ` Artur Flinta
2012-02-05 1:43 Joey Lee
2011-07-15 12:15 Pali Rohár
[not found] ` <CAGB3EUSiZ+DqTwVQ4CMhT=LNAAWhj-VriiArmoibm0t+x5jLXw@mail.gmail.com>
2011-12-09 14:49 ` Fwd: " joeyli
[not found] ` <3404497.1txogKB5v8@pali-elitebook>
2011-12-18 10:59 ` Pali Rohár
2011-12-18 15:47 ` joeyli
2011-12-20 8:13 ` joeyli
2011-12-20 13:20 ` Pali Rohár
2011-12-21 2:16 ` joeyli
2011-12-21 4:05 ` joeyli
[not found] ` <9136713.SR1vWBOW6T@pali-elitebook>
2012-01-17 18:10 ` Pali Rohár
2012-01-20 3:12 ` joeyli
2012-01-20 3:28 ` joeyli
[not found] ` <2085914.rvJjIXUC3h@pali>
2012-02-03 15:24 ` Pali Rohár
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=5231555.Ogc4s2l7Ug@pali \
--to=pali.rohar@gmail.com \
--cc=aflinta@gmail.com \
--cc=alexdeucher@gmail.com \
--cc=dri-devel@lists.freedesktop.org \
--cc=jlee@suse.com \
--cc=kronos.it@gmail.com \
--cc=linux-acpi@vger.kernel.org \
--cc=platform-driver-x86@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.