[alsa-devel] Testers wanted: New SB X-Fi driver

[Takashi Iwai]

At Thu, 14 May 2009 23:07:59 +1000,
Michael (Micksa) Slade wrote:
> 
> Takashi Iwai wrote:
> 
>     At Thu, 14 May 2009 22:34:09 +1000,
>     Michael (Micksa) Slade wrote:
> 
>         Swtiched audacious to 24bit.  It very quietly plays a badly distorted version
>         of the song :)
> 
>     What about aplay with 32bit samples?
> 
> Seems to work fine.  Also 24bit.  Guess it's audacious' fault.  Probably.

It's possible that audacious tries to access 24bit, not 32bit format, 
and ctxfi driver has a bug around that.

Could you try the patch below?


Takashi

===
diff --git a/sound/pci/ctxfi/ctpcm.c b/sound/pci/ctxfi/ctpcm.c
index a64cb0e..aae9eb1 100644
--- a/sound/pci/ctxfi/ctpcm.c
+++ b/sound/pci/ctxfi/ctpcm.c
@@ -30,7 +30,6 @@ static struct snd_pcm_hardware ct_pcm_playback_hw = {
 				   SNDRV_PCM_FMTBIT_S16_LE |
 				   SNDRV_PCM_FMTBIT_U16_LE |
 				   SNDRV_PCM_FMTBIT_S24_3LE |
-				   SNDRV_PCM_FMTBIT_S24_LE |
 				   SNDRV_PCM_FMTBIT_S32_LE),
 	.rates			= (SNDRV_PCM_RATE_CONTINUOUS |
 				   SNDRV_PCM_RATE_8000_192000),
@@ -81,7 +80,6 @@ static struct snd_pcm_hardware ct_pcm_capture_hw = {
 				   SNDRV_PCM_FMTBIT_S16_LE |
 				   SNDRV_PCM_FMTBIT_U16_LE |
 				   SNDRV_PCM_FMTBIT_S24_3LE |
-				   SNDRV_PCM_FMTBIT_S24_LE |
 				   SNDRV_PCM_FMTBIT_S32_LE),
 	.rates			= (SNDRV_PCM_RATE_CONTINUOUS |
 				   SNDRV_PCM_RATE_8000_96000),