diff mbox series

[V2,1/2] imx: imx8mp_evk: fix boot issue

Message ID 20200511071830.10493-1-peng.fan@nxp.com
State Changes Requested
Delegated to: Stefano Babic
Headers show
Series [V2,1/2] imx: imx8mp_evk: fix boot issue | expand

Commit Message

Peng Fan May 11, 2020, 7:18 a.m. UTC
The u-boot-spl.bin pad with ddr firmware conflicts with the
CONFIG_MALLOC_F_ADDR area, the ddr firmware will be overwritten
by malloc in SPL stage and cause ddr initialization not able
to finish. So update the related addresses to fix the issue.

Reported-by: Fabio Estevam <festevam@gmail.com>
Signed-off-by: Peng Fan <peng.fan@nxp.com>
---

V2:
 None

 include/configs/imx8mp_evk.h | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

Comments

Fabio Estevam May 11, 2020, 12:09 p.m. UTC | #1
Hi Peng,

On Mon, May 11, 2020 at 3:55 AM Peng Fan <peng.fan@nxp.com> wrote:
>
> The u-boot-spl.bin pad with ddr firmware conflicts with the
> CONFIG_MALLOC_F_ADDR area, the ddr firmware will be overwritten
> by malloc in SPL stage and cause ddr initialization not able
> to finish. So update the related addresses to fix the issue.
>
> Reported-by: Fabio Estevam <festevam@gmail.com>
> Signed-off-by: Peng Fan <peng.fan@nxp.com>

It does allow to boot U-Boot, but 'reset' is broken:

U-Boot 2020.07-rc1-00001-gaedb60e072-dirty (May 11 2020 - 09:01:16 -0300)

CPU:   Freescale i.MX8MP rev1.0 at 1000 MHz
Reset cause: POR
Model: NXP i.MX8MPlus EVK board
DRAM:  6 GiB
WDT:   Started with servicing (60s timeout)
MMC:   FSL_SDHC: 1, FSL_SDHC: 2
Loading Environment from MMC... OK
In:    serial
Out:   serial
Err:   serial
Net:   No ethernet found.
Hit any key to stop autoboot:  0
u-boot=> reset
resetting ...
"Synchronous Abort" handler, esr 0x5e000000
elr: 000000004020011c lr : 00000000402001a8 (reloc)
elr: 00000000fff6311c lr : 00000000fff631a8
x0 : 0000000084000009 x1 : 0000000000000000
x2 : 0000000000000000 x3 : 0000000000000000
x4 : 0000000000000000 x5 : 0000000000000000
x6 : 0000000000000000 x7 : 0000000000000000
x8 : 00000000fdf5e0b0 x9 : 000000000000000c
x10: 00000000000008d4 x11: 00000000fdf5dbfc
x12: 00000000000008b5 x13: 0000000000003908
x14: 00000000fdf5dfa8 x15: 0000000000000002
x16: 0000000000001080 x17: 0000000000004190
x18: 00000000fdf62dd0 x19: 00000000ffffffda
x20: 0000000000000001 x21: 0000000000000000
x22: 00000000fdf71680 x23: 0000000000000001
x24: 00000000fffdf924 x25: 0000000000000000
x26: 0000000000000000 x27: 0000000000000000
x28: 00000000fdf716c0 x29: 00000000fdf5dd90

Code: d65f03c0 17ffffc7 ffffffff d4000003 (f94003e4)
Resetting CPU ...

Also, could you please make sure it can boot a NXP 5.4.3_2.0.0 kernel?

