From: Jerone Young <jerone.young@canonical.com>
To: Henrique de Moraes Holschuh <hmh@hmh.eng.br>
Cc: linux-acpi@vger.kernel.org
Subject: Re: [PATCH] Fix mute key on older Thinkpads by OSI blacklisting them
Date: Fri, 23 Apr 2010 21:19:17 -0500 [thread overview]
Message-ID: <1272075557.29792.10.camel@laptop> (raw)
In-Reply-To: <1272075251.29792.9.camel@laptop>
Errr .. Below I meant an earlier email I did on the thinkpad-acpi
mailing list a while back testing out the mute led support you had.
Thanks,
Jerone
On Fri, 2010-04-23 at 21:14 -0500, Jerone Young wrote:
> On Fri, 2010-04-23 at 23:01 -0300, Henrique de Moraes Holschuh wrote:
> > On Fri, 23 Apr 2010, Jerone Young wrote:
> > > This patch fixes that mute keys for X & R & W series Thinkpads. For
> > > these Thinkpads in the BIOS if OS exposes itself as "Linux" it will
> > > switch the functionality of the mute key to send a mute key press to the
> > > OS. These machines will not be getting any bios updates in the future.
> >
> > Lenovo is getting really cheap those days if they now kill a BIOS that
> > isn't even two years old.
> Well they just will not have any update that would change the behavior.
> But yeah the BIOS for these are done.
>
> >
> > > All of these Thinkpads do not have a little light that would indicate
> > > the mute is on. So having them send the mute key to the OS makes a
> > > better user experience and also will match behavior of newer Thinkpads.
> > > These all have the same quirk the T400 & T500 have.
> >
> > Depends on how the key behaves. Does the EC change from mute always
> > mutes (and vol up/down unmutes) mode to mute does nothing (and thus you
> > can dumb it down into a toggle) depending on OSI(Linux) ?
>
> Yeap. So by setting OSI=Linux, the EC basically just makes the mute
> button an OS key press. The OS then mutes from the driver or higher
> level. This way both speakers & headphone jack get muted.
>
>
>
> >
> > Also, should I scrap any support for Lenovo mute-only hardware mixers?
> > OSI(Linux) seems to break it badly (but I'd appreciate some extensive
> > testing of this, if you can do it... regular users often can't get
> > userspace to quiet down enough to test it without external influences
> > and I get mixed reports). Bit 6 of EC register 0x30 should control
> > speaker muting.
>
>
> I think scrapping the support would be a good idea. After discovering
> the hardware mute was only wired to the speakers, and not the headphone
> jack. It seems best to let userspace do it, and this at least it is what
> Lenovo is doing for the newer Thinkpads now. But I can also understand
> that by doing this you depend on userspace to quiet down.
>
> Well the T410 has interesting behavior. It actually does both at the
> same time. It sends the OS the mute key press & does a hardware mute to
> the speakers. This can fall out of sync of course if a user mutes from
> the userspace applet. But is easily remedied once the user presses the
> up or down volume key.
>
> Given some of the new behavior it's probably best to crap it. I did some
> testing though before on this on the acpi-devel mailing list with my
> X301. I want to send the OSI patch for that as well. But the X300 & the
> X301 have the nice hardware mute light that some people love. So I will
> just have to add the command line for my machine for a while.
>
> Thanks,
> Jerone
>
>
>
> >
>
next prev parent reply other threads:[~2010-04-24 2:19 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-04-23 22:48 [PATCH] Fix mute key on older Thinkpads by OSI blacklisting them Jerone Young
2010-04-24 2:01 ` Henrique de Moraes Holschuh
2010-04-24 2:14 ` Jerone Young
2010-04-24 2:19 ` Jerone Young [this message]
2010-04-24 3:10 ` Henrique de Moraes Holschuh
2010-04-24 4:15 ` Jerone Young
2010-04-24 16:01 ` Matthew Garrett
2010-04-24 20:49 ` Jerone Young
2010-04-24 21:16 ` Matthew Garrett
2010-04-25 0:08 ` Jerone Young
2010-04-25 2:28 ` Henrique de Moraes Holschuh
2010-04-25 18:28 ` Jerone Young
2010-04-25 18:44 ` Matthew Garrett
2010-04-25 18:57 ` Jerone Young
2010-04-25 18:59 ` Matthew Garrett
2010-04-25 19:19 ` Jerone Young
2010-04-26 10:11 ` Henrique de Moraes Holschuh
2010-04-27 5:50 ` Jerone Young
2010-04-25 19:02 ` Jerone Young
2010-04-26 10:38 ` Henrique de Moraes Holschuh
2010-04-27 5:44 ` Jerone Young
2010-04-25 0:19 ` Jerone Young
2010-05-03 11:45 ` Andy Lutomirski
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=1272075557.29792.10.camel@laptop \
--to=jerone.young@canonical.com \
--cc=hmh@hmh.eng.br \
--cc=linux-acpi@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.