On 03/10/2014 04:25 AM, Li.Xiubo@freescale.com wrote:
--- a/sound/soc/generic/simple-card.c +++ b/sound/soc/generic/simple-card.c @@ -165,7 +165,7 @@ static int asoc_simple_card_parse_of(struct device_node *node, snd_soc_of_parse_card_name(&priv->snd_card, "simple-audio-card,name");
/* get CPU/CODEC common format via simple-audio-card,format */
- priv->daifmt = snd_soc_of_parse_daifmt(node, "simple-audio-card,") &
- priv->daifmt = snd_soc_of_parse_daifmt(node, "simple-audio-card,format")
@Peter
Sorry, the interface of snd_soc_of_parse_daifmt(struct device_node *np, const char *prefix), it's intended not only to parse the 'XXX,format', but also 'XXX,continuous-clock', 'XXX, bitclock-inversion', 'XXX, frame-inversion', 'XXX,bitclock-master' and 'XXX,frame-master' at the same time.
Yes, I overlooked that. I have not checked the snd_soc_of_parse_daifmt() before.
Maybe just the comment here is not very correct.
Probably it would be better to re-word it since the code at this point only interested on the i2s coding and the bit/frame inversion.
Thanks,
-- Best Regards, Xiubo
& (SND_SOC_DAIFMT_FORMAT_MASK | SND_SOC_DAIFMT_INV_MASK);
/* off-codec widgets */
1.9.0