On Fri, May 31, 2019 at 03:28:33PM +0800, Shawn Guo wrote: > On Tue, May 28, 2019 at 10:10:43AM +0300, Daniel Baluta wrote: > > On Mon, May 20, 2019 at 10:33 PM Fabio Estevam <[email protected]> wrote: > > > > > > On Thu, May 16, 2019 at 3:35 PM Fabio Estevam <[email protected]> wrote: > > > > > > > > On Wed, May 15, 2019 at 11:42 AM Daniel Baluta <[email protected]> > > > > 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 <[email protected]> > > > > Shawn, > > > > Can you have a look? > > I cannot apply this one, because there are '=20' in the patch content.
Talk to NXP colleague Anson Huang <[email protected]> to find out how to fix it. https://patchwork.kernel.org/patch/10944169/#22656941 Shawn
