BL31 not loading on Kirkstone

[dpog]

I'm currently facing a problem after switching from dunfell to the kirkstone branch. Within the dunfell branch we used U-Boot 2020.4 and Kernel 5.4, on kirkstone we use U-Boot 2022.4 (u-boot-imx) and Kernel 6.0.8 (linux-fslc). Right now we're reapply the patches to the newer uboot and kernel, but somehow we can't change the console to Uart3. 

With "console=ttymxc2 ....etc" the kernel uses the right console, but when changing uboot to use uart3, even uuu isn't working anymore. 

When changing the settings in imx8mm_evk.h and imx8mm_evk.c, the SPL prints do uart3, but when it should show:

NOTICE: BL31: v2.6(release):lf-5.15.71-2.2.0-0-g3c1583ba0-dirty
NOTICE: BL31: Built : 11:00:38, Nov 21 2022

it is stuck. When booting from a sd-card it does the same. My suspicion is that the BL31-Env is somehow blocking because it should init the wrong console. 

Any suggestions?

[Sanket_Parekh]

Hi @dpog ,

I hope you are doing well.

Please provide me with a patch of changes that were applied.

Please make sure that you have made changes to imx-atf as below.

- IMX_BOOT_UART_BASE ?= 0x30890000

+ IMX_BOOT_UART_BASE ?= 0x30880000

I using op-tee, Please make changes to /core/arch/arm/plat-imx/conf.mk as below.

  ifneq (,$(filter $(PLATFORM_FLAVOR),mx8mmevk))
  CFG_DDR_SIZE ?= 0x80000000
- CFG_UART_BASE ?= UART2_BASE

+ CFG_UART_BASE ?= UART3_BASE
   endif

Thanks & Regards,

Sanket Parekh

[dpog]

I applied the changes to imx-atf as you stated.

These are the changes I made to u-boot-imx:

diff --git a/arch/arm/dts/imx8mm-ddr4-evk-u-boot.dtsi b/arch/arm/dts/imx8mm-ddr4-evk-u-boot.dtsi
index e66bd5830d..8cd270b527 100644
--- a/arch/arm/dts/imx8mm-ddr4-evk-u-boot.dtsi
+++ b/arch/arm/dts/imx8mm-ddr4-evk-u-boot.dtsi
@@ -7,7 +7,7 @@

