Thread (2 messages) 2 messages, 2 authors, 2019-07-24

Re: [PATCH] arm64: dts: imx8mm: Fix boot hang at clk init

From: Abel Vesa <hidden>
Date: 2019-07-24 20:28:46
Also in: linux-clk

On 19-07-24 22:17:15, Leonard Crestez wrote:
quoted hunk ↗ jump to hunk
The assigned-clock-rates property is incorrect (shorter than
assigned-clocks) and boot hangs becuause incorrect rates are assigned.

Fix by inserting another <0>

Fixes: df1703896a17 ("arm64: dts: imx8mm: Init rates and parents configs for clocks")

Signed-off-by: Leonard Crestez <redacted>
---
Found in next-20190724 because that's when df1703896a17 was included.
Perhaps this should be squashed?
diff --git a/arch/arm64/boot/dts/freescale/imx8mm.dtsi b/arch/arm64/boot/dts/freescale/imx8mm.dtsi
index c9a7c0054c5c..8737f833cf9c 100644
--- a/arch/arm64/boot/dts/freescale/imx8mm.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8mm.dtsi
@@ -461,10 +461,11 @@
 						<&clk IMX8MM_SYS_PLL3>,
 						<&clk IMX8MM_VIDEO_PLL1>;
 				assigned-clock-parents = <&clk IMX8MM_SYS_PLL3_OUT>,
 							 <&clk IMX8MM_SYS_PLL1_800M>;
 				assigned-clock-rates = <0>,
+							<0>,
In our tree we have this in the clock provider:

	clk_set_rate(clks[IMX8MM_CLK_AUDIO_AHB], 400000000);
	clk_set_rate(clks[IMX8MM_CLK_IPG_AUDIO_ROOT], 400000000);


So I guess instead of the newly added <0> you'll have to put <400000000>
which would be the rate for IMX8MM_CLK_IPG_AUDIO_ROOT.
 							<400000000>,
 							<750000000>,
 							<594000000>;
 			};
 
-- 
2.17.1
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help