[pulseaudio-discuss] [PATCH] fix profile configurations for IEC/SPDIF outputs

[Colin Guthrie]

'Twas brillig, and Colin Guthrie at 19/06/10 08:57 did gyre and gimble:
> 'Twas brillig, and pl bossart at 19/06/10 02:28 did gyre and gimble:
>> Also removed profiles such iec-surround40 with do not make any sense
>> (SPDIF) is stereo.
> 
> NAK. These are for working with the alsa a52 plugin which can encode
> on-the-fly.
> 
> I recently posted a patch that fixes a PA assert() when this plugin was
> used.
> 
> I can now enable the 5.1 Digital output but the timing is all messed up
> and all the sound is compressed into a very short period of time. On the
> plus side it does come out the right speakers :D

Oh I see you completely removed the a52 plugin support... any reason?
I'd rather the support was just fixed rather than removed completely.

My patch should fix the assert, but not sure what the next stage of
fixing it would be.

Also your patch seems to use "hw:" names in the profiles. This is bad.
any reason why it was changed? If there is a problem using iec958:%f on
your machine, I'd expect it's an alsa bug. Using hw:%f,1 is almost
certainly broken.

Col

-- 

Colin Guthrie
gmane(at)colin.guthr.ie
http://colin.guthr.ie/

Day Job:
  Tribalogic Limited [http://www.tribalogic.net/]
Open Source:
  Mandriva Linux Contributor [http://www.mandriva.com/]
  PulseAudio Hacker [http://www.pulseaudio.org/]
  Trac Hacker [http://trac.edgewall.org/]