[PATCH] imx8mn-venice-u-boot: Fix broken boot
When the imx8mm.dtsi file was pulled in from Linux, the UARTs were moved into an spba sub-node which wasn't being included in the SPL device tree. This meant the references to the UART weren't being handled properly and when booting the system would constantly reboot. Fix this by adding the spba node to the spl device tree to restore normal booting.
Based on the patch from Adam Ford for the imx8mn-beacon-kit-u-boot board.
Fixes: 4e5114daf9eb ("imx8mn: synchronise device tree with linux") Signed-off-by: Fabio Estevam festevam@denx.de --- Hi Tim,
Could you please test this patch?
It fixes boot regression on imx8mn-evk and imx8mn-beacon.
arch/arm/dts/imx8mn-venice-u-boot.dtsi | 4 ++++ 1 file changed, 4 insertions(+)
diff --git a/arch/arm/dts/imx8mn-venice-u-boot.dtsi b/arch/arm/dts/imx8mn-venice-u-boot.dtsi index 9fb3871452..9debf9675d 100644 --- a/arch/arm/dts/imx8mn-venice-u-boot.dtsi +++ b/arch/arm/dts/imx8mn-venice-u-boot.dtsi @@ -20,6 +20,10 @@ u-boot,dm-spl; };
+&spba1 { + u-boot,dm-spl; +}; + &clk { u-boot,dm-spl; u-boot,dm-pre-reloc;
Hi
On Mon, Oct 3, 2022 at 3:59 PM Fabio Estevam festevam@denx.de wrote:
When the imx8mm.dtsi file was pulled in from Linux, the UARTs were moved into an spba sub-node which wasn't being included in the SPL device tree. This meant the references to the UART weren't being handled properly and when booting the system would constantly reboot. Fix this by adding the spba node to the spl device tree to restore normal booting.
Based on the patch from Adam Ford for the imx8mn-beacon-kit-u-boot board.
Fixes: 4e5114daf9eb ("imx8mn: synchronise device tree with linux") Signed-off-by: Fabio Estevam festevam@denx.de
Hi Tim,
Could you please test this patch?
It fixes boot regression on imx8mn-evk and imx8mn-beacon.
arch/arm/dts/imx8mn-venice-u-boot.dtsi | 4 ++++ 1 file changed, 4 insertions(+)
diff --git a/arch/arm/dts/imx8mn-venice-u-boot.dtsi b/arch/arm/dts/imx8mn-venice-u-boot.dtsi index 9fb3871452..9debf9675d 100644 --- a/arch/arm/dts/imx8mn-venice-u-boot.dtsi +++ b/arch/arm/dts/imx8mn-venice-u-boot.dtsi @@ -20,6 +20,10 @@ u-boot,dm-spl; };
+&spba1 {
u-boot,dm-spl;+};
&clk { u-boot,dm-spl; u-boot,dm-pre-reloc;
Reviewed-by: Michael Trimarchi michael@amarulasolutions.com
I have seen a series already here I think
-- 2.25.1
When the imx8mm.dtsi file was pulled in from Linux, the UARTs were moved into an spba sub-node which wasn't being included in the SPL device tree. This meant the references to the UART weren't being handled properly and when booting the system would constantly reboot. Fix this by adding the spba node to the spl device tree to restore normal booting. Based on the patch from Adam Ford for the imx8mn-beacon-kit-u-boot board. Fixes: 4e5114daf9eb ("imx8mn: synchronise device tree with linux") Signed-off-by: Fabio Estevam festevam@denx.de Reviewed-by: Michael Trimarchi michael@amarulasolutions.com
Applied to u-boot-imx, master, thanks !
Best regards, Stefano Babic
Hi Stefano,
On Fri, Oct 21, 2022 at 8:50 AM sbabic@denx.de wrote:
When the imx8mm.dtsi file was pulled in from Linux, the UARTs were moved into an spba sub-node which wasn't being included in the SPL device tree. This meant the references to the UART weren't being handled properly and when booting the system would constantly reboot. Fix this by adding the spba node to the spl device tree to restore normal booting. Based on the patch from Adam Ford for the imx8mn-beacon-kit-u-boot board. Fixes: 4e5114daf9eb ("imx8mn: synchronise device tree with linux") Signed-off-by: Fabio Estevam festevam@denx.de Reviewed-by: Michael Trimarchi michael@amarulasolutions.com
Applied to u-boot-imx, master, thanks !
Thanks, but this one is already present in U-Boot 2022.10 final.
Tom applied it directly to avoid the boot regression.
It seems that the u-boot-imx base is not up to date.
participants (4)
-
Fabio Estevam -
Fabio Estevam -
Michael Nazzareno Trimarchi -
sbabic＠denx.de