All of lore.kernel.org
 help / color / mirror / Atom feed
From: Zhang Rui <rui.zhang@intel.com>
To: Chris Bagwell <chris@cnpbagwell.com>
Cc: Matthew Garrett <mjg59@srcf.ucam.org>,
	Alan Jenkins <sourcejedi.lkml@googlemail.com>,
	ACPI Devel Maling List <linux-acpi@vger.kernel.org>,
	"acpi4asus-user@lists.sourceforge.net"
	<acpi4asus-user@lists.sourceforge.net>,
	"platform-driver-x86@vger.kernel.org"
	<platform-driver-x86@vger.kernel.org>
Subject: Re: [Acpi4asus-user] 1005PE's and backlight controls
Date: Thu, 15 Apr 2010 10:29:13 +0800	[thread overview]
Message-ID: <1271298553.7167.230.camel@rzhang1-desktop> (raw)
In-Reply-To: <j2vdb599ead1004141910r2a3206f8nb17d055db0ba2959@mail.gmail.com>

On Thu, 2010-04-15 at 10:10 +0800, Chris Bagwell wrote:
> On Wed, Apr 14, 2010 at 8:42 PM, Zhang Rui <rui.zhang@intel.com> wrote:
> > On Thu, 2010-04-15 at 09:30 +0800, Zhang Rui wrote:
> >> On Thu, 2010-04-15 at 06:18 +0800, Chris Bagwell wrote:
> >> > On Wed, Apr 14, 2010 at 12:26 PM, Matthew Garrett <mjg59@srcf.ucam.org> wrote:
> >> > > On Wed, Apr 14, 2010 at 12:14:04PM -0500, Chris Bagwell wrote:
> >> > >
> >> > >> Does the acpi-video logic not have logic on its own to send key
> >> > >> events?  So I guess laptops that don't have custom modules to handle
> >> > >> this type of stuff don't get visual feedback from gnome-power-manager?
> >> > >
> >> > > It does, but it's dependent upon the firmware sending them.
> >> >
> >> > I think the following tells me firmware is sending them.  If I leave
> >> > acpi_osi="Windows 2009" so that eeepc_laptop doesn't get loaded, I see
> >> > events like this on /pro/acpi/event:
> >> >
> >> > video LCDD 00000087 00000000
> >> > video LCDD 00000087 00000000
> >> > video LCDD 00000086 00000000
> >> > video LCDD 00000086 00000000
> >> >
> >> > Thats a couple decreases followed by a couple increases.
> >> >
> >> I have a EEEpc 1005PE. I'm looking at the backlight problem on this
> >> machine, but it seems to be different from this one.
> >>
> >> The hotkey seems to work perfectly when ACPI video driver is loaded.
> >> i.e. I get a single hotkey event when pressing the hotkey and the value
> >> of /sys/class/backlight/acpi_video0/brightness changes correctly.
> >>
> >> The only problem I get is that the actual brightness does not change
> >> consistently.
> >> say, there are 15 brightness levels in all,
> >> level 0, level 5 and level 12 give me a screen with lowest brightness.
> >> If I want to get maximum backlight, I need to set it to level 4 or level
> >> 11.
> 
> I'm curious, do you still see this issue if you first kill
> gnome-power-manager first?
> 
> >>
> > Oh, this have already been fixed in the latest BIOS.
> 
> Wasn't fixed in my case.  I'd be curious to here if it fixes for you.
> 
no, the problem still exists.
I misread your comment at
https://bugzilla.kernel.org/show_bug.cgi?id=15182#c33

> >
> > Chris,
> > do you mean you get duplicate hotkey events after upgrading the BIOS?
> 
> With latest firmware, I get zero hotkey events over /dev/input/event*
> unless I add acpi_backlight=vendor to boot options.

I just upgrade the BIOS to 1003, and I can still get input event
from /dev/input/event4 (which is ACPI video bus).
and there is no duplicate input events when pressing hotkey.

thanks,
rui


  reply	other threads:[~2010-04-15  2:28 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <y2g3b2dfeb01004131934x67e23d80vc73dfd6d5af133fa@mail.gmail.com>
     [not found] ` <o2z71cd59b01004132316jd06b2a40m593ee5376e4dfa28@mail.gmail.com>
     [not found]   ` <r2odb599ead1004140624g56c2465qc4aaa3d1ee1c79b3@mail.gmail.com>
2010-04-14 13:45     ` [Acpi4asus-user] 1005PE's and backlight controls Corentin Chary
2010-04-14 14:00       ` Chris Bagwell
2010-04-15  3:34         ` Wang, Yong Y
2010-04-15  4:35           ` Wang, Yong Y
     [not found] ` <y2g3b2dfeb01004131934x67e23d80vc73dfd6d5af133fa-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2010-04-14 16:23   ` Alan Jenkins
2010-04-14 16:40     ` [Acpi4asus-user] " Matthew Garrett
2010-04-14 17:14       ` Chris Bagwell
2010-04-14 17:26         ` Matthew Garrett
2010-04-14 22:18           ` Chris Bagwell
2010-04-15  1:30             ` Zhang Rui
2010-04-15  1:42               ` Zhang Rui
2010-04-15  2:10                 ` [Acpi4asus-user] " Chris Bagwell
2010-04-15  2:29                   ` Zhang Rui [this message]
2010-04-16  0:02                     ` Chris Bagwell
2010-04-15  2:00               ` Chris Bagwell
2010-04-14 20:07       ` Corentin Chary

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=1271298553.7167.230.camel@rzhang1-desktop \
    --to=rui.zhang@intel.com \
    --cc=acpi4asus-user@lists.sourceforge.net \
    --cc=chris@cnpbagwell.com \
    --cc=linux-acpi@vger.kernel.org \
    --cc=mjg59@srcf.ucam.org \
    --cc=platform-driver-x86@vger.kernel.org \
    --cc=sourcejedi.lkml@googlemail.com \
    /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.