Re: How to put 16bit none audio PCM data stream into a 32bit none audio S/P-DIF transport stream?

[Takashi Iwai <tiwai@suse.de>]

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

At Wed, 9 Apr 2003 13:43:12 +0200,
Gregoire Favre wrote:
> 
> On Wed, Apr 09, 2003 at 01:09:12PM +0200, Takashi Iwai wrote:
> 
> > i can hear ac3 5.1 output over spdif with a terratec xfire 1024
> > without problems.  i guess it's not the choice of receiver but the
> > card itself.  there are differences between the cards even with the
> > same cs46xx chip depending on the hardware implementation.
> 
> Well, I can't speak about ALSA, but for DVB card, it only works with
> receivers that detect the encapsulated stream (a lots of DVB users have
> tested different receivers).
> So I ask if it's really not a problem with some receivers here too...
> When I plug my 5.1 to my standalone DVD player, I have two choice:
> 
>        	* SPDIF/RAW
>        	* SPDIF/PCM
> 
> And if I choose SPDIF/PCM it don't work, but SPDIF/RAW works perfectly.

hmm, then your receiver needs explicitly the non-audio bit.
i guess that my receiver reads the signature bytes for each spdif
frame.  i tested 44.1kHZ ac3 from diatons, and it works on mine.


> > yes it should be a driver bug.
> > but note that without the real (problematic) hardware, it's difficult
> > to check (especially if the system of developers do really work :) ...
> 
> So I only have the choice to change either the card, or my receier...

before giving up:  could you try the attached patch (to cvs version)?


Takashi

[-- Attachment #2: cs46xx-spdif-bits.dif --]
[-- Type: application/octet-stream, Size: 986 bytes --]

Index: alsa-kernel/pci/cs46xx/cs46xx_lib.c
===================================================================
RCS file: /suse/tiwai/cvs/alsa/alsa-kernel/pci/cs46xx/cs46xx_lib.c,v
retrieving revision 1.53
diff -u -r1.53 cs46xx_lib.c
--- alsa-kernel/pci/cs46xx/cs46xx_lib.c	2 Apr 2003 14:33:48 -0000	1.53
+++ alsa-kernel/pci/cs46xx/cs46xx_lib.c	9 Apr 2003 13:08:02 -0000
@@ -2162,12 +2162,11 @@
 	int change;
 
 	down (&chip->spos_mutex);
-	val = _wrap_all_bits(((u32)ucontrol->value.iec958.status[0] << 24)) |
-	      _wrap_all_bits(((u32)ucontrol->value.iec958.status[1] << 16)) |
-	      _wrap_all_bits( (u32)ucontrol->value.iec958.status[3]) |
+	val = ((u32)_wrap_all_bits(ucontrol->value.iec958.status[0]) << 24) |
+		((u32)_wrap_all_bits(ucontrol->value.iec958.status[1]) << 16) |
+		(u32)_wrap_all_bits(ucontrol->value.iec958.status[3]) |
 		/* left and right validity bit */
 		(1 << 13) | (1 << 12);
-
 
 	change = ins->spdif_csuv_stream != val;
 	ins->spdif_csuv_stream = val;