Re: [PATCH] imx93-u-boot: Describe the CPU clocks in the devicetree

[Fabio Estevam]

Hi Sébastien,

On Fri, Jul 19, 2024 at 11:00 AM Sébastien Szymanski
<sebastien.szyman...@armadeus.com> wrote:

> > +&A55_0 {
> > +     clocks = <&clk IMX93_CLK_ARM_PLL>;
>
> Why IMX93_CLK_ARM_PLL and not IMX93_CLK_A55_SEL ?

IMX93_CLK_A55_SEL is a mux that can select between a55_alt versus arm_pll.

The real CPU clock is IMX93_CLK_ARM_PLL, so this one better represents it.

This is also consistent with the other i.MX8M SoCs.

Take imx8mm.dtsi for example:

A53_0: cpu@0 {
....
     clocks = <&clk IMX8MM_CLK_ARM>;

This also uses the "real" CPU clock, not the mux one (imx8mm_a53_sels).

I also plan to upstream this clock change to the Linux devicetree.

Thanks