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: Wed, 14 Mar 2012 07:12:53 +0800 [thread overview] Message-ID: <1331680373.10557.169.camel@linux-s257.site> (raw) In-Reply-To: <1331688294.2577.19.camel@debian.Gayathri> Hi Pradeep, 於 二,2012-03-13 於 21:24 -0400,Pradeep Subrahmanion 提到: > On Tue, 2012-03-13 at 13:34 +0000, Matthew Garrett wrote: > > On Tue, Mar 13, 2012 at 06:56:16PM +0530, Pradeep Subrahmanion wrote: > > > I tried giving acpi_backlight = vendor . In that case hot key for > > > brightness control is working. But i think , it is not calculating the > > > correct value for brightness because increasing brightness after maximum > > > level gives blank screen . > > > > Which backlight device appears then? > > > > 'intel_backlight' appears when i gave option acpi_backlight = vendor. Writing to /sys/class/backlight/intel_backlight/brightness > > does not cause any change in brightness. > The above command not work, that means EC didn't change backlight value: Method (_BCM, 1, NotSerialized) { Divide (Arg0, 0x0A, Local0, Local1) Decrement (Local1) Store (Local1, ^^^^LPC.EC0.BRTS) <=== write backlight value to EC register } Per my understood, EC firmware should change brightness but didn't do that, another way is touch i915 register in _BCM. Acer machine provide a broken _BCM implementation and they didn't test it. > > > By ' ACPI interface' , I mean 'acpi_video0' inside the > > > /sys/class/backlight. I havn't tried the /sys/class/backlight interface > > > directly . I will try that also. > > > > So writing values into /sys/class/backlight/acpi_video0/brightness does > > nothing? > > > No change in value when writing > to /sys/class/backlight/acpi_video0/brightness. > > Another thing is that when i did boot with acpi_backlight = 'acer_wmi' , > in new kernel (3.3.0-rc7) , it shows following messages , > > [ 8.350825] wmi: Mapper loaded > [ 10.363975] acer_wmi: Acer Laptop ACPI-WMI Extras > [ 10.396186] acer_wmi: Function bitmap for Communication Device: 0x91 > [ 10.396385] acer_wmi: Brightness must be controlled by generic video > driver > > Also there was no interface inside /sys/class/backlight for acer_wmi. > Yes, acer_wmi support backlight control with AMW0 interface, your machine didn't have AMW0 interface. Normally, backlight should control by standard acpi interface. > I also tried writing directly to Embedded controller register .But no > change. The machine has broken _BCM method, because EC should do something after _BCM changed EC register. > ---- > > Thanks , > > Pradeep Subrahmanion > Thanks a lot! Joey Lee
WARNING: multiple messages have this Message-ID (diff)
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: Tue, 13 Mar 2012 23:12:53 +0000 [thread overview] Message-ID: <1331680373.10557.169.camel@linux-s257.site> (raw) In-Reply-To: <1331688294.2577.19.camel@debian.Gayathri> Hi Pradeep, 於 二,2012-03-13 於 21:24 -0400,Pradeep Subrahmanion 提到: > On Tue, 2012-03-13 at 13:34 +0000, Matthew Garrett wrote: > > On Tue, Mar 13, 2012 at 06:56:16PM +0530, Pradeep Subrahmanion wrote: > > > I tried giving acpi_backlight = vendor . In that case hot key for > > > brightness control is working. But i think , it is not calculating the > > > correct value for brightness because increasing brightness after maximum > > > level gives blank screen . > > > > Which backlight device appears then? > > > > 'intel_backlight' appears when i gave option acpi_backlight = vendor. Writing to /sys/class/backlight/intel_backlight/brightness > > does not cause any change in brightness. > The above command not work, that means EC didn't change backlight value: Method (_BCM, 1, NotSerialized) { Divide (Arg0, 0x0A, Local0, Local1) Decrement (Local1) Store (Local1, ^^^^LPC.EC0.BRTS) <== write backlight value to EC register } Per my understood, EC firmware should change brightness but didn't do that, another way is touch i915 register in _BCM. Acer machine provide a broken _BCM implementation and they didn't test it. > > > By ' ACPI interface' , I mean 'acpi_video0' inside the > > > /sys/class/backlight. I havn't tried the /sys/class/backlight interface > > > directly . I will try that also. > > > > So writing values into /sys/class/backlight/acpi_video0/brightness does > > nothing? > > > No change in value when writing > to /sys/class/backlight/acpi_video0/brightness. > > Another thing is that when i did boot with acpi_backlight = 'acer_wmi' , > in new kernel (3.3.0-rc7) , it shows following messages , > > [ 8.350825] wmi: Mapper loaded > [ 10.363975] acer_wmi: Acer Laptop ACPI-WMI Extras > [ 10.396186] acer_wmi: Function bitmap for Communication Device: 0x91 > [ 10.396385] acer_wmi: Brightness must be controlled by generic video > driver > > Also there was no interface inside /sys/class/backlight for acer_wmi. > Yes, acer_wmi support backlight control with AMW0 interface, your machine didn't have AMW0 interface. Normally, backlight should control by standard acpi interface. > I also tried writing directly to Embedded controller register .But no > change. The machine has broken _BCM method, because EC should do something after _BCM changed EC register. > ---- > > Thanks , > > Pradeep Subrahmanion > Thanks a lot! Joey Lee
next prev parent reply other threads:[~2012-03-13 23:14 UTC|newest] Thread overview: 69+ 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-11 19:42 ` Florian Tobias Schandinat 2012-03-12 17:36 ` Pradeep Subrahmanion 2012-03-13 3:12 ` Pradeep Subrahmanion 2012-03-12 17:51 ` Matthew Garrett 2012-03-13 12:09 ` Pradeep Subrahmanion 2012-03-13 12:47 ` Matthew Garrett 2012-03-13 13:29 ` Pradeep Subrahmanion 2012-03-13 13:41 ` Pradeep Subrahmanion [not found] ` <CABNxG=Dqg26EHmC3vibf3-SjVhby1qgQfMniQObUeh9eJ6SwEw@mail.gmail.com> 2012-03-13 13:34 ` Matthew Garrett 2012-03-13 15:49 ` Pradeep Subrahmanion 2012-03-14 1:24 ` Pradeep Subrahmanion 2012-03-13 23:12 ` joeyli [this message] 2012-03-13 23:12 ` joeyli 2012-03-14 2:43 ` Pradeep Subrahmanion 2012-03-14 2:55 ` Pradeep Subrahmanion 2012-03-14 5:51 ` joeyli 2012-03-14 5:51 ` joeyli 2012-03-14 6:17 ` Pradeep Subrahmanion 2012-03-14 6:29 ` Pradeep Subrahmanion 2012-03-15 8:05 ` joeyli 2012-03-15 8:05 ` joeyli 2012-03-18 5:10 ` Pradeep Subrahmanion 2012-03-18 5:22 ` Pradeep Subrahmanion 2012-03-19 2:01 ` joeyli 2012-03-19 2:01 ` joeyli 2012-03-19 11:33 ` Pradeep Subrahmanion 2012-03-19 11:45 ` Pradeep Subrahmanion 2012-03-20 3:55 ` joeyli 2012-03-20 3:55 ` joeyli 2012-03-20 11:09 ` joeyli 2012-03-20 11:09 ` joeyli 2012-03-20 18:55 ` Pradeep Subrahmanion 2012-03-20 18:55 ` Pradeep Subrahmanion 2012-03-21 3:00 ` joeyli 2012-03-21 3:00 ` joeyli 2012-03-21 19:09 ` Pradeep Subrahmanion 2012-03-21 19:21 ` Pradeep Subrahmanion 2012-03-22 1:33 ` joeyli 2012-03-22 1:33 ` joeyli 2012-03-22 2:33 ` Pradeep Subrahmanion 2012-03-22 2:45 ` Pradeep Subrahmanion 2012-03-22 3:25 ` joeyli 2012-03-22 3:25 ` joeyli 2012-03-22 3:32 ` Pradeep Subrahmanion 2012-03-22 3:44 ` Pradeep Subrahmanion 2012-03-22 3:54 ` joeyli 2012-03-22 3:54 ` joeyli 2012-03-22 5:56 ` Pradeep Subrahmanion 2012-03-22 5:57 ` Pradeep Subrahmanion 2012-03-22 9:34 ` joeyli 2012-03-22 9:34 ` joeyli 2012-03-22 16:17 ` Pradeep Subrahmanion 2012-03-22 16:29 ` Pradeep Subrahmanion 2012-03-23 3:36 ` Pradeep Subrahmanion 2012-03-23 3:48 ` Pradeep Subrahmanion 2012-03-23 4:25 ` joeyli 2012-03-23 4:25 ` joeyli 2012-03-18 5:12 ` Pradeep Subrahmanion 2012-03-18 5:24 ` Pradeep Subrahmanion 2012-03-12 23:07 ` Joe Perches 2012-03-12 23:07 ` Joe Perches 2012-03-12 17:40 ` Pradeep Subrahmanion 2012-03-13 3:16 ` Pradeep Subrahmanion 2012-03-13 3:10 ` joeyli 2012-03-13 3:10 ` joeyli 2012-03-13 13:12 ` Pradeep Subrahmanion 2012-03-13 4:35 ` joeyli 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=1331680373.10557.169.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: linkBe 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.