From: Henrique de Moraes Holschuh <hmh@hmh.eng.br>
To: Takashi Iwai <tiwai@suse.de>
Cc: ibm-acpi-devel@lists.sourceforge.net, alsa-devel@lists.sourceforge.net
Subject: Re: [ibm-acpi-devel] Adding a alsa mixer interface to ibm-acpi
Date: Mon, 26 Feb 2007 14:41:45 -0300 [thread overview]
Message-ID: <20070226174145.GF2909@khazad-dum.debian.net> (raw)
In-Reply-To: <s5hr6scswnp.wl%tiwai@suse.de>
On Mon, 26 Feb 2007, Takashi Iwai wrote:
> > It is not directly related to the sound card *at* *all*. It is done by the
> > thinkpad embedded controller firmware.
> >
> > So, it really is a "different card", and doesn't belong in any of the many
> > sound card drivers that work on the various thinkpad models. Since it is
> > only one extra mixer control, if the API allows it, we *could* piggy back it
> > on the mixer for the real sound card, but that looks like extra complexity
> > and layering violations for no good reason to me.
>
> Hm, I think the problem is how to export the sound card object to the
> outside. Currently the created instance is closed in the ALSA tree,
> and there is no direct way to get the pointer from another driver.
> That's why I thought of including the code in the ALSA tree. But, we
> can add an export to enable the hook in each card driver code.
Well, adding a new card that only has a mixer and no PCM or other sound
properties seems to work just fine, and it is *much* simpler than piggying
back a new *system global* control on top of AC97, many HDA chips, etc. It
is also that much easier to deal with.
Can I keep it as a separate card?
> > If I have it as a separate snd-thinkpadmixer module or somesuch, that module
> > will need some code from the main ibm-acpi module to work. It needs the
> > ACPI bus, and also thinpad-specific knowledge... sounds messy to have it on
> > a different tree, thus the question about how complicated would it be to
> > have it in the ibm-acpi driver instead of submitting it for the alsa tree.
>
> I think it's no big problem to leave it in another tree since the
> control API is fairly stable right now.
Then I should have a proper version of the thinkpad volume control as an
alsa mixer ready soon, registering it as a separate card. I can change that
to a piggy-back mixer control later, if you'd rather I did that.
The ibm-acpi module does not follow the module parameter specs of alsa sound
modules, is that a problem? I don't expect alsa configurator tools to ever
deal with the ibm-acpi mixer, but...
--
"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
-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
next prev parent reply other threads:[~2007-02-26 17:41 UTC|newest]
Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-02-25 1:20 Adding a alsa mixer interface to ibm-acpi Henrique de Moraes Holschuh
2007-02-26 15:31 ` Takashi Iwai
[not found] ` <s5h649pue8z.wl%tiwai-l3A5Bk7waGM@public.gmane.org>
2007-02-26 16:28 ` [Alsa-devel] " Henrique de Moraes Holschuh
2007-02-26 16:36 ` [ibm-acpi-devel] " Takashi Iwai
2007-02-26 17:41 ` Henrique de Moraes Holschuh [this message]
[not found] ` <20070226174145.GF2909-ZGHd14iZgfaRjzvQDGKj+xxZW9W5cXbT@public.gmane.org>
2007-02-27 1:50 ` [Alsa-devel] " Theodore Tso
2007-02-27 2:14 ` [ibm-acpi-devel] " Henrique de Moraes Holschuh
2007-03-07 21:59 ` Takashi Iwai
[not found] ` <s5hd53kaf5r.wl%tiwai-l3A5Bk7waGM@public.gmane.org>
2007-03-07 23:02 ` [Alsa-devel] " Henrique de Moraes Holschuh
2007-03-07 23:12 ` Shem Multinymous
2007-03-07 23:23 ` [ibm-acpi-devel] " Tobin Davis
2007-03-07 23:41 ` Shem Multinymous
2007-03-07 23:42 ` Henrique de Moraes Holschuh
2007-03-07 23:55 ` Tobin Davis
[not found] ` <1173311709.4531.30.camel-lPoCD4h/KdUU2NE2KMwFWnnhMCiq3JZZ@public.gmane.org>
2007-03-08 0:38 ` [Alsa-devel] " Henrique de Moraes Holschuh
2007-03-09 16:29 ` [ibm-acpi-devel] " Takashi Iwai
[not found] ` <s5htzwugz22.wl%tiwai-l3A5Bk7waGM@public.gmane.org>
2007-03-10 3:50 ` [Alsa-devel] " Henrique de Moraes Holschuh
2007-03-12 12:03 ` [ibm-acpi-devel] " Takashi Iwai
2007-03-12 21:00 ` Henrique de Moraes Holschuh
2007-03-12 22:00 ` Alex Deucher
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=20070226174145.GF2909@khazad-dum.debian.net \
--to=hmh@hmh.eng.br \
--cc=alsa-devel@lists.sourceforge.net \
--cc=ibm-acpi-devel@lists.sourceforge.net \
--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