From mboxrd@z Thu Jan 1 00:00:00 1970 From: Lars-Peter Clausen Subject: Re: [PATCH] ASoC: Added PCM short and long frame sync formats Date: Tue, 27 Mar 2012 09:33:01 +0200 Message-ID: <4F716D2D.50502@metafoo.de> References: <20120326133305.GR3098@opensource.wolfsonmicro.com> <1332819135-5416-1-git-send-email-omair.m.abdullah@linux.intel.com> <4F71353A.9020704@linux.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from smtp-out-057.synserver.de (smtp-out-058.synserver.de [212.40.185.58]) by alsa0.perex.cz (Postfix) with ESMTP id 406E2104191 for ; Tue, 27 Mar 2012 09:41:32 +0200 (CEST) In-Reply-To: <4F71353A.9020704@linux.intel.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: alsa-devel-bounces@alsa-project.org Errors-To: alsa-devel-bounces@alsa-project.org To: Omair Mohammed Abdullah Cc: alsa-devel@alsa-project.org, broonie@opensource.wolfsonmicro.com, lrg@ti.com List-Id: alsa-devel@alsa-project.org On 03/27/2012 05:34 AM, Omair Mohammed Abdullah wrote: > On Tuesday 27 March 2012 09:02 AM, Omair Mohammed Abdullah wrote: >> The PCM short frame sync sends a short (1 clock length) sync signal before the >> MSB of data. The PCM long frame sync sends a 1 word length sync signal along >> with the data. >> >> Signed-off-by: Omair Mohammed Abdullah >> --- >> include/sound/soc-dai.h | 2 ++ >> 1 files changed, 2 insertions(+), 0 deletions(-) >> >> diff --git a/include/sound/soc-dai.h b/include/sound/soc-dai.h >> index c429f24..dc94b25 100644 >> --- a/include/sound/soc-dai.h >> +++ b/include/sound/soc-dai.h >> @@ -32,6 +32,8 @@ struct snd_soc_dapm_widget; >> #define SND_SOC_DAIFMT_DSP_B 5 /* L data MSB during FRM LRC */ >> #define SND_SOC_DAIFMT_AC97 6 /* AC97 */ >> #define SND_SOC_DAIFMT_PDM 7 /* Pulse density modulation */ >> +#define SND_SOC_DAIFMT_SHORT_SYNC 8 /* FRM for 1 clock before MSB */ >> +#define SND_SOC_DAIFMT_LONG_SYNC 9 /* FRM for 1 word starting with MSB */ >> >> /* left and right justified also known as MSB and LSB respectively */ >> #define SND_SOC_DAIFMT_MSB SND_SOC_DAIFMT_LEFT_J > > What I am looking for is something like this. But > SND_SOC_DAIFMT_SHORT_SYNC might be the same as SND_SOC_DAIFMT_DSP_A. Yes it is. And if you really only have two slots your long sync is SND_SOC_DAIFMT_LEFT_J, but with the bit clock inverted.