At Wed, 12 Nov 2008 11:55:47 +0000, Mark Brown wrote:
From: Christian Pellegrin chripell@gmail.com
fixes playing/recording of 8 bit audio files.
Generated on 20081108 against v2.6.27
Signed-off-by: Christian Pellegrin chripell@fsfe.org Signed-off-by: Mark Brown broonie@opensource.wolfsonmicro.com
sound/soc/s3c24xx/s3c24xx-i2s.c | 7 +++++++ 1 files changed, 7 insertions(+), 0 deletions(-)
diff --git a/sound/soc/s3c24xx/s3c24xx-i2s.c b/sound/soc/s3c24xx/s3c24xx-i2s.c index ba4476b..c18977b 100644 --- a/sound/soc/s3c24xx/s3c24xx-i2s.c +++ b/sound/soc/s3c24xx/s3c24xx-i2s.c @@ -261,10 +261,17 @@ static int s3c24xx_i2s_hw_params(struct snd_pcm_substream *substream,
switch (params_format(params)) { case SNDRV_PCM_FORMAT_S8:
iismod &= ~S3C2410_IISMOD_16BIT;
((struct s3c24xx_pcm_dma_params *)
break; case SNDRV_PCM_FORMAT_S16_LE: iismod |= S3C2410_IISMOD_16BIT;rtd->dai->cpu_dai->dma_data)->dma_size = 1;
((struct s3c24xx_pcm_dma_params *)
break;rtd->dai->cpu_dai->dma_data)->dma_size = 2;
I still feel a bit uncomfortable with this as it changes the global variables. Maybe better to assign the corresponding fields in the local struct.
Takashi