Hi Nicolin > > static struct asoc_simple_card_info fsi_wm8978_info = { > > ... > > .daifmt = SND_SOC_DAIFMT_I2S | SND_SOC_DAIFMT_CBM_CFM, > > .cpu_dai = { > > ... > > .fmt = SND_SOC_DAIFMT_IB_NF, > > }, > > .codec_dai = { > > ... > > .fmt = SND_SOC_DAIFMT_NB_NF, > > }, > > }; > > This would be better imo. > > And ideally we should also keep the xB_xF identical like Mark said > _identical_. > Just some cpu dai drivers might do an incorrect settings for it, like > regarding > NB as sampling on rising edge and IF as active low (I'm saying this without a > careful check though), which results people need to re-set bitclock-invert and > frame-invert if they switch the DAI format from left_j to i2s for example.
Wow... I had misunderstood... I need to send fixup patch after lunch. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/