On Tue, 3 Dec 2002, James Stafford wrote: > Attached are two patches that add basic Envy24HT support to the ICE1712 > driver and VT1616 to the AC97 codec driver. They were made against the cvs > from today. > > Issues: > Quote from VT1616 data sheet: "The VT1616 powers up with pins 39 and 41 > assigned to the Front channel DACs as described in the 2.2 specification. > When this bit is to “0”, the output pins get assigned to the Rear stereo > DAC pair with an independent volume control." > The reference design uses pins 39 and 41 for the left and right rear > outputs. I have added a control to set this bit to 0 but the default is 1. > Should a patch function be used to set the default to 0?
I think that the default should be left in the AC97 code and write the appropriate configuration after snd_ac97_mixer() call when you detect your hardware and VT1616 codec. > When using applications such as MPlayer and ac3dec, the centre/lfe and rear > outputs are swapped so that the data for the rear comes out of centre/lfe > and vice versa. Is there something I have missed here? We send data in this order: channel0,1 = front; channel2,3 = rear; channel4,5 = center/lfe ; if your hardware has swapped some stereo paths, you need to swap channels in software - see alsa-lib/src/conf/cards/ICH.conf - ICH.pcm.surround51.0 {} section I've applied your patch and fixed one missing update in ice1712.h. Also, the driver id is changed in CVS. Jaroslav ----- Jaroslav Kysela <[EMAIL PROTECTED]> Linux Kernel Sound Maintainer ALSA Project, SuSE Labs ------------------------------------------------------- This SF.net email is sponsored by: Microsoft Visual Studio.NET comprehensive development tool, built to increase your productivity. Try a free online hosted session at: http://ads.sourceforge.net/cgi-bin/redirect.pl?micr0003en _______________________________________________ Alsa-devel mailing list [EMAIL PROTECTED] https://lists.sourceforge.net/lists/listinfo/alsa-devel