public inbox for linux-acpi@vger.kernel.org
 help / color / mirror / Atom feed
From: Andrew Lutomirski <luto@mit.edu>
To: Henrique de Moraes Holschuh <hmh@hmh.eng.br>
Cc: Matthew Garrett <mjg59@srcf.ucam.org>,
	linux-thinkpad@linux-thinkpad.org,
	ACPI Devel Maling List <linux-acpi@vger.kernel.org>
Subject: Re: [ltp] Re: [PATCH] Set osi=Linux for the ThinkPad X200s
Date: Tue, 10 Nov 2009 08:32:17 -0500	[thread overview]
Message-ID: <cb0375e10911100532l484e69aeyd78e422c23d730ce@mail.gmail.com> (raw)
In-Reply-To: <20091110092418.GC1872@khazad-dum.debian.net>

On Tue, Nov 10, 2009 at 4:24 AM, Henrique de Moraes Holschuh
<hmh@hmh.eng.br> wrote:
> On Mon, 09 Nov 2009, Matthew Garrett wrote:
>> When you hit the mixer button, does the ACPI interrupt count increase?
>> If so, we're certainly in a position to do something with it.
>
> It is supposed to go over the KDC.  But I would be _really_ interested on
> any events related to volume key presses on the X200s.  I expect it is
> issuing one of the ACPI volume notification events with OSI(Linux), and none
> without.

On brief inspection, with OSI(Linux), mute generates no ACPI
interrupts.  Without OSI(Linux), I see an ACPI interrupt only when the
mixer (the one in /proc/acpi/ibm/volume) changes as a result of
muting.  So pushing mute once generates an interrupt, but then there
are no more interrupts until I push volume up.

>
>> But it does nothing to solve the underlying issue, and then we get to
>> apply the same patch again for the next Thinkpad product cycle. If we
>> fix it properly, we don't need to bother.
>
> If I get a full description of behaviour, etc.  I can simply ask Lenovo
> about it.
>
> AFAIK, the native volume control through windows is being done in a
> different way, nowadays.  I'd need to know, without OSI(Linux), where the
> events are going.  That'd be enough to know when to read EC 0x30 and check
> if it is now mute/unmute.

What's the point of the IBM mixer (/proc/acpi/ibm/volume)?  We have a
perfectly good mixer in the form of ALSA, which already knows how to
mute and unmute the sound and even comes with nice user interfaces
these days.

With OSI(Linux) off, pressing mute turns off the sound with no UI
response whatsoever, fiddling with the ALSA mixer (or PulseAudio, for
that matter) has no effect, and the only ways to get sound back are to
play with /proc/acpi/ibm/volume or to press a different hardware
volume key.  But pressing that different hardware volume key *also*
changes the ALSA mixer because it generates a keypress.  In my book,
that means that the mute button messes up the sound hardware.

Do we really want every thinkpad to need a special UI to indicate the
state of the extra hardware muting function?

--Andy
--
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

  parent reply	other threads:[~2009-11-10 13:32 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-11-09 16:47 [PATCH] Set osi=Linux for the ThinkPad X200s Andrew Lutomirski
2009-11-09 16:54 ` Matthew Garrett
2009-11-09 17:49   ` Andrew Lutomirski
2009-11-09 17:54     ` Matthew Garrett
2009-11-10  9:24       ` [ltp] " Henrique de Moraes Holschuh
2009-11-10 12:42         ` Matthew Garrett
2009-11-10 13:32         ` Andrew Lutomirski [this message]
2009-11-10 14:14           ` Matthew Garrett
2009-11-10 17:49           ` Henrique de Moraes Holschuh
2009-11-10  9:19 ` [ltp] " Henrique de Moraes Holschuh
2009-11-25  4:50 ` Len Brown
2009-11-25 17:30   ` Jerone Young
2009-11-25 18:24     ` Len Brown
2009-11-25 19:56       ` Jerone Young
2009-11-26  5:07       ` Matthew Garrett
2009-11-26  6:00         ` Andrew Lutomirski
2009-11-26 17:02           ` Stefan Monnier
2009-11-26 13:43         ` Jerone Young
2009-11-25 19:01   ` Andrew Lutomirski
2009-11-30  5:47   ` Jerone Young

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=cb0375e10911100532l484e69aeyd78e422c23d730ce@mail.gmail.com \
    --to=luto@mit.edu \
    --cc=hmh@hmh.eng.br \
    --cc=linux-acpi@vger.kernel.org \
    --cc=linux-thinkpad@linux-thinkpad.org \
    --cc=mjg59@srcf.ucam.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox