2008-03-07 10:02:45

by Atsushi Nemoto

[permalink] [raw]
Subject: [PATCH] at73c213: Monaural support

This patch add support for monaural playback to at73c213 driver. The
sound will be apear on L-channel. Tested on AT91SAM9260-EK.

Signed-off-by: Atsushi Nemoto <[email protected]>
---
diff --git a/sound/spi/at73c213.c b/sound/spi/at73c213.c
index 89d6e9c..a7839cc 100644
--- a/sound/spi/at73c213.c
+++ b/sound/spi/at73c213.c
@@ -118,7 +118,7 @@ static struct snd_pcm_hardware snd_at73c213_playback_hw = {
.rates = SNDRV_PCM_RATE_CONTINUOUS,
.rate_min = 8000, /* Replaced by chip->bitrate later. */
.rate_max = 50000, /* Replaced by chip->bitrate later. */
- .channels_min = 2,
+ .channels_min = 1,
.channels_max = 2,
.buffer_bytes_max = 64 * 1024 - 1,
.period_bytes_min = 512,
@@ -228,6 +228,14 @@ static int snd_at73c213_pcm_close(struct snd_pcm_substream *substream)
static int snd_at73c213_pcm_hw_params(struct snd_pcm_substream *substream,
struct snd_pcm_hw_params *hw_params)
{
+ struct snd_at73c213 *chip = snd_pcm_substream_chip(substream);
+ int channels = params_channels(hw_params);
+ int val;
+
+ val = ssc_readl(chip->ssc->regs, TFMR);
+ val = SSC_BFINS(TFMR_DATNB, channels - 1, val);
+ ssc_writel(chip->ssc->regs, TFMR, val);
+
return snd_pcm_lib_malloc_pages(substream,
params_buffer_bytes(hw_params));
}
@@ -249,10 +257,12 @@ static int snd_at73c213_pcm_prepare(struct snd_pcm_substream *substream)

ssc_writel(chip->ssc->regs, PDC_TPR,
(long)runtime->dma_addr);
- ssc_writel(chip->ssc->regs, PDC_TCR, runtime->period_size * 2);
+ ssc_writel(chip->ssc->regs, PDC_TCR,
+ runtime->period_size * runtime->channels);
ssc_writel(chip->ssc->regs, PDC_TNPR,
(long)runtime->dma_addr + block_size);
- ssc_writel(chip->ssc->regs, PDC_TNCR, runtime->period_size * 2);
+ ssc_writel(chip->ssc->regs, PDC_TNCR,
+ runtime->period_size * runtime->channels);

return 0;
}
@@ -375,7 +385,8 @@ static irqreturn_t snd_at73c213_interrupt(int irq, void *dev_id)

ssc_writel(chip->ssc->regs, PDC_TNPR,
(long)runtime->dma_addr + offset);
- ssc_writel(chip->ssc->regs, PDC_TNCR, runtime->period_size * 2);
+ ssc_writel(chip->ssc->regs, PDC_TNCR,
+ runtime->period_size * runtime->channels);
retval = IRQ_HANDLED;
}


2008-03-07 12:14:39

by Hans-Christian Egtvedt

[permalink] [raw]
Subject: Re: [PATCH] at73c213: Monaural support

Atsushi Nemoto wrote:
> This patch add support for monaural playback to at73c213 driver. The
> sound will be apear on L-channel. Tested on AT91SAM9260-EK.
>

This works fine for me on AT32AP7000 on STK1000 as well.

Although, I had to use 32 * 1024 buffer_bytes_max and 16 * 1024
period_bytes_max. Higher buffer_bytes_max resulted in glitching sound
when only using one channel. Nothing to do with this patch.

> Signed-off-by: Atsushi Nemoto <[email protected]>

Acked-by: Hans-Christian Egtvedt <[email protected]>

