public inbox for linux-acpi@vger.kernel.org
 help / color / mirror / Atom feed
From: Henrique de Moraes Holschuh <hmh@hmh.eng.br>
To: Ian Molton <ian.molton@collabora.co.uk>
Cc: Takashi Iwai <tiwai@suse.de>, Len Brown <lenb@kernel.org>,
	linux-acpi@vger.kernel.org, ibm-acpi-devel@lists.sourceforge.net,
	Ingo Molnar <mingo@elte.hu>, Amerigo Wang <amwang@redhat.com>,
	Helight Xu <helight.xu@gmail.com>
Subject: Re: [PATCH 3/5] thinkpad-acpi: make volume subdriver optional
Date: Mon, 28 Dec 2009 11:17:45 -0200	[thread overview]
Message-ID: <20091228131744.GC19362@khazad-dum.debian.net> (raw)
In-Reply-To: <4B37E1AF.6030105@collabora.co.uk>

On Sun, 27 Dec 2009, Ian Molton wrote:
> Not directly related to your patch but I thought I'd mention...
> 
> I've noticed issues with my X200 re: routing audio before / after docking.

ThinkPads used to route line-out to the dock, with a fixed 1.1dB amplifier
(0.1dB increase) in the audio path, and had the volume control for
headphones and speakers done in a different audio path, extracted before the
line-out amplifier.  The PeeCee beeper channel was routed to the headphones
and speakers, only.

If Lenovo tried to reproduce that with the HDA mixer but without the "Laptop
Apudio Controller" the IBM thinkpads had, I wouldn't be surprised if it
causes massive ALSA disruption should the firmware screw with the HDA mixer
registers behind ALSA's back.  Try asking the kernel to enable OSI(Linux) in
ACPI...  I always forget the kernel command line parameter to do that, but
it is documented in the usual place.

-- 
  "One disk to rule them all, One disk to find them. One disk to bring
  them all and in the darkness grind them. In the Land of Redmond
  where the shadows lie." -- The Silicon Valley Tarot
  Henrique Holschuh

  reply	other threads:[~2009-12-28 13:17 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-12-27  0:52 [GIT PATCH] thinkpad-acpi fixes for 2.6.32-rc2 Henrique de Moraes Holschuh
2009-12-27  0:52 ` [PATCH 1/5] thinkpad-acpi: don't take the first ALSA slot by default Henrique de Moraes Holschuh
2009-12-27  0:52 ` [PATCH 2/5] thinkpad-acpi: don't fail to load the entire module due to ALSA problems Henrique de Moraes Holschuh
2009-12-27  0:52 ` [PATCH 3/5] thinkpad-acpi: make volume subdriver optional Henrique de Moraes Holschuh
2009-12-27  9:38   ` Takashi Iwai
     [not found]     ` <s5hr5qgvjsa.wl%tiwai-l3A5Bk7waGM@public.gmane.org>
2009-12-27 18:32       ` Henrique de Moraes Holschuh
2009-12-27 22:37         ` Ian Molton
2009-12-28 13:17           ` Henrique de Moraes Holschuh [this message]
     [not found]             ` <20091228131744.GC19362-ZGHd14iZgfaRjzvQDGKj+xxZW9W5cXbT@public.gmane.org>
2009-12-28 21:30               ` Jerone Young
2009-12-27  0:52 ` [PATCH 4/5] thinkpad-acpi: update volume documentation Henrique de Moraes Holschuh
2009-12-27  0:52 ` [PATCH 5/5] thinkpad-acpi: improve Kconfig help text Henrique de Moraes Holschuh
2009-12-28 13:30   ` tytso
2009-12-28 13:35     ` Henrique de Moraes Holschuh
2009-12-29 14:21       ` tytso
2009-12-27  3:41 ` [GIT PATCH] thinkpad-acpi fixes for 2.6.32-rc2 Len Brown

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=20091228131744.GC19362@khazad-dum.debian.net \
    --to=hmh@hmh.eng.br \
    --cc=amwang@redhat.com \
    --cc=helight.xu@gmail.com \
    --cc=ian.molton@collabora.co.uk \
    --cc=ibm-acpi-devel@lists.sourceforge.net \
    --cc=lenb@kernel.org \
    --cc=linux-acpi@vger.kernel.org \
    --cc=mingo@elte.hu \
    --cc=tiwai@suse.de \
    /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