All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] arm: dts: k3-am625-verdin-r5: Change CPU frequency to 800MHz
@ 2024-03-19 14:04 Joao Paulo Goncalves
  2024-03-19 15:05 ` Francesco Dolcini
  2024-03-22 19:26 ` Tom Rini
  0 siblings, 2 replies; 4+ messages in thread
From: Joao Paulo Goncalves @ 2024-03-19 14:04 UTC (permalink / raw)
  To: Marcel Ziswiler, Tom Rini; +Cc: Joao Paulo Goncalves, u-boot

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(-)

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>,
-- 
2.34.1


^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH] arm: dts: k3-am625-verdin-r5: Change CPU frequency to 800MHz
  2024-03-19 14:04 [PATCH] arm: dts: k3-am625-verdin-r5: Change CPU frequency to 800MHz Joao Paulo Goncalves
@ 2024-03-19 15:05 ` Francesco Dolcini
  2024-03-19 19:04   ` Tom Rini
  2024-03-22 19:26 ` Tom Rini
  1 sibling, 1 reply; 4+ messages in thread
From: Francesco Dolcini @ 2024-03-19 15:05 UTC (permalink / raw)
  To: Tom Rini
  Cc: Marcel Ziswiler, Joao Paulo Goncalves, u-boot,
	Joao Paulo Goncalves, Vignesh Raghavendra

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



^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] arm: dts: k3-am625-verdin-r5: Change CPU frequency to 800MHz
  2024-03-19 15:05 ` Francesco Dolcini
@ 2024-03-19 19:04   ` Tom Rini
  0 siblings, 0 replies; 4+ messages in thread
From: Tom Rini @ 2024-03-19 19:04 UTC (permalink / raw)
  To: Francesco Dolcini
  Cc: Marcel Ziswiler, Joao Paulo Goncalves, u-boot,
	Joao Paulo Goncalves, Vignesh Raghavendra

[-- Attachment #1: Type: text/plain, Size: 917 bytes --]

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

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 659 bytes --]

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] arm: dts: k3-am625-verdin-r5: Change CPU frequency to 800MHz
  2024-03-19 14:04 [PATCH] arm: dts: k3-am625-verdin-r5: Change CPU frequency to 800MHz Joao Paulo Goncalves
  2024-03-19 15:05 ` Francesco Dolcini
@ 2024-03-22 19:26 ` Tom Rini
  1 sibling, 0 replies; 4+ messages in thread
From: Tom Rini @ 2024-03-22 19:26 UTC (permalink / raw)
  To: Marcel Ziswiler, Joao Paulo Goncalves; +Cc: Joao Paulo Goncalves, u-boot

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



^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2024-03-22 19:26 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-03-19 14:04 [PATCH] arm: dts: k3-am625-verdin-r5: Change CPU frequency to 800MHz Joao Paulo Goncalves
2024-03-19 15:05 ` Francesco Dolcini
2024-03-19 19:04   ` Tom Rini
2024-03-22 19:26 ` Tom Rini

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.