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

[Mark Brown]

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.