[alsa-devel] [PATCH 3/5] [RFC] ALSA ASOC Adds DSP DAI format support for platform driver
Arun KS
arunks at mistralsolutions.com
Tue Sep 30 08:41:45 CEST 2008
On Mon, Sep 29, 2008 at 4:45 PM, Jarkko Nikula <jarkko.nikula at nokia.com> wrote:
> On Mon, 29 Sep 2008 15:20:12 +0530
> "ext Arun KS" <arunks at mistralsolutions.com> wrote:
>
>> Enables DSP DAI format for mcbsp in omap platform driver
>>
>> Signed-off-by: Arun KS <arunks at mistralsolutions.com>
>> ---
>> sound/soc/omap/omap-mcbsp.c | 6 ++++++
>> 1 files changed, 6 insertions(+), 0 deletions(-)
>>
>> diff --git a/sound/soc/omap/omap-mcbsp.c b/sound/soc/omap/omap-mcbsp.c
>> index 3a4cc4b..1c024aa 100644
>> --- a/sound/soc/omap/omap-mcbsp.c
>> +++ b/sound/soc/omap/omap-mcbsp.c
>> @@ -292,6 +292,12 @@ static int omap_mcbsp_dai_set_dai_fmt(struct
>> snd_soc_dai *cpu_dai,
>> regs->rcr2 |= RDATDLY(1);
>> regs->xcr2 |= XDATDLY(1);
>> break;
>> + case SND_SOC_DAIFMT_DSP_A:
>> + /* 0-bit data delay */
>> + regs->rcr2 |= RDATDLY(0);
>> + regs->xcr2 |= XDATDLY(0);
>> + break;
>> +
>> default:
>
> I think this is not enough. The word clock length equals to one BCLK
> cycle in DSP mode. Not n channel bit BCLK cycles like in I2S.
>
In DSP Mode, Frame sync is followed by two data words.
I tested this patch and its working. Am i missing something ?
>
> Jarkko
> --
> To unsubscribe from this list: send the line "unsubscribe alsa-devel" in
> the body of a message to majordomo at vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
--
To unsubscribe from this list: send the line "unsubscribe alsa-devel" in
the body of a message to majordomo at vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
More information about the Alsa-devel
mailing list