Re: [PATCH] arm: dts: k3-am625-verdin-r5: Change CPU frequency to 800MHz

2024-03-22 Thread Tom Rini
On Tue, 19 Mar 2024 11:04:27 -0300, Joao Paulo Goncalves wrote:

> The lowest speed grade of Toradex AM62 SoMs is K speed, resulting in a
> max value of 800MHz for the CPU operating frequency. A solution with
> runtime selection of the CPU frequency is already planned to avoid these
> kinds of problems in the future.
> 
> 

Applied to u-boot/master, thanks!

-- 
Tom




Re: [PATCH] arm: dts: k3-am625-verdin-r5: Change CPU frequency to 800MHz

2024-03-19 Thread Tom Rini
On Tue, Mar 19, 2024 at 04:05:31PM +0100, Francesco Dolcini wrote:
> Hello Tom,
> 
> On Tue, Mar 19, 2024 at 11:04:27AM -0300, Joao Paulo Goncalves wrote:
> > From: Joao Paulo Goncalves 
> > 
> > The lowest speed grade of Toradex AM62 SoMs is K speed, resulting in a
> > max value of 800MHz for the CPU operating frequency. A solution with
> > runtime selection of the CPU frequency is already planned to avoid these
> > kinds of problems in the future.
> > 
> > Fixes: 8fb8a6d49977 ("arm: dts: k3-am625-verdin-r5:Change CPU
> > frequency to 1000MHz")
> > Signed-off-by: Joao Paulo Goncalves 
> 
> We have a patch ready to make the frequency adjusted at runtime that
> should be on the mailing list in a day or so, as suggested by Vignesh,
> targetting uboot-next, this one fix should go into master however.

OK, noted, thanks.

-- 
Tom


signature.asc
Description: PGP signature


Re: [PATCH] arm: dts: k3-am625-verdin-r5: Change CPU frequency to 800MHz

2024-03-19 Thread Francesco Dolcini
Hello Tom,

On Tue, Mar 19, 2024 at 11:04:27AM -0300, Joao Paulo Goncalves wrote:
> From: Joao Paulo Goncalves 
> 
> The lowest speed grade of Toradex AM62 SoMs is K speed, resulting in a
> max value of 800MHz for the CPU operating frequency. A solution with
> runtime selection of the CPU frequency is already planned to avoid these
> kinds of problems in the future.
> 
> Fixes: 8fb8a6d49977 ("arm: dts: k3-am625-verdin-r5:Change CPU
> frequency to 1000MHz")
> Signed-off-by: Joao Paulo Goncalves 

We have a patch ready to make the frequency adjusted at runtime that
should be on the mailing list in a day or so, as suggested by Vignesh,
targetting uboot-next, this one fix should go into master however.

Thanks,
Francesco




[PATCH] arm: dts: k3-am625-verdin-r5: Change CPU frequency to 800MHz

2024-03-19 Thread Joao Paulo Goncalves
From: Joao Paulo Goncalves 

The lowest speed grade of Toradex AM62 SoMs is K speed, resulting in a
max value of 800MHz for the CPU operating frequency. A solution with
runtime selection of the CPU frequency is already planned to avoid these
kinds of problems in the future.

Fixes: 8fb8a6d49977 ("arm: dts: k3-am625-verdin-r5:Change CPU
frequency to 1000MHz")
Signed-off-by: Joao Paulo Goncalves 
---
 arch/arm/dts/k3-am625-verdin-r5.dts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm/dts/k3-am625-verdin-r5.dts 
b/arch/arm/dts/k3-am625-verdin-r5.dts
index 6b03e7405af..2b333e70f5c 100644
--- a/arch/arm/dts/k3-am625-verdin-r5.dts
+++ b/arch/arm/dts/k3-am625-verdin-r5.dts
@@ -23,7 +23,7 @@
 */
assigned-clocks = <_clks 61 0>, <_clks 135 0>, <_clks 
157 20>;
assigned-clock-parents = <_clks 61 2>, <0>, <_clks 157 
22>;
-   assigned-clock-rates = <2>, <10>, <2500>;
+   assigned-clock-rates = <2>, <8>, <2500>;
clocks = <_clks 61 0>;
power-domains = <_pds 61 TI_SCI_PD_EXCLUSIVE>,
<_pds 135 TI_SCI_PD_EXCLUSIVE>,
-- 
2.34.1