Hi Jerome Thank you for your feedback > > @@ -118,24 +118,25 @@ unsigned int meson_card_parse_daifmt(struct device_node *node, > > struct device_node *bitclkmaster = NULL; > > struct device_node *framemaster = NULL; > > unsigned int daifmt; > > + unsigned int daiclk; > > Why did you need to add this local ? If you don't like using daiclk here, I can merge these into daifmt. I will wait other feedback, and will post patch next week. Thank you for your help !! Best regards --- Kuninori Morimoto