[PATCH 1/4] arm64: dts: imx8mn: Include 32kHz oscillator clock in SPL DTs

[Adam Ford]

On Thu, Nov 21, 2024 at 7:59 PM Marek Vasut <marex at denx.de> wrote:
>
> Since 50cdd3f74af3 ("clk: imx: clk-imx8mn Fix nand and spi clock parent"),
> drivers/clk/imx/clk-imx8mn.c clk_get_by_name(dev, "osc_24m", &osc_24m_clk)
> fails with error -22 EINVAL in SPL. This is because clk_get_by_name() in the
> end calls fdtdec_parse_phandle_with_args(), which iterates over all phandles
> in clock-controller at 30380000 { clocks = <&osc_32k>, <&osc_24m>, ... } node
> 'clocks' property in an attempt to find the "osc_24m" clock, but fails to
> resolve the &osc_32k phandle and returns with -EINVAL.

Thanks for doing this.

>
> Include the osc_32k clock in SPL DTs as a low risk fix for v2025.01 release.
> This way, fdtdec_parse_phandle_with_args() can resolve both the osc_32k and
> following osc_24m phandle and successfully look up the osc_24m clock.
>
> Fixes: 50cdd3f74af3 ("clk: imx: clk-imx8mn Fix nand and spi clock parent")
> Signed-off-by: Marek Vasut <marex at denx.de>

Tested-by: Adam Ford <aford173 at gmail.com> #imx8mn-beacon
> ---
> Cc: "NXP i.MX U-Boot Team" <uboot-imx at nxp.com>
> Cc: Bryan Brattlof <bb at ti.com>
> Cc: Fabio Estevam <festevam at gmail.com>
> Cc: Greg Malysa <greg.malysa at timesys.com>
> Cc: Ian Roberts <ian.roberts at timesys.com>
> Cc: Ilias Apalodimas <ilias.apalodimas at linaro.org>
> Cc: Jonathan Humphreys <j-humphreys at ti.com>
> Cc: Masahisa Kojima <kojima.masahisa at socionext.com>
> Cc: Nathan Barrett-Morrison <nathan.morrison at timesys.com>
> Cc: Rasmus Villemoes <rasmus.villemoes at prevas.dk>
> Cc: Simon Glass <sjg at chromium.org>
> Cc: Stefano Babic <sbabic at denx.de>
> Cc: Sumit Garg <sumit.garg at linaro.org>
> Cc: Tim Harvey <tharvey at gateworks.com>
> Cc: Tom Rini <trini at konsulko.com>
> Cc: u-boot at lists.denx.de
> ---
> NOTE: For v2025.01
> ---
>  arch/arm/dts/imx8mn-u-boot.dtsi | 5 +++++
>  1 file changed, 5 insertions(+)
>
> diff --git a/arch/arm/dts/imx8mn-u-boot.dtsi b/arch/arm/dts/imx8mn-u-boot.dtsi
> index 6875c6d44ff..6d80d856365 100644
> --- a/arch/arm/dts/imx8mn-u-boot.dtsi
> +++ b/arch/arm/dts/imx8mn-u-boot.dtsi
> @@ -68,6 +68,11 @@
>         bootph-all;
>  };
>
> +&osc_32k {
> +       bootph-pre-ram;
> +       bootph-all;
> +};
> +
>  #ifdef CONFIG_FSL_CAAM
>  &sec_jr0 {
>         bootph-pre-ram;
> --
> 2.45.2
>