Re: [PATCH v2 2/2] alsa: s6105 IP camera machine specific ASoC code

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Sat, Mar 28, 2009 at 07:47:02PM +0100, Daniel Gl??ckner wrote:

> +	/* set codec DAI configuration */
> +	ret = snd_soc_dai_set_fmt(codec_dai, SND_SOC_DAIFMT_I2S |
> +					     SND_SOC_DAIFMT_CBM_CFM);
> +	if (ret < 0)
> +		return ret;

> +	/* set cpu DAI configuration */
> +	ret = snd_soc_dai_set_fmt(cpu_dai, SND_SOC_DAIFMT_CBM_CFM |
> +					   SND_SOC_DAIFMT_IB_IF);
> +	if (ret < 0)
> +		return ret;

Why do these two differ?  It's very unusual to see different formats and
generally indicates a bug in one of the drivers involved that's being
worked around.  The omission of I2S is understandable since the CPU only
does I2S (though ideally it'd be specified) but the inversion of the bit
and frame clocks for the CPU only looks like the CPU is generating
inverted clocks by default.
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel

[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux