On Thu, May 16, 2019 at 3:35 PM Fabio Estevam <festevam@xxxxxxxxx> wrote: > > On Wed, May 15, 2019 at 11:42 AM Daniel Baluta <daniel.baluta@xxxxxxx> wrote: > > > + simple-audio-card,codec { > > + sound-dai = <&wm8524>; > > + clocks = <&clk IMX8MM_CLK_SAI3_ROOT>; > > IMX8MM_CLK_SAI3_ROOT is the internal clock that drives the SAI3 > interface, not an external clock that feeds the codec. > > It seems you should remove this 'clocks' entry. Just checked the schematics and the SAI3_MCLK pin clocks the codec, so the representation is correct: Reviewed-by: Fabio Estevam <festevam@xxxxxxxxx>