linux-fbdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: joeyli <jlee@suse.com>
To: Pradeep Subrahmanion <subrahmanion.pradeep@gmail.com>
Cc: Matthew Garrett <mjg@redhat.com>,
	rpurdie@rpsys.net, FlorianSchandinat@gmx.de,
	akpm@linux-foundation.org, linux-fbdev@vger.kernel.org,
	linux-kernel@vger.kernel.org
Subject: Re: [PATCH] Added backlight driver for Acer Aspire 4736
Date: Fri, 23 Mar 2012 04:25:39 +0000	[thread overview]
Message-ID: <1332476739.10557.438.camel@linux-s257.site> (raw)
In-Reply-To: <1332473811.3051.5.camel@debian.Gayathri>

於 五,2012-03-23 於 09:06 +0530,Pradeep Subrahmanion 提到:
> On Wed, 2012-03-21 at 11:00 +0800, joeyli wrote:
> > think the hotkey of backlight control only works with acpi_osi=Linux ?
> 
> yes , hot key control only works with this option.Any ideas about where
> the hot key events gets handled ? I tried logging inside
> 'acpi_video_device_notify' method in video.c . But the control does not
> seem to reach here . 
> 
> Thanks, 
> 
> Pradeep Subrahmanion

Hotkey change status is through _Q11 and _Q12 event but not wmi:


    Method (_Q11, 0, NotSerialized)             /* Brightness down */
    {
        If (LGreaterEqual (OSYS, 0x07D6))       /* Vista or later */
        {
            If (LEqual (OBV, 0xFF))
            {
                Notify (^^^PEGP.VGA.LCD, 0x87)
            }
            Else
            {
                Notify (^^^OVGA.DD03, 0x87)
            }
        }
        Else                                    /* 0x07D1 (XP) or 0x03E8 (Linux) */
        {
            ^^^OVGA.AINT (One, BRTS)		/* access AINT, it touch BCLP register */
            If (LEqual (^^^WMID.BAEF, One))
            {
                Store (BRTS, Local1)
                Store (^^^WMID.LBL0, Local2)
                Add (Local2, Local1, Local2)
                Store (Local2, ^^^WMID.NTDC)
                Notify (WMID, 0x80)
            }
        }
    }

Method (AINT, 2, NotSerialized)
{
...
    Else
    {
        If (LEqual (Arg0, One))         /* Linux or XP */
        {
            Add (Arg1, One, Arg1)
            Store (Divide (Multiply (Arg1, 0xFF), 0x0A, ), BCLP)
            Or (BCLP, 0x80000000, BCLP)		/* touch BCLP register */
            Store (0x02, ASLC)
        }


That's why I said acpi_osi"!Windows 2006" should also works to you. Unfortunately,
there have something wrong in video driver for support this machine on XP mode.


thanks a lot!
Joey Lee


  reply	other threads:[~2012-03-23  4:25 UTC|newest]

Thread overview: 35+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <CABNxG=CU+bOWUauLYfcS2vtFqKvXA-9axgokNoYz+KuU1Mzztw@mail.gmail.com>
2012-03-11 19:42 ` [PATCH] Added backlight driver for Acer Aspire 4736 Florian Tobias Schandinat
2012-03-12 17:36   ` Pradeep Subrahmanion
2012-03-12 17:51     ` Matthew Garrett
     [not found]       ` <1331640592.3485.50.camel@debian.Gayathri>
2012-03-13 12:47         ` Matthew Garrett
     [not found]           ` <CABNxG=Dqg26EHmC3vibf3-SjVhby1qgQfMniQObUeh9eJ6SwEw@mail.gmail.com>
2012-03-13 13:34             ` Matthew Garrett
2012-03-13 15:49               ` Pradeep Subrahmanion
2012-03-13 23:12                 ` joeyli
2012-03-14  2:55                   ` Pradeep Subrahmanion
2012-03-14  5:51                     ` joeyli
2012-03-14  6:29                       ` Pradeep Subrahmanion
2012-03-15  8:05                         ` joeyli
2012-03-18  5:22                           ` Pradeep Subrahmanion
2012-03-19  2:01                             ` joeyli
2012-03-19 11:45                               ` Pradeep Subrahmanion
2012-03-20  3:55                                 ` joeyli
2012-03-20 11:09                               ` joeyli
2012-03-20 18:55                                 ` Pradeep Subrahmanion
2012-03-21  3:00                                   ` joeyli
2012-03-21 19:21                                     ` Pradeep Subrahmanion
2012-03-22  1:33                                       ` joeyli
2012-03-22  2:45                                         ` Pradeep Subrahmanion
2012-03-22  3:25                                           ` joeyli
2012-03-22  3:44                                             ` Pradeep Subrahmanion
2012-03-22  3:54                                               ` joeyli
2012-03-22  5:57                                                 ` Pradeep Subrahmanion
2012-03-22  9:34                                                   ` joeyli
2012-03-22 16:29                                                     ` Pradeep Subrahmanion
2012-03-23  3:48                                     ` Pradeep Subrahmanion
2012-03-23  4:25                                       ` joeyli [this message]
2012-03-18  5:24                           ` Pradeep Subrahmanion
2012-03-13 13:41           ` Pradeep Subrahmanion
2012-03-12 23:07     ` Joe Perches
2012-03-12 17:40   ` Pradeep Subrahmanion
2012-03-13  3:10   ` joeyli
     [not found]     ` <1331644360.3319.6.camel@debian.Gayathri>
2012-03-13  4:35       ` joeyli

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=1332476739.10557.438.camel@linux-s257.site \
    --to=jlee@suse.com \
    --cc=FlorianSchandinat@gmx.de \
    --cc=akpm@linux-foundation.org \
    --cc=linux-fbdev@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mjg@redhat.com \
    --cc=rpurdie@rpsys.net \
    --cc=subrahmanion.pradeep@gmail.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).