From: David Henningsson <david.henningsson@canonical.com>
To: Takashi Iwai <tiwai@suse.de>
Cc: ALSA Development Mailing List <alsa-devel@alsa-project.org>
Subject: Codecs without auto-parser (was: ALSA: HDA: Fix microphone(s) on Lenovo Edge 13)
Date: Tue, 01 Feb 2011 08:37:25 +0100 [thread overview]
Message-ID: <4D47B835.7090502@canonical.com> (raw)
In-Reply-To: <s5h1v3xw7o7.wl%tiwai@suse.de>
On 2011-01-28 13:10, Takashi Iwai wrote:
> At Fri, 28 Jan 2011 12:51:30 +0100,
> David Henningsson wrote:
>>
>> On 2011-01-28 09:01, Takashi Iwai wrote:
>>> And I hope that we should go further a bit for now -- more clean up of
>>> the cxt5066 code either checking BIOS pins or hp/mic/spk pre-defined
>>> pins. Currently, the code is fairly messy (partly because of olpc
>>> support), and now is a good chance to improve it a bit more.
>>
>> Maybe so. I'm not sure I can commit to doing that work right now, as I
>> have other work commitments waiting for me to take care of them.
>
> OK.
Hmm. I got a few things cleared up yesterday, and so I might have some
time to actually do this. However, it also seems like I have to fix an
AD1984A machine, and that driver is in equally bad shape (as in: no
auto-parser, just a list of models). I could add another model, but
suppose I'd do this the right way, how would I do it?
I guess there are three options to start off with:
1) ad1988 seems to have an auto-parser. There are a lot of hardcoded
nids in there, but I guess I could copy-and-paste some into a new
auto-parser for ad1984a.
2) It seems to me like the most competent candidate for auto-parser
currently is the cx_auto stuff - compared to other auto-parsers the
hardcoded nids are fewer. Perhaps this one could be extended to also do
auto-parsing for codecs from other vendors? Or at least some convenience
functions moved to hda_codec.c in order to remove duplication between
vendors?
3) There is also the generic parser. It seems to be the one most capable
of handling randomly complex graphs, but is lacking automute/jack-detect
support, and maybe other functions as well (?).
--
David Henningsson, Canonical Ltd.
http://launchpad.net/~diwic
next prev parent reply other threads:[~2011-02-01 7:37 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-01-27 9:37 [PATCH] ALSA: HDA: Fix microphone(s) on Lenovo Edge 13 David Henningsson
2011-01-28 8:01 ` Takashi Iwai
2011-01-28 11:51 ` David Henningsson
2011-01-28 12:10 ` Takashi Iwai
2011-01-28 12:40 ` David Henningsson
2011-01-31 11:08 ` Takashi Iwai
2011-02-01 7:37 ` David Henningsson [this message]
2011-02-01 21:34 ` Codecs without auto-parser (was: ALSA: HDA: Fix microphone(s) on Lenovo Edge 13) Takashi Iwai
2011-02-02 15:16 ` Codecs without auto-parser David Henningsson
2011-02-03 13:16 ` Takashi Iwai
2011-02-04 9:37 ` David Henningsson
2011-01-28 15:59 ` [PATCH] ALSA: HDA: Fix microphone(s) on Lenovo Edge 13 Andy Robinson
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=4D47B835.7090502@canonical.com \
--to=david.henningsson@canonical.com \
--cc=alsa-devel@alsa-project.org \
--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 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.