[alsa-devel] [PATCH] ALSA: HDA: Realtek: Take vmaster dac from multiout dac list
david.henningsson at canonical.com
Mon Dec 12 11:31:58 CET 2011
2011-12-12 18:19, Takashi Iwai skrev:
> At Mon, 12 Dec 2011 18:13:44 +0800,
> David Henningsson wrote:
>> 2011-12-12 17:48, Takashi Iwai skrev:
>>> At Mon, 12 Dec 2011 13:37:41 +0800,
>>> David Henningsson wrote:
>>>> With the auto-parser we can choose the dac nid for vmaster from
>>>> the DACs we already know, instead of hard-coding it. This is more
>>>> future-proof and was actually wrong on one machine.
>>>> Reported-by: David Chen<david.chen at canonical.com>
>>>> Signed-off-by: David Henningsson<david.henningsson at canonical.com>
>>> Unfortunately this doesn't work correctly for some codecs.
>>> Codecs like ALC262 have no volume in DAC (e.g. 0x02) but only in mixer
>>> (e.g. 0x0c).
>>> In future, we should build up the I/O paths like in patch_via.c,
>>> so that such info can be checked more easily.
>> Ah, of course. Thanks for spotting. I believe the patch should look like
>> the attached one instead, what do you think?
> The check of cfg->line_outs can be skipped. The primary output must
> be present in any case. Or, at most, just add a non-zero check for
> cfg->line_out_pins. Otherwise it looks good.
Yeah, I was thinking that too, but then patch_alc662 had the no-hp
fixup, and with all fixups moving here and there, I decided to better be
safe than sorry. But I trust you to know better than me, so attaching
modified patch. :-)
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 0 bytes
Desc: not available
Url : http://mailman.alsa-project.org/pipermail/alsa-devel/attachments/20111212/622ae14c/attachment.patch
More information about the Alsa-devel