[PATCH] ARM: dts: imx6qdl-apalis: Fix sgtl5000 detection issue

[Tim Harvey]

On Mon, Mar 28, 2022 at 12:13 PM Fabio Estevam <festevam at gmail.com> wrote:
>
> Hi Tim,
>
> On Mon, Mar 28, 2022 at 4:02 PM Tim Harvey <tharvey at gateworks.com> wrote:
>
> > Fabio,
> >
> > Intesting... was there a specific kernel version or set of conditions
> > such as what's configured as modules that made this issue show up? It
> > looks like there are a lot of IMX6 boards that define the MCLK in the
> > audmuxgrp.... some use simple-audio-card and some do not in case that
> > matters.
>
> I observed this issue running a custom defconfig.
>
> To be on the safe side, we should better not rely on kernel probe
> order/defconfig and let the sgtl5000 node
> to configure the MCLK pintctrl early, so that the MCLK becomes
> available and the codec registers can be read safely read.
>
> Regards,
>
> Fabio Estevam

Fabio,

Agreed - the MCLK is needed for the codec not for audmux.

Reviewed-by: Tim Harvey <tharvey at gateworks.com>

Best Regards,

Tim