ALC655 in CVS: Are Center/LFE switched with Surround?

ALC655 in CVS: Are Center/LFE switched with Surround?

[Richard Chan]

I've finally got 5.1 out of my via82xx/ALC655 system using the 
Line-In/Mic jacks!
Thanks Takashi.

However I seem to have got Center/LFE switched with Surround.
I.e. if I play xine & mplayer (L, R, RearL, RearR, Center, LFE)
through surround51 I get Center/LFE switched with Rear speakers.

I must use the following to work in .asoundrc
pcm_slave.sltest {
    pcm "hw:0,1"
    channels 6
}

pcm.via51 {
    type plug
    ttable.0.0  1
    ttable.1.1  1
    ttable.2.4  1
    ttable.3.5  1
    ttable.4.2  1
    ttable.5.3  1
    slave sltest
}

Has anyone else observed this problem?



-------------------------------------------------------
This SF.net email is sponsored by: SF.net Giveback Program.
Does SourceForge.net help you be more productive?  Does it
help you create better code?  SHARE THE LOVE, and help us help
YOU!  Click Here: http://sourceforge.net/donate/

Re: ALC655 in CVS: Are Center/LFE switched with Surround?

[Takashi Iwai]

[-- Attachment #1: Type: text/plain, Size: 1088 bytes --]

At Wed, 19 Nov 2003 23:04:57 +0800,
Richard Chan wrote:
> 
> 
> I've finally got 5.1 out of my via82xx/ALC655 system using the 
> Line-In/Mic jacks!
> Thanks Takashi.

glad to hear that it works now.  what was the problem?

 
> However I seem to have got Center/LFE switched with Surround.
> I.e. if I play xine & mplayer (L, R, RearL, RearR, Center, LFE)
> through surround51 I get Center/LFE switched with Rear speakers.

ah, this reminds me an old story.
there is a mixer switch "Swap Surround Slot".  and, IIRC, starngely
enough, for 5.1 this should be turned on for 5.1 output while it's off
for 4.0.

> 
> I must use the following to work in .asoundrc
> pcm_slave.sltest {
>     pcm "hw:0,1"
>     channels 6
> }
> 
> pcm.via51 {
>     type plug
>     ttable.0.0  1
>     ttable.1.1  1
>     ttable.2.4  1
>     ttable.3.5  1
>     ttable.4.2  1
>     ttable.5.3  1
>     slave sltest
> }

this is also a solution, but the attached would be more "light" way.
copy the attached file to /usr/share/alsa/cards.
also, if you have chance, please test 4.0 output, too.


thanks,

Takashi

[-- Attachment #2: VIA8233.conf --]
[-- Type: application/octet-stream, Size: 1744 bytes --]

#
# Configuration for the VIA8233/VIA8233C/VIA8235 chip with 4/6 multi-channel support
#

<confdir:pcm/front.conf>

VIA8233.pcm.front.0 {
	@args [ CARD ]
	@args.CARD {
		type string
	}
	type hw
	card $CARD
	device 0
}	

<confdir:pcm/surround40.conf>

VIA8233.pcm.surround40.0 {
	@args [ CARD ]
	@args.CARD {
		type string
	}
	type hooks
	slave.pcm {
		type hw
		card $CARD
		device 1
	}
	hooks.0 {
		type ctl_elems
		hook_args [
			{
				name "Swap Surround Slot"
				device 0
				lock true
				preserve true
				value flase
				optional true
			}
		]
	}
}

<confdir:pcm/surround41.conf>
<confdir:pcm/surround50.conf>
<confdir:pcm/surround51.conf>

VIA8233.pcm.surround51.0 {
	@args [ CARD ]
	@args.CARD {
		type string
	}
	type hooks
	slave.pcm {
		type hw
		card $CARD
		device 1
	}
	hooks.0 {
		type ctl_elems
		hook_args [
			{
				name "Swap Surround Slot"
				device 0
				lock true
				preserve true
				value true
				optional true
			}
		]
	}
}

<confdir:pcm/iec958.conf>

VIA8233.pcm.iec958.0 {
	@args [ CARD AES0 AES1 AES2 AES3 ]
	@args.CARD {
		type string
	}
	@args.DEV {
		type integer
	}
	@args.AES0 {
		type integer
	}
	@args.AES1 {
		type integer
	}
	@args.AES2 {
		type integer
	}
	@args.AES3 {
		type integer
	}
	type hooks
	slave.pcm {
		type hw
		card $CARD
		device 0
		subdevice 3
	}
	hooks.0 {
		type ctl_elems
		hook_args [
			{
				name "IEC958 Playback AC97-SPSA"
				lock true
				preserve true
				value 3
			}
			{
				name "IEC958 Playback Default"
				lock true
				preserve true
				value [ $AES0 $AES1 $AES2 $AES3 ]
			}
			{
				name "IEC958 Playback Switch"
				lock true
				preserve true
				value true
			}
			{
				name "IEC958 Output Switch"
				lock true
				preserve true
				value true
			}
		]
	}
}

[-- Attachment #3: Type: text/plain, Size: 1 bytes --]

Re: ALC655 in CVS: Are Center/LFE switched with Surround?

[Richard Chan]

Takashi Iwai wrote:

>At Wed, 19 Nov 2003 23:04:57 +0800,
>Richard Chan wrote:
>  
>
>>I've finally got 5.1 out of my via82xx/ALC655 system using the 
>>Line-In/Mic jacks!
>>Thanks Takashi.
>>    
>>
>
>glad to hear that it works now.  what was the problem?
>
> 
>  
>
>  
>
I was using
 amixer  set 'Line-In As Surround' unmute
I guess I needed to use
 amixer set 'Line-In As Surround' on

Thanks - will try the new config files and report back.

Richard



-------------------------------------------------------
This SF.net email is sponsored by: SF.net Giveback Program.
Does SourceForge.net help you be more productive?  Does it
help you create better code?  SHARE THE LOVE, and help us help
YOU!  Click Here: http://sourceforge.net/donate/

Re: ALC655 in CVS: Are Center/LFE switched with Surround?

[Richard Chan]

Alas, the new VIA82XX.conf doesn't fix the problem.
(ALC655 does not have the "Slot Modify" feature of ALC650).
Also I realise I still need a plug with slave.channels 6

If I use surround51 directly I cannot get sound out of the other
four speakers. This plug gives me rear sound

pcm.testrear {
        type plug
####Aargh!! should be ttable.0.2 and ttable.0.3####
####Changing Swap Surround Slot doesn't help###
####ALC655 does not have Slot Modify control bits in register 6A###
        ttable.0.4  1
        ttable.1.5  1
        slave.pcm surround51
        slave.channels 6
}

(I notice surround50.conf has the slave channels set; can something similar
be put inside surround51.conf?)

xine will also not product surround sound with the surround51 device -
I need to use a plug with channels set to 6.

Funny though mplayer works with the surround51 device (though
channels are swapped).


Takashi Iwai wrote:

>At Wed, 19 Nov 2003 23:04:57 +0800,
>Richard Chan wrote:
>  
>
>>I've finally got 5.1 out of my via82xx/ALC655 system using the 
>>Line-In/Mic jacks!
>>Thanks Takashi.
>>    
>>
>
>glad to hear that it works now.  what was the problem?
>
> 
>  
>
>>However I seem to have got Center/LFE switched with Surround.
>>I.e. if I play xine & mplayer (L, R, RearL, RearR, Center, LFE)
>>through surround51 I get Center/LFE switched with Rear speakers.
>>    
>>
>
>ah, this reminds me an old story.
>there is a mixer switch "Swap Surround Slot".  and, IIRC, starngely
>enough, for 5.1 this should be turned on for 5.1 output while it's off
>for 4.0.
>
>  
>
>>I must use the following to work in .asoundrc
>>pcm_slave.sltest {
>>    pcm "hw:0,1"
>>    channels 6
>>}
>>
>>pcm.via51 {
>>    type plug
>>    ttable.0.0  1
>>    ttable.1.1  1
>>    ttable.2.4  1
>>    ttable.3.5  1
>>    ttable.4.2  1
>>    ttable.5.3  1
>>    slave sltest
>>}
>>    
>>
>
>this is also a solution, but the attached would be more "light" way.
>copy the attached file to /usr/share/alsa/cards.
>also, if you have chance, please test 4.0 output, too.
>
>
>thanks,
>
>Takashi
>  
>
>------------------------------------------------------------------------
>
>
>  
>




-------------------------------------------------------
This SF.net email is sponsored by: SF.net Giveback Program.
Does SourceForge.net help you be more productive?  Does it
help you create better code?  SHARE THE LOVE, and help us help
YOU!  Click Here: http://sourceforge.net/donate/

Testers of via82xx 5.1 output (Re: ALC655 in CVS: Are Center/LFE switched with Surround?)

[Takashi Iwai]

[-- Attachment #1: Type: text/plain, Size: 1215 bytes --]

At Thu, 20 Nov 2003 00:35:10 +0800,
Richard Chan wrote:
> 
> Alas, the new VIA82XX.conf doesn't fix the problem.
> (ALC655 does not have the "Slot Modify" feature of ALC650).
> Also I realise I still need a plug with slave.channels 6
 
oh yeah, you're right, ALC655 is not completely compatible with
ALC655...

ok, the attached patch is a quick hack for better supporting ALC655
and ALC658.  it includes the swap of via82xx 5.1 output (again).

IIRC, the slot assignment was determined by reports of someone else,
and we confirmed that is the correct order.  but, apparently, it's not
true on your board.
possibly this depends on the codec or mobo..?  not sure, why the slots
are swapped on yours.

***

so, now the "CALL FOR TESTS".

does anyone have 5.1 output-capable boards with VIA823x chipset?
the VIA8233A is exception.  it needs a different way to handle.
if it's one of the others, VIA8233,A,C or 8235, please check whether
the jack assignment of 5.1 output is correct.

regardless whether the output is correct or not, please give the
feedback together with the info, which ac97 codec is on it.  see
/proc/asound/card0/ac97#0 for that.

oh, don't forget to update to 1.0.0pre1 version!


thanks,

Takashi

[-- Attachment #2: via-alc655-fix.dif --]
[-- Type: application/octet-stream, Size: 7338 bytes --]

Index: alsa-kernel/pci/via82xx.c
===================================================================
RCS file: /suse/tiwai/cvs/alsa/alsa-kernel/pci/via82xx.c,v
retrieving revision 1.76
diff -u -r1.76 via82xx.c
--- alsa-kernel/pci/via82xx.c	18 Nov 2003 15:23:27 -0000	1.76
+++ alsa-kernel/pci/via82xx.c	19 Nov 2003 17:34:39 -0000
@@ -972,8 +972,10 @@
 		case 2: slots = (1<<0) | (2<<4); break;
 		case 3: slots = (1<<0) | (2<<4) | (5<<8); break;
 		case 4: slots = (1<<0) | (2<<4) | (3<<8) | (4<<12); break;
-		case 5: slots = (1<<0) | (2<<4) | (3<<8) | (4<<12) | (5<<16); break;
-		case 6: slots = (1<<0) | (2<<4) | (3<<8) | (4<<12) | (5<<16) | (6<<20); break;
+		// case 5: slots = (1<<0) | (2<<4) | (3<<8) | (4<<12) | (5<<16); break;
+		case 5: slots = (1<<0) | (2<<4) | (5<<8) | (6<<12) | (3<<16); break;
+		// case 6: slots = (1<<0) | (2<<4) | (3<<8) | (4<<12) | (5<<16) | (6<<20); break;
+		case 6: slots = (1<<0) | (2<<4) | (5<<8) | (6<<12) | (3<<16) | (4<<20); break;
 		default: slots = 0; break;
 		}
 	}
Index: alsa-kernel/pci/ac97/ac97_codec.c
===================================================================
RCS file: /suse/tiwai/cvs/alsa/alsa-kernel/pci/ac97/ac97_codec.c,v
retrieving revision 1.101
diff -u -r1.101 ac97_codec.c
--- alsa-kernel/pci/ac97/ac97_codec.c	18 Nov 2003 11:48:28 -0000	1.101
+++ alsa-kernel/pci/ac97/ac97_codec.c	19 Nov 2003 17:35:25 -0000
@@ -112,7 +112,8 @@
 { 0x414c4721, 0xfffffff0, "ALC650D",		patch_alc650,	NULL },
 { 0x414c4722, 0xfffffff0, "ALC650E",		patch_alc650,	NULL },
 { 0x414c4723, 0xfffffff0, "ALC650F",		patch_alc650,	NULL },
-{ 0x414c4760, 0xfffffff0, "ALC655",		patch_alc650,	NULL },
+{ 0x414c4760, 0xfffffff0, "ALC655",		patch_alc655,	NULL },
+{ 0x414c4780, 0xfffffff0, "ALC658",		patch_alc655,	NULL },
 { 0x414c4730, 0xffffffff, "ALC101",		NULL,		NULL },
 { 0x414c4740, 0xfffffff0, "ALC202",		NULL,		NULL },
 { 0x414c4750, 0xfffffff0, "ALC250",		NULL,		NULL },
@@ -1985,9 +1986,9 @@
 	snd_ac97_write(ac97, AC97_GENERAL_PURPOSE, 0);
 
 	snd_ac97_write(ac97, AC97_POWERDOWN, ac97->regs[AC97_POWERDOWN]);
-	snd_ac97_write(ac97, AC97_MASTER, 0x8000);
+	snd_ac97_write(ac97, AC97_MASTER, 0x8101);
 	for (i = 0; i < 10; i++) {
-		if (snd_ac97_read(ac97, AC97_MASTER) == 0x8000)
+		if (snd_ac97_read(ac97, AC97_MASTER) == 0x8101)
 			break;
 		mdelay(1);
 	}
@@ -1996,7 +1997,7 @@
 	if (ac97->bus->init)
 		ac97->bus->init(ac97);
 
-	is_ad18xx = (ac97->id & 0xffffff40) == AC97_ID_AD1881;
+	is_ad18xx = (ac97->flags & AC97_AD_MULTI);
 	if (is_ad18xx) {
 		/* restore the AD18xx codec configurations */
 		for (codec = 0; codec < 3; codec++) {
Index: alsa-kernel/pci/ac97/ac97_patch.c
===================================================================
RCS file: /suse/tiwai/cvs/alsa/alsa-kernel/pci/ac97/ac97_patch.c,v
retrieving revision 1.31
diff -u -r1.31 ac97_patch.c
--- alsa-kernel/pci/ac97/ac97_patch.c	3 Nov 2003 18:03:53 -0000	1.31
+++ alsa-kernel/pci/ac97/ac97_patch.c	19 Nov 2003 17:37:37 -0000
@@ -983,6 +983,8 @@
 	/* 7: Independent Master Volume Left */
 	/* 8: reserved */
 	AC97_SINGLE("Line-In As Surround", AC97_ALC650_MULTICH, 9, 1, 0),
+	/* 10: mic, see below */
+	/* 11-13: in IEC958 controls */
 	AC97_SINGLE("Swap Surround Slot", AC97_ALC650_MULTICH, 14, 1, 0),
 #if 0 /* always set in patch_alc650 */
 	AC97_SINGLE("IEC958 Input Clock Enable", AC97_ALC650_CLOCK, 0, 1, 0),
@@ -1109,6 +1111,98 @@
 	return 0;
 }
 
+static const snd_kcontrol_new_t snd_ac97_controls_alc655[] = {
+	AC97_SINGLE("Duplicate Front", AC97_ALC650_MULTICH, 0, 1, 0),
+	AC97_SINGLE("Line-In As Surround", AC97_ALC650_MULTICH, 9, 1, 0),
+	AC97_SINGLE("Mic As Center/LFE", AC97_ALC650_MULTICH, 10, 1, 0),
+};
+
+static int alc655_iec958_route_info(snd_kcontrol_t *kcontrol, snd_ctl_elem_info_t *uinfo)
+{
+	static char *texts_655[3] = { "PCM", "Analog In", "IEC958 In" };
+	static char *texts_658[4] = { "PCM", "Analog1 In", "Analog2 In", "IEC958 In" };
+	ac97_t *ac97 = snd_kcontrol_chip(kcontrol);
+
+	uinfo->type = SNDRV_CTL_ELEM_TYPE_ENUMERATED;
+	uinfo->count = 1;
+	uinfo->value.enumerated.items = ac97->spec.dev_flags ? 4 : 3;
+	if (uinfo->value.enumerated.item >= uinfo->value.enumerated.items)
+		uinfo->value.enumerated.item = uinfo->value.enumerated.items - 1;
+	strcpy(uinfo->value.enumerated.name,
+	       ac97->spec.dev_flags ?
+	       texts_658[uinfo->value.enumerated.item] :
+	       texts_655[uinfo->value.enumerated.item]);
+	return 0;
+
+}
+
+static int alc655_iec958_route_get(snd_kcontrol_t *kcontrol, snd_ctl_elem_value_t *ucontrol)
+{
+	ac97_t *ac97 = snd_kcontrol_chip(kcontrol);
+	unsigned short val;
+
+	val = ac97->regs[AC97_ALC650_MULTICH];
+	val = (val >> 12) & 3;
+	if (ac97->spec.dev_flags && val == 3)
+		val = 0;
+	ucontrol->value.enumerated.item[0] = val;
+	return 0;
+}
+
+static int alc655_iec958_route_put(snd_kcontrol_t *kcontrol, snd_ctl_elem_value_t *ucontrol)
+{
+	ac97_t *ac97 = snd_kcontrol_chip(kcontrol);
+	return snd_ac97_update_bits(ac97, AC97_ALC650_MULTICH, 3 << 12,
+				    (unsigned short)ucontrol->value.enumerated.item[0]);
+}
+
+static const snd_kcontrol_new_t snd_ac97_spdif_controls_alc655[] = {
+        AC97_SINGLE("IEC958 Capture Switch", AC97_ALC650_MULTICH, 11, 1, 0),
+        AC97_SINGLE("IEC958 Input Monitor", AC97_ALC650_MULTICH, 14, 1, 0),
+	{
+		.iface  = SNDRV_CTL_ELEM_IFACE_MIXER,
+		.name   = "IEC958 Playback Route",
+		.info   = alc655_iec958_route_info,
+		.get    = alc655_iec958_route_get,
+		.put    = alc655_iec958_route_put,
+	},
+};
+
+static int patch_alc655_specific(ac97_t * ac97)
+{
+	int err;
+
+	if ((err = patch_build_controls(ac97, snd_ac97_controls_alc655, ARRAY_SIZE(snd_ac97_controls_alc655))) < 0)
+		return err;
+	if (ac97->ext_id & AC97_EI_SPDIF) {
+		if ((err = patch_build_controls(ac97, snd_ac97_spdif_controls_alc655, ARRAY_SIZE(snd_ac97_spdif_controls_alc655))) < 0)
+			return err;
+	}
+	return 0;
+}
+
+static struct snd_ac97_build_ops patch_alc655_ops = {
+	.build_specific	= patch_alc655_specific
+};
+
+int patch_alc655(ac97_t * ac97)
+{
+	ac97->spec.dev_flags = (ac97->id == 0x414c4780); /* ALC658 */
+
+	ac97->build_ops = &patch_alc655_ops;
+
+	/* enable spdif in */
+	snd_ac97_write_cache(ac97, AC97_ALC650_CLOCK,
+			     snd_ac97_read(ac97, AC97_ALC650_MULTICH) | 0x8000);
+	snd_ac97_write_cache(ac97, AC97_ALC650_CLOCK,
+			     snd_ac97_read(ac97, AC97_ALC650_CLOCK) | 0x02);
+
+	/* full DAC volume */
+	snd_ac97_write_cache(ac97, AC97_ALC650_SURR_DAC_VOL, 0x0808);
+	snd_ac97_write_cache(ac97, AC97_ALC650_LFE_DAC_VOL, 0x0808);
+	return 0;
+}
+
 static const snd_kcontrol_new_t snd_ac97_cm9738_controls[] = {
 	AC97_SINGLE("Line-In As Surround", AC97_CM9738_VENDOR_CTRL, 10, 1, 0),
 	AC97_SINGLE("Duplicate Front", AC97_CM9738_VENDOR_CTRL, 13, 1, 0),
Index: alsa-kernel/pci/ac97/ac97_patch.h
===================================================================
RCS file: /suse/tiwai/cvs/alsa/alsa-kernel/pci/ac97/ac97_patch.h,v
retrieving revision 1.11
diff -u -r1.11 ac97_patch.h
--- alsa-kernel/pci/ac97/ac97_patch.h	3 Nov 2003 18:03:53 -0000	1.11
+++ alsa-kernel/pci/ac97/ac97_patch.h	19 Nov 2003 17:34:39 -0000
@@ -46,6 +46,7 @@
 int patch_ad1981b(ac97_t * ac97);
 int patch_ad1985(ac97_t * ac97);
 int patch_alc650(ac97_t * ac97);
+int patch_alc655(ac97_t * ac97);
 int patch_cm9738(ac97_t * ac97);
 int patch_cm9739(ac97_t * ac97);
 int patch_vt1616(ac97_t * ac97);

Re: Testers of via82xx 5.1 output - report - swapped output still there

[Richard Chan]

via82xx with ALC655/8 patch and 1.0.0pre1

Summary -
Swapped output 2,3==Center/LFE and 4,5==Surround
has not been fixed by the patch.

/proc/asound> cat version
Advanced Linux Sound Architecture Driver Version 1.0.0pre1.
Compiled on Nov 20 2003 for kernel 2.6.0-test9.2R.

/proc/asound> cat card0/codec97#0/ac97#0-0
0-0/0: Realtek ALC655 rev 0
 
Capabilities     :
DAC resolution   : 16-bit
ADC resolution   : 16-bit
3D enhancement   : No 3D Stereo Enhancement
 
Current setup
Mic gain         : +0dB [+0dB]
POP path         : pre 3D
Sim. stereo      : off
3D enhancement   : off
Loudness         : off
Mono output      : MIX
Mic select       : Mic1
ADC/DAC loopback : off
Extended ID      : codec=0 rev=2 LDAC SDAC CDAC DSA=0 SPDIF
Extended status  : SPCV LDAC SDAC CDAC SPDIF=res SPDIF
SPDIF Control    : Consumer PCM Category=0x2 Generation=1 Rate=48kHz

/proc/asound> cat card0/codec97#0/ac97#0-0+regs
0:00 = 0000
0:02 = 0606
0:04 = 0000
0:06 = 801f
0:08 = 0000
0:0a = 801e
0:0c = 801f
0:0e = 801f
0:10 = 9f1f
0:12 = 0000
0:14 = 0000
0:16 = 9f1f
0:18 = 0000
0:1a = 0000
0:1c = 0000
0:1e = 0000
0:20 = 0400
0:22 = 0000
0:24 = 0000
0:26 = 000f
0:28 = 09c4
0:2a = 05f4
0:2c = bb80
0:2e = bb80
0:30 = bb80
0:32 = bb80
0:34 = 0000
0:36 = 0000
0:38 = 0000
0:3a = 2824
0:3c = 0000
0:3e = 0000
0:40 = 0000
0:42 = 0000
0:44 = 0000
0:46 = 0000
0:48 = 0000
0:4a = 0000
0:4c = 0000
0:4e = 0000
0:50 = 0000
0:52 = 0000
0:54 = 0000
0:56 = 0000
0:58 = 0000
0:5a = 0000
0:5c = 0000
0:5e = 0000
0:60 = 0000
0:62 = 0000
0:64 = 0808
0:66 = 0808
0:68 = 0aea
0:6a = 0600
0:6c = 0000
0:6e = 0004
0:70 = 81e0
0:72 = 22e8
0:74 = 8000
0:76 = 0000
0:78 = 0007
0:7a = 0042
0:7c = 414c
0:7e = 4760





-------------------------------------------------------
This SF.net email is sponsored by: SF.net Giveback Program.
Does SourceForge.net help you be more productive?  Does it
help you create better code?  SHARE THE LOVE, and help us help
YOU!  Click Here: http://sourceforge.net/donate/

Re: Re: Testers of via82xx 5.1 output - report - swapped output still there

[Takashi Iwai]

At Thu, 20 Nov 2003 18:29:17 +0800,
Richard Chan wrote:
> 
> via82xx with ALC655/8 patch and 1.0.0pre1
> 
> Summary -
> Swapped output 2,3==Center/LFE and 4,5==Surround
> has not been fixed by the patch.

hmm...i'm puzzled.
there must be no longer "Swap Surround Slot" control, anyway.
can you confirm this?

also, what happens if you revert the via82xx code (modification of
slots bit flags in snd_via8233_multi_prepare) ?
to be sure: is you chip 8233 or 8235, and NOT 8233A, right?  check
/proc/asound/cards.


Takashi


-------------------------------------------------------
This SF.net email is sponsored by: SF.net Giveback Program.
Does SourceForge.net help you be more productive?  Does it
help you create better code?  SHARE THE LOVE, and help us help
YOU!  Click Here: http://sourceforge.net/donate/

Re: Re: Testers of via82xx 5.1 output - report - swapped output still there

[Takashi Iwai]

At Thu, 20 Nov 2003 22:53:16 +0800,
Richard Chan wrote:
> 
> Takashi Iwai wrote:
> 
> >At Thu, 20 Nov 2003 18:29:17 +0800,
> >Richard Chan wrote:
> >  
> >
> >>via82xx with ALC655/8 patch and 1.0.0pre1
> >>
> >>Summary -
> >>Swapped output 2,3==Center/LFE and 4,5==Surround
> >>has not been fixed by the patch.
> >>    
> >>
> >
> >hmm...i'm puzzled.
> >there must be no longer "Swap Surround Slot" control, anyway.
> >can you confirm this?
> >
> >  
> >
> Yes - no more 'Swap Surround Slot'.
> Reverting the slot assignment makes no difference at all:
> I can confirm that the 4 channel assignment is correct - if a use a plug
> with slave.channels 4 I get rear sound.
> 
> If I use a plug with slave.channels 6 the rear is always channel 4 5.
> These lines don't make any difference to 6 channel.
> // case 6: slots = (1<<0) | (2<<4) | (3<<8) | (4<<12) | (5<<16) | 
> (6<<20); break;
> case 6: slots = (1<<0) | (2<<4) | (5<<8) | (6<<12) | (3<<16) | (4<<20); 
> break;

interesting, then your chip doesn't follow the spec written in the
datasheet.  (or, it can be a newer model.)
if this bit switches have no effect, it's difficult to fix on the
driver side.  perhaps the routing on alsa-lib is the only solution...


> /usr/src/linux-2.6.0-test9.2/sound/pci> cat /proc/asound/cards
> 0 [rev60 ]: VIA8233 - VIA 823x rev60
> VIA 823x rev60 at 0xe400, irq 22

i don't know of this revision.  really a new one..?


Takashi


-------------------------------------------------------
This SF.net email is sponsored by: SF.net Giveback Program.
Does SourceForge.net help you be more productive?  Does it
help you create better code?  SHARE THE LOVE, and help us help
YOU!  Click Here: http://sourceforge.net/donate/