Hi Alvin > --- a/sound/soc/generic/audio-graph-card2.c > +++ b/sound/soc/generic/audio-graph-card2.c (snip) > /* > * convert bit_frame > * We need to flip clock_provider if it was CPU node, > * because it is Codec base. > */ > daiclk = snd_soc_daifmt_clock_provider_from_bitmap(bit_frame); > - if (is_cpu_node) > + if (is_cpu_node && !dai_link->symmetric_clock_roles) > daiclk = snd_soc_daifmt_clock_provider_flipped(daiclk); > > dai_link->dai_fmt = daifmt | daiclk; Hmm ? I'm confusing [2/4] patch handling fliping, and [3/4] also handling fliping. Nothing changed ? Current SND_SOC_DAIFMT_xx_xx are very confusable, framework and driver are different (flipped). and also, audio-graph-card2 is using intuitive DT settings (need flip for CPU). Thank you for your help !! Best regards --- Kuninori Morimoto