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

Fabio Estevam festevam at gmail.com
Fri Jul 19 18:06:51 CEST 2024


Hi Sébastien,

On Fri, Jul 19, 2024 at 12:47 PM Sébastien Szymanski
<sebastien.szymanski at armadeus.com> wrote:

> I did look at imx8mm.dtsi but the clock IMX8MM_CLK_ARM is not defined in
> U-Boot and in Linux it's defined as:
>
> hws[IMX8MM_CLK_ARM] = imx_clk_hw_cpu("arm", "arm_a53_core",
>                                       hws[IMX8MM_CLK_A53_CORE]->clk,
>                                      hws[IMX8MM_CLK_A53_CORE]->clk,
>                                       hws[IMX8MM_ARM_PLL_OUT]->clk,
>                                      hws[IMX8MM_CLK_A53_DIV]->clk);
>
> So in Linux for the i.MX93, the clocks property in the device tree has
> to be IMX93_CLK_A55_CORE:
>
> clks[IMX93_CLK_A55_CORE] = imx_clk_hw_cpu("a55_core", "a55_sel",
>                                         clks[IMX93_CLK_A55_SEL]->clk,
>                                         clks[IMX93_CLK_A55_SEL]->clk,
>                                         clks[IMX93_CLK_ARM_PLL]->clk,
>                                         clks[IMX93_CLK_A55_GATE]->clk);
>
> ?

Yes, that's correct. I will use IMX93_CLK_A55_CORE when upstreaming it.

At some point, we should align these clock names between Linux and U-Boot.

For now, as I am touching arch/arm/dts/imx93-u-boot.dtsi this is OK
for U-Boot purposes.

Thanks


More information about the U-Boot mailing list