[PATCH] arm: imx: imx8m: Enable the SError exception

To work with commit 2f3c920(imx8m: workaround ROM serror), we need to enable the SError exception and install vector in SPL.
Signed-off-by: Ye Li ye.li@nxp.com Reviewed-by: Peng Fan peng.fan@nxp.com --- arch/arm/mach-imx/imx8m/Kconfig | 3 +++ 1 file changed, 3 insertions(+)
diff --git a/arch/arm/mach-imx/imx8m/Kconfig b/arch/arm/mach-imx/imx8m/Kconfig index 67da198..88ef732 100644 --- a/arch/arm/mach-imx/imx8m/Kconfig +++ b/arch/arm/mach-imx/imx8m/Kconfig @@ -11,14 +11,17 @@ config IMX8MQ bool select IMX8M select CLK_IMX8MQ + select ARMV8_SPL_EXCEPTION_VECTORS
config IMX8MM bool select IMX8M + select ARMV8_SPL_EXCEPTION_VECTORS
config IMX8MN bool select IMX8M + select ARMV8_SPL_EXCEPTION_VECTORS
config IMX8MP bool

On Fri, Jan 12, 2024 at 4:07 AM Ye Li ye.li@nxp.com wrote:
To work with commit 2f3c920(imx8m: workaround ROM serror), we need to enable the SError exception and install vector in SPL.
Signed-off-by: Ye Li ye.li@nxp.com Reviewed-by: Peng Fan peng.fan@nxp.com
Reported-by: Thomas Schaefer thomas.schaefer@kontron.com Reviewed-by: Fabio Estevam festevam@gmail.com

Hi Ye Li,
On Fri, Jan 12, 2024 at 4:07 AM Ye Li ye.li@nxp.com wrote:
To work with commit 2f3c920(imx8m: workaround ROM serror), we need to enable the SError exception and install vector in SPL.
Signed-off-by: Ye Li ye.li@nxp.com Reviewed-by: Peng Fan peng.fan@nxp.com
This causes an image increase that makes SPL to not fit into the internal SRAM on some boards:
https://source.denx.de/u-boot/custodians/u-boot-imx/-/jobs/769422

Hi Ye Li,
I can confirm that our i.MX8MNano LPD4 EVK with field return fuse blown can boot to u-boot prompt after applying your patch.
Thank you for your support.
Best regards, Thomas
-----Ursprüngliche Nachricht----- Von: Fabio Estevam festevam@gmail.com Gesendet: Freitag, 12. Januar 2024 19:01 An: Ye Li ye.li@nxp.com Cc: sbabic@denx.de; u-boot@lists.denx.de; peng.fan@nxp.com; uboot-imx@nxp.com; Thomas Schaefer thomas.schaefer@kontron.com Betreff: Re: [PATCH] arm: imx: imx8m: Enable the SError exception
Hi Ye Li,
On Fri, Jan 12, 2024 at 4:07 AM Ye Li ye.li@nxp.com wrote:
To work with commit 2f3c920(imx8m: workaround ROM serror), we need to enable the SError exception and install vector in SPL.
Signed-off-by: Ye Li ye.li@nxp.com Reviewed-by: Peng Fan peng.fan@nxp.com
This causes an image increase that makes SPL to not fit into the internal SRAM on some boards:
https://source.denx.de/u-boot/custodians/u-boot-imx/-/jobs/769422

On Fri, Jan 12, 2024 at 3:00 PM Fabio Estevam festevam@gmail.com wrote:
Hi Ye Li,
On Fri, Jan 12, 2024 at 4:07 AM Ye Li ye.li@nxp.com wrote:
To work with commit 2f3c920(imx8m: workaround ROM serror), we need to enable the SError exception and install vector in SPL.
Signed-off-by: Ye Li ye.li@nxp.com Reviewed-by: Peng Fan peng.fan@nxp.com
This causes an image increase that makes SPL to not fit into the internal SRAM on some boards:
https://source.denx.de/u-boot/custodians/u-boot-imx/-/jobs/769422
To fix this problem, what if we enable LTO for all imx8m platforms?
diff --git a/arch/arm/mach-imx/imx8m/Kconfig b/arch/arm/mach-imx/imx8m/Kconfig index b1937198402f..6617a631b4d6 100644 --- a/arch/arm/mach-imx/imx8m/Kconfig +++ b/arch/arm/mach-imx/imx8m/Kconfig @@ -7,19 +7,23 @@ config IMX8M select HAS_CAAM select ROM_UNIFIED_SECTIONS select ARMV8_CRYPTO + select LTO
config IMX8MQ
What do you think?

On Fri, Jan 12, 2024 at 1:07 AM Ye Li ye.li@nxp.com wrote:
To work with commit 2f3c920(imx8m: workaround ROM serror), we need to enable the SError exception and install vector in SPL.
Signed-off-by: Ye Li ye.li@nxp.com Reviewed-by: Peng Fan peng.fan@nxp.com
arch/arm/mach-imx/imx8m/Kconfig | 3 +++ 1 file changed, 3 insertions(+)
diff --git a/arch/arm/mach-imx/imx8m/Kconfig b/arch/arm/mach-imx/imx8m/Kconfig index 67da198..88ef732 100644 --- a/arch/arm/mach-imx/imx8m/Kconfig +++ b/arch/arm/mach-imx/imx8m/Kconfig @@ -11,14 +11,17 @@ config IMX8MQ bool select IMX8M select CLK_IMX8MQ
select ARMV8_SPL_EXCEPTION_VECTORS
config IMX8MM bool select IMX8M
select ARMV8_SPL_EXCEPTION_VECTORS
config IMX8MN bool select IMX8M
select ARMV8_SPL_EXCEPTION_VECTORS
config IMX8MP bool -- 2.7.4
Would this not also work for 8MP? If that's the case, should the section of ARMV8_SPL_EXCEPTION_VECTORS fall under "config IMX8M" for simplicity?
adam
participants (4)
-
Adam Ford
-
Fabio Estevam
-
Thomas Schaefer
-
Ye Li