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

2024-07-22 Thread Fabio Estevam
On Thu, Jul 18, 2024 at 1:55 PM Fabio Estevam wrote: > > Currently, there is an error when the i.MX93 CPU frequency is > read: > > Could not read CPU frequency: -2 > CPU: NXP i.MX93(52) Rev1.1 A55 at 0 MHz > > Fix it by describing the A55 clock nodes in the devicetree, like done > on other i.MX

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

2024-07-19 Thread Fabio Estevam
Hi Sébastien, On Fri, Jul 19, 2024 at 12:47 PM Sébastien Szymanski 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[

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

2024-07-19 Thread Sébastien Szymanski
On 7/19/24 16:13, Fabio Estevam wrote: Hi Sébastien, On Fri, Jul 19, 2024 at 11:00 AM Sébastien Szymanski 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.

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

2024-07-19 Thread Fabio Estevam
Hi Sébastien, On Fri, Jul 19, 2024 at 11:00 AM Sébastien Szymanski 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_

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

2024-07-19 Thread Sébastien Szymanski
Hello Fabio, On 7/18/24 18:54, Fabio Estevam wrote: Currently, there is an error when the i.MX93 CPU frequency is read: Could not read CPU frequency: -2 CPU: NXP i.MX93(52) Rev1.1 A55 at 0 MHz Fix it by describing the A55 clock nodes in the devicetree, like done on other i.MX SoCs. With thi

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

2024-07-19 Thread Peng Fan
> Subject: [PATCH] imx93-u-boot: Describe the CPU clocks in the > devicetree > > Currently, there is an error when the i.MX93 CPU frequency is > read: > > Could not read CPU frequency: -2 > CPU: NXP i.MX93(52) Rev1.1 A55 at 0 MHz > > Fix it by describing the A55

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

2024-07-18 Thread Fabio Estevam
Currently, there is an error when the i.MX93 CPU frequency is read: Could not read CPU frequency: -2 CPU: NXP i.MX93(52) Rev1.1 A55 at 0 MHz Fix it by describing the A55 clock nodes in the devicetree, like done on other i.MX SoCs. With this change, the CPU frequency error is gone and it can be