Thanks
Peng Fan May 11, 2020, 12:23 p.m. UTC | #2
> Subject: Re: [PATCH V2 1/2] imx: imx8mp_evk: fix boot issue
> 
> Hi Peng,
> 
> On Mon, May 11, 2020 at 3:55 AM Peng Fan <peng.fan@nxp.com> wrote:
> >
> > The u-boot-spl.bin pad with ddr firmware conflicts with the
> > CONFIG_MALLOC_F_ADDR area, the ddr firmware will be overwritten by
> > malloc in SPL stage and cause ddr initialization not able to finish.
> > So update the related addresses to fix the issue.
> >
> > Reported-by: Fabio Estevam <festevam@gmail.com>
> > Signed-off-by: Peng Fan <peng.fan@nxp.com>
> 
> It does allow to boot U-Boot, but 'reset' is broken:
> 
> U-Boot 2020.07-rc1-00001-gaedb60e072-dirty (May 11 2020 - 09:01:16
> -0300)
> 
> CPU:   Freescale i.MX8MP rev1.0 at 1000 MHz
> Reset cause: POR
> Model: NXP i.MX8MPlus EVK board
> DRAM:  6 GiB
> WDT:   Started with servicing (60s timeout)
> MMC:   FSL_SDHC: 1, FSL_SDHC: 2
> Loading Environment from MMC... OK
> In:    serial
> Out:   serial
> Err:   serial
> Net:   No ethernet found.
> Hit any key to stop autoboot:  0
> u-boot=> reset
> resetting ...
> "Synchronous Abort" handler, esr 0x5e000000
> elr: 000000004020011c lr : 00000000402001a8 (reloc)
> elr: 00000000fff6311c lr : 00000000fff631a8
> x0 : 0000000084000009 x1 : 0000000000000000
> x2 : 0000000000000000 x3 : 0000000000000000
> x4 : 0000000000000000 x5 : 0000000000000000
> x6 : 0000000000000000 x7 : 0000000000000000
> x8 : 00000000fdf5e0b0 x9 : 000000000000000c
> x10: 00000000000008d4 x11: 00000000fdf5dbfc
> x12: 00000000000008b5 x13: 0000000000003908
> x14: 00000000fdf5dfa8 x15: 0000000000000002
> x16: 0000000000001080 x17: 0000000000004190
> x18: 00000000fdf62dd0 x19: 00000000ffffffda
> x20: 0000000000000001 x21: 0000000000000000
> x22: 00000000fdf71680 x23: 0000000000000001
> x24: 00000000fffdf924 x25: 0000000000000000
> x26: 0000000000000000 x27: 0000000000000000
> x28: 00000000fdf716c0 x29: 00000000fdf5dd90
> 
> Code: d65f03c0 17ffffc7 ffffffff d4000003 (f94003e4) Resetting CPU ...

U-Boot proper should use ATF to reset. I think it might be
using wdog here.

> 
> Also, could you please make sure it can boot a NXP 5.4.3_2.0.0 kernel?

ok, will give a try.

Thanks,
Peng.

> 
> Thanks
diff mbox series

Patch

diff --git a/include/configs/imx8mp_evk.h b/include/configs/imx8mp_evk.h
index 80e5738961..b90a4f6932 100644
--- a/include/configs/imx8mp_evk.h
+++ b/include/configs/imx8mp_evk.h
@@ -23,15 +23,15 @@ 
 #ifdef CONFIG_SPL_BUILD
 /*#define CONFIG_ENABLE_DDR_TRAINING_DEBUG*/
 #define CONFIG_SPL_LDSCRIPT		"arch/arm/cpu/armv8/u-boot-spl.lds"
-#define CONFIG_SPL_STACK		0x990000
-#define CONFIG_SPL_BSS_START_ADDR      0x0095e000
-#define CONFIG_SPL_BSS_MAX_SIZE        0x2000	/* 8 KB */
+#define CONFIG_SPL_STACK		0x98fc00
+#define CONFIG_SPL_BSS_START_ADDR      0x0098fc00
+#define CONFIG_SPL_BSS_MAX_SIZE        0x400	/* 1 KB */
 #define CONFIG_SYS_SPL_MALLOC_START    0x42200000
 #define CONFIG_SYS_SPL_MALLOC_SIZE     SZ_512K	/* 512 KB */
 #define CONFIG_SYS_ICACHE_OFF
 #define CONFIG_SYS_DCACHE_OFF
 
-#define CONFIG_MALLOC_F_ADDR		0x940000
+#define CONFIG_MALLOC_F_ADDR		0x950000
 
 #define CONFIG_SPL_ABORT_ON_RAW_IMAGE