diff mbox series

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

Message ID 20240319140427.73721-1-jpaulo.silvagoncalves@gmail.com
State Accepted
Commit 2bb18248c64e6a51b53c2571c0ca323a687c49c5
Delegated to: Tom Rini
Headers show
Series arm: dts: k3-am625-verdin-r5: Change CPU frequency to 800MHz | expand

Commit Message

Joao Paulo Goncalves March 19, 2024, 2:04 p.m. UTC 
From: Joao Paulo Goncalves <joao.goncalves@toradex.com>

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 <joao.goncalves@toradex.com>
---
 arch/arm/dts/k3-am625-verdin-r5.dts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Francesco Dolcini March 19, 2024, 3:05 p.m. UTC | #1 
Hello Tom,

On Tue, Mar 19, 2024 at 11:04:27AM -0300, Joao Paulo Goncalves wrote:
> From: Joao Paulo Goncalves <joao.goncalves@toradex.com>
> 
> 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 <joao.goncalves@toradex.com>

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
Tom Rini March 19, 2024, 7:04 p.m. UTC | #2 
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 <joao.goncalves@toradex.com>
> > 
> > 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 <joao.goncalves@toradex.com>
> 
> 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 Rini March 22, 2024, 7:26 p.m. UTC | #3 
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!
diff mbox series

Patch

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 = <&k3_clks 61 0>, <&k3_clks 135 0>, <&k3_clks 157 20>;
 		assigned-clock-parents = <&k3_clks 61 2>, <0>, <&k3_clks 157 22>;
-		assigned-clock-rates = <200000000>, <1000000000>, <25000000>;
+		assigned-clock-rates = <200000000>, <800000000>, <25000000>;
 		clocks = <&k3_clks 61 0>;
 		power-domains = <&k3_pds 61 TI_SCI_PD_EXCLUSIVE>,
 				<&k3_pds 135 TI_SCI_PD_EXCLUSIVE>,