On Sun, Oct 11, 2009 at 10:18 AM, Takashi Iwai tiwai@suse.de wrote:
At Sat, 10 Oct 2009 19:07:09 +0800, Logan Li wrote:
Following patch series mainly includes: - support VT1718S, VT2002P, VT1812S, VT1716S, ... - support smart 5.1 - add jack detection for VT1708 - power saving functions
Thanks for the patches. They are all well formatted now.
I applied your patches except for 17, ALSA: HDA VIA: Add 2nd S/PDIF out for VT1708S and VT1702 as Robert reported a regression.
This is likely because of the behavior change by this patch. Now the second SPDIF has to be specified manually with the explicit substream number. This itself is fine, but I see the problem with pulseaudio, for example. There is no good way to specify this substream automatically for PA with some symbols like "spdif" or "hdmi".
This is basically a problem of the current ALSA core and HD-audio core implementation. So, we should solve all together.
Actually, the latest posted version didn't have the mixer problem I had before (I haven't looked in detail at what was different). But I agree using a different substream for a different output isn't ideal since there's no way for software to detect that this is the case. Can it be a separate PCM like hw:0,2 or something?
Hopefully VIA will also look into the SPDIF no-output problem I have with VT1828S..
On most machines, it's not too big deal to have it the secondary digital as a slave. If this really matters, we can add another hint to make the individual digital substreams conditionally.
Anyway, please check the latest sound GIT tree (or alsa-driver snapshot) whether it works. I only tested the compilation, so far.
thanks,
Takashi