At Tue, 12 Jan 2010 10:07:02 +0100, I wrote:
At Fri, 8 Jan 2010 09:09:54 +0100 (CET), Jaroslav Kysela wrote:
On Sun, 3 Jan 2010, knprox@mi.rr.com wrote:
From: Ken Prox kprox@users.sourceforge.net
Signed-off-by: Ken Prox kprox@users.sourceforge.net
I've applied your patch to my "devel" git tree with space/tab replacements. Please, use same coding style matching other ALSA code and run "scripts/checkpatch.pl" on your patch before posting. Thanks.
I pulled your branches now to sound git tree.
And, now I'm checking the code and your change again while I'm working on fixing / clean-up of cxt5051 code. I don't understand why your patch works. Does it really work as is?
For example, the pin 0x17 has no multiple connections. So, changing the amp for the input #1 doesn't make sense at all. If it supports only the pin 0x1d, then the init verbs have to be different.
Doesn't another model, e.g. model=hp-dv6736, work as is?
thanks,
Takashi