/ {
chosen {
- bootargs = "console=ttymxc1,115200 earlycon=ec_imx6q,0x30890000,115200";
+ bootargs = "console=ttymxc2,115200 earlycon=ec_imx6q,0x30880000,115200";
};

wdt-reboot {
@@ -52,7 +52,7 @@
u-boot,dm-spl;
};

-&pinctrl_uart2 {
+&pinctrl_uart3 {
u-boot,dm-spl;
};

@@ -84,7 +84,7 @@
u-boot,dm-spl;
};

-&uart2 {
+&uart3 {
u-boot,dm-spl;
};

diff --git a/arch/arm/dts/imx8mm-evk-u-boot.dtsi b/arch/arm/dts/imx8mm-evk-u-boot.dtsi
index 7ff878b5f9..0d6c2ee8dd 100644
--- a/arch/arm/dts/imx8mm-evk-u-boot.dtsi
+++ b/arch/arm/dts/imx8mm-evk-u-boot.dtsi
@@ -48,7 +48,7 @@
u-boot,dm-spl;
};

-&pinctrl_uart2 {
+&pinctrl_uart3 {
u-boot,dm-spl;
};

@@ -84,7 +84,7 @@
u-boot,dm-spl;
};

-&uart2 {
+&uart3 {
u-boot,dm-spl;
};

diff --git a/arch/arm/dts/imx8mm-evk.dts b/arch/arm/dts/imx8mm-evk.dts
index 809b3143b7..1253067cc2 100644
--- a/arch/arm/dts/imx8mm-evk.dts
+++ b/arch/arm/dts/imx8mm-evk.dts
@@ -13,7 +13,7 @@
compatible = "fsl,imx8mm-evk", "fsl,imx8mm";

chosen {
- bootargs = "console=ttymxc1,115200 earlycon=ec_imx6q,0x30890000,115200";
+ bootargs = "console=ttymxc2,115200 earlycon=ec_imx6q,0x30880000,115200";
};

aliases {
diff --git a/arch/arm/dts/imx8mm-evk.dtsi b/arch/arm/dts/imx8mm-evk.dtsi
index 7b80a6650d..36dc7242a3 100644
--- a/arch/arm/dts/imx8mm-evk.dtsi
+++ b/arch/arm/dts/imx8mm-evk.dtsi
@@ -10,7 +10,7 @@

/ {
chosen {
- stdout-path = &uart2;
+ stdout-path = &uart3;
};

memory@40000000 {
@@ -373,9 +373,9 @@
status = "okay";
};

-&uart2 { /* console */
+&uart3 { /* console */
pinctrl-names = "default";
- pinctrl-0 = <&pinctrl_uart2>;
+ pinctrl-0 = <&pinctrl_uart3>;
status = "okay";
};

@@ -565,10 +565,10 @@
>;
};

- pinctrl_uart2: uart2grp {
+ pinctrl_uart3: uart3grp {
fsl,pins = <
- MX8MM_IOMUXC_UART2_RXD_UART2_DCE_RX 0x140
- MX8MM_IOMUXC_UART2_TXD_UART2_DCE_TX 0x140
+ MX8MM_IOMUXC_UART3_RXD_UART3_DCE_RX 0x140
+ MX8MM_IOMUXC_UART3_TXD_UART3_DCE_TX 0x140
>;
};

diff --git a/board/freescale/imx8mm_evk/imx8mm_evk.c b/board/freescale/imx8mm_evk/imx8mm_evk.c
index 39f6d6281c..eac5029921 100644
--- a/board/freescale/imx8mm_evk/imx8mm_evk.c
+++ b/board/freescale/imx8mm_evk/imx8mm_evk.c
@@ -29,8 +29,8 @@ DECLARE_GLOBAL_DATA_PTR;
#define WDOG_PAD_CTRL (PAD_CTL_DSE6 | PAD_CTL_ODE | PAD_CTL_PUE | PAD_CTL_PE)

static iomux_v3_cfg_t const uart_pads[] = {
- IMX8MM_PAD_UART2_RXD_UART2_RX | MUX_PAD_CTRL(UART_PAD_CTRL),
- IMX8MM_PAD_UART2_TXD_UART2_TX | MUX_PAD_CTRL(UART_PAD_CTRL),
+ IMX8MM_PAD_UART3_RXD_UART3_RX | MUX_PAD_CTRL(UART_PAD_CTRL),
+ IMX8MM_PAD_UART3_TXD_UART3_TX | MUX_PAD_CTRL(UART_PAD_CTRL),
};

static iomux_v3_cfg_t const wdog_pads[] = {
@@ -98,7 +98,7 @@ int board_early_init_f(void)

imx_iomux_v3_setup_multiple_pads(uart_pads, ARRAY_SIZE(uart_pads));

- init_uart_clk(1);
+ init_uart_clk(2);

#ifdef CONFIG_NAND_MXS
setup_gpmi_nand(); /* SPL will call the board_early_init_f */
diff --git a/include/configs/imx8mm_evk.h b/include/configs/imx8mm_evk.h
index 0da9de9819..b7177788e9 100644
--- a/include/configs/imx8mm_evk.h
+++ b/include/configs/imx8mm_evk.h
@@ -206,7 +206,7 @@
#define PHYS_SDRAM 0x40000000
#define PHYS_SDRAM_SIZE 0x40000000 /* 2GB DDR */

-#define CONFIG_MXC_UART_BASE UART2_BASE_ADDR
+#define CONFIG_MXC_UART_BASE UART3_BASE_ADDR

/* Monitor Command Prompt */
#define CONFIG_SYS_CBSIZE 2048

[Sanket_Parekh · Accepted solution]

Hi @dpog ,

I hope you are doing well.

The issue seems to be due to yocto overriding ATF changes.

From the yocto Kirkstone version, ATF UART can be configured in yocto without modifying 

Please refer to the below commit.

imx-atf: allow setting the UART used during boot

SERIAL_CONSOLES = "115200;ttymxc1"   // at meta-freescale/conf/machine/include/imx8mm-evk.inc

SOC_ATF_BOOT_UART_BASE:mx8m-generic-bsp   = "0x30890000"  // meta-freescale/conf/machine/include/imx-base.inc

Please change below variables in custom machine configuration file or at meta-freescale/conf/machine/imx8mm-lpddr4-evk.conf 

SERIAL_CONSOLES = "115200;ttymxc2" 

ATF_BOOT_UART_BASE   = "0x30880000"

Thanks & Regards,

Sanket Parekh

[dpog]

Thanks! This did the trick,

Greetings,
Dominik