[alsa-devel] [PATCH 2/3] ASoC: OMAP: Fix setup of XCCR and RCCR registers in McBSP DAI

Eduardo Valentin eduardo.valentin at nokia.com
Tue Aug 25 08:26:06 CEST 2009


On Sun, Aug 23, 2009 at 11:24:26AM +0200, Jarkko Nikula wrote:
> Commit ca6e2ce08679c094878d7f39a0349a7db1d13675 is setting up few XCCR and
> RCCR bits for I2S and DPS_A formats. Part of the bits are already set
> for all formats and I believe that XDISABLE and RDISABLE bits are
> format independent.
> 
> As XCCR and RCCR are found only from OMAP2430 and OMAP34xx, I move setup
> of XDISABLE and RDISABLE to where those cpu's are tested and remove format
> dependent part for simplicity.
> 
> Signed-off-by: Jarkko Nikula <jhnikula at gmail.com>
> Cc: Eero Nurkkala <ext-eero.nurkkala at nokia.com>
> Cc: Peter Ujfalusi <peter.ujfalusi at nokia.com>

Acked-by: Eduardo Valentin <eduardo.valentin at nokia.com>

> ---
>  sound/soc/omap/omap-mcbsp.c |    8 ++------
>  1 files changed, 2 insertions(+), 6 deletions(-)
> 
> diff --git a/sound/soc/omap/omap-mcbsp.c b/sound/soc/omap/omap-mcbsp.c
> index f5387d9..89e8bce 100644
> --- a/sound/soc/omap/omap-mcbsp.c
> +++ b/sound/soc/omap/omap-mcbsp.c
> @@ -379,8 +379,8 @@ static int omap_mcbsp_dai_set_dai_fmt(struct snd_soc_dai *cpu_dai,
>  		regs->xcr2	|= XFIG;
>  	}
>  	if (cpu_is_omap2430() || cpu_is_omap34xx()) {
> -		regs->xccr = DXENDLY(1) | XDMAEN;
> -		regs->rccr = RFULL_CYCLE | RDMAEN;
> +		regs->xccr = DXENDLY(1) | XDMAEN | XDISABLE;
> +		regs->rccr = RFULL_CYCLE | RDMAEN | RDISABLE;
>  	}
>  
>  	switch (fmt & SND_SOC_DAIFMT_FORMAT_MASK) {
> @@ -388,15 +388,11 @@ static int omap_mcbsp_dai_set_dai_fmt(struct snd_soc_dai *cpu_dai,
>  		/* 1-bit data delay */
>  		regs->rcr2	|= RDATDLY(1);
>  		regs->xcr2	|= XDATDLY(1);
> -		regs->rccr	|= RFULL_CYCLE | RDMAEN | RDISABLE;
> -		regs->xccr	|= (DXENDLY(1) | XDMAEN | XDISABLE);
>  		break;
>  	case SND_SOC_DAIFMT_DSP_A:
>  		/* 1-bit data delay */
>  		regs->rcr2      |= RDATDLY(1);
>  		regs->xcr2      |= XDATDLY(1);
> -		regs->rccr	|= RFULL_CYCLE | RDMAEN | RDISABLE;
> -		regs->xccr	|= (DXENDLY(1) | XDMAEN | XDISABLE);
>  		/* Invert FS polarity configuration */
>  		temp_fmt ^= SND_SOC_DAIFMT_NB_IF;
>  		break;
> -- 
> 1.6.3.3
> 
> --
> To unsubscribe from this list: send the line "unsubscribe linux-omap" in
> the body of a message to majordomo at vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html

-- 
Eduardo Valentin


More information about the Alsa-devel mailing list