> ---
> diff --git a/sound/spi/at73c213.c b/sound/spi/at73c213.c
> index 89d6e9c..a7839cc 100644
> --- a/sound/spi/at73c213.c
> +++ b/sound/spi/at73c213.c
> @@ -118,7 +118,7 @@ static struct snd_pcm_hardware snd_at73c213_playback_hw = {
> .rates = SNDRV_PCM_RATE_CONTINUOUS,
> .rate_min = 8000, /* Replaced by chip->bitrate later. */
> .rate_max = 50000, /* Replaced by chip->bitrate later. */
> - .channels_min = 2,
> + .channels_min = 1,
> .channels_max = 2,
> .buffer_bytes_max = 64 * 1024 - 1,
> .period_bytes_min = 512,
> @@ -228,6 +228,14 @@ static int snd_at73c213_pcm_close(struct snd_pcm_substream *substream)
> static int snd_at73c213_pcm_hw_params(struct snd_pcm_substream *substream,
> struct snd_pcm_hw_params *hw_params)
> {
> + struct snd_at73c213 *chip = snd_pcm_substream_chip(substream);
> + int channels = params_channels(hw_params);
> + int val;
> +
> + val = ssc_readl(chip->ssc->regs, TFMR);
> + val = SSC_BFINS(TFMR_DATNB, channels - 1, val);
> + ssc_writel(chip->ssc->regs, TFMR, val);
> +
> return snd_pcm_lib_malloc_pages(substream,
> params_buffer_bytes(hw_params));
> }
> @@ -249,10 +257,12 @@ static int snd_at73c213_pcm_prepare(struct snd_pcm_substream *substream)
>
> ssc_writel(chip->ssc->regs, PDC_TPR,
> (long)runtime->dma_addr);
> - ssc_writel(chip->ssc->regs, PDC_TCR, runtime->period_size * 2);
> + ssc_writel(chip->ssc->regs, PDC_TCR,
> + runtime->period_size * runtime->channels);
> ssc_writel(chip->ssc->regs, PDC_TNPR,
> (long)runtime->dma_addr + block_size);
> - ssc_writel(chip->ssc->regs, PDC_TNCR, runtime->period_size * 2);
> + ssc_writel(chip->ssc->regs, PDC_TNCR,
> + runtime->period_size * runtime->channels);
>
> return 0;
> }
> @@ -375,7 +385,8 @@ static irqreturn_t snd_at73c213_interrupt(int irq, void *dev_id)
>
> ssc_writel(chip->ssc->regs, PDC_TNPR,
> (long)runtime->dma_addr + offset);
> - ssc_writel(chip->ssc->regs, PDC_TNCR, runtime->period_size * 2);
> + ssc_writel(chip->ssc->regs, PDC_TNCR,
> + runtime->period_size * runtime->channels);
> retval = IRQ_HANDLED;
> }
>

2008-03-07 15:25:37

by Atsushi Nemoto

[permalink] [raw]
Subject: Re: [PATCH] at73c213: Monaural support

On Fri, 07 Mar 2008 13:13:57 +0100, Hans-Christian Egtvedt <[email protected]> wrote:
> This works fine for me on AT32AP7000 on STK1000 as well.

Thanks, good news.

> Although, I had to use 32 * 1024 buffer_bytes_max and 16 * 1024
> period_bytes_max. Higher buffer_bytes_max resulted in glitching sound
> when only using one channel. Nothing to do with this patch.

Hmm, it sounds strange. But I still do not understand sound buffer
management well...

---
Atsushi Nemoto

2008-03-07 15:28:54

by Hans-Christian Egtvedt

[permalink] [raw]
Subject: Re: [PATCH] at73c213: Monaural support

Atsushi Nemoto wrote:
> On Fri, 07 Mar 2008 13:13:57 +0100, Hans-Christian Egtvedt <[email protected]> wrote:

>> Although, I had to use 32 * 1024 buffer_bytes_max and 16 * 1024
>> period_bytes_max. Higher buffer_bytes_max resulted in glitching sound
>> when only using one channel. Nothing to do with this patch.
>
> Hmm, it sounds strange. But I still do not understand sound buffer
> management well...
>

Me neither, alsa is a bit big and tons of layer hiding where stuff is
actually done. Still unsure what to do about the buffer and period size,
but I think setting them to 32k and 16k will work for most people.

--
HC, which just got a new email address