Linux/PROCESSOR-SDK-AM65X: UART boot halts when using MAIN_UART1 as serial console

Matt,

Also, please take a look at the attached powerdomain patch to uart to see if it helps.

Rex

https://e2e.ti.com/cfs-file/__key/communityserver-discussions-components-files/791/uart_5F00_powerdomain.patch

Hi Rex,

Thank you for the response. When I say the boot process fails, I mean that output stops and no further interaction is possible with the board. The following log is from stdout-path= being updated in both the r5-base-board and base-board dts files:

U-Boot SPL 2018.01-00465-gb864b20-dirty (Dec 12 2018 - 12:27:33)
SYSFW ABI: 2.4 (firmware rev 0x0012 '18.8.2-v2018.08b (Curious Crow)')
Trying to boot from UART
CCCCxyzModem - CRC mode, 1(SOH)/493(STX)/0(CAN) packets, 6 retries
Loaded 503912 bytes
Starting ATF on ARM64 core...

<HALTS HERE>

Just to reiterate, this is with a known-working configuration with only references to serial2 being changed to serial3 in stdout-path= in all of the files where it exists, UART1 being muxed and enabled, as well as the UART0 base address being changed to the UART1 base address of 0x02810000.

I did attempt to change the UART configurations in both ATF and OP-TEE, but the "U-Boot for AM65x Family of SoCs" document as well as the board/ti/am65x/README file points users to branches of both ATF and OP-TEE that do not reflect the pre-compiled versions of these components that are included with the latest Linux SDK. Binaries compiled from these sources fail to boot regardless of how they are configured. Can you point me to the exact branches and commits and/or tags used to compile the versions of ATF and OP-TEE included with the latest Linux SDK?

Thanks,

Matt McKee

Hi Rex,

We reworked the AM65x EVM to send the UART1 signals to the FTDI chip without having to worry about software support of the GPIO expander in U-Boot SPL. In software, we modified the following:

ATF, using commit 516abc5fb4826e28be0acdbe6e22fd1b1b476c59,

diff --git a/plat/ti/k3/include/platform_def.h b/plat/ti/k3/include/platform_def.h
index ab0739e..92c0921 100644
--- a/plat/ti/k3/include/platform_def.h
+++ b/plat/ti/k3/include/platform_def.h
@@ -125,7 +125,7 @@
 
 /* Platform default console definitions */
 #ifndef K3_USART_BASE_ADDRESS
-#define K3_USART_BASE_ADDRESS 0x02800000
+#define K3_USART_BASE_ADDRESS 0x02810000
 #endif
 
 /* USART has a default size for address space */

OP-TEE, using the commit 48952f9f76d885df4cb65f8479a4cf7dedf0ee54,

diff --git a/core/arch/arm/plat-k3/platform_config.h b/core/arch/arm/plat-k3/platform_config.h
index b6f0997..700b743 100644
--- a/core/arch/arm/plat-k3/platform_config.h
+++ b/core/arch/arm/plat-k3/platform_config.h
@@ -14,7 +14,7 @@
 #define UART2_BASE      0x02820000
 
 /* UART0 */
-#define CONSOLE_UART_BASE       UART0_BASE
+#define CONSOLE_UART_BASE       UART1_BASE
 #define CONSOLE_BAUDRATE        115200
 #define CONSOLE_UART_CLK_IN_HZ  48000000

U-Boot, using the version included with the SDK,

diff --git a/arch/arm/dts/k3-am654-base-board-u-boot.dtsi b/arch/arm/dts/k3-am654-base-board-u-boot.dtsi
index 3211b86..26b1129 100644
--- a/arch/arm/dts/k3-am654-base-board-u-boot.dtsi
+++ b/arch/arm/dts/k3-am654-base-board-u-boot.dtsi
@@ -5,7 +5,7 @@
 
 / {
        chosen {
-               stdout-path = "serial2:115200n8";
+               stdout-path = "serial3:115200n8";
                tick-timer = &timer1;
        };
diff --git a/arch/arm/dts/k3-am654-base-board.dts b/arch/arm/dts/k3-am654-base-board.dts
index 01c7b5e..88595e3 100644
--- a/arch/arm/dts/k3-am654-base-board.dts
+++ b/arch/arm/dts/k3-am654-base-board.dts
@@ -14,8 +14,8 @@
        model = "Texas Instruments AM654 Base Board";
 
        chosen {
-               stdout-path = "serial2:115200n8";
-               bootargs = "earlycon=ns16550a,mmio32,0x02800000";
+               stdout-path = "serial3:115200n8";
+               bootargs = "earlycon=ns16550a,mmio32,0x02810000";
        };
 
        memory@80000000 {
@@ -116,6 +116,12 @@
        status = "okay";
 };
 
+&main_uart1 {
+       pinctrl-names = "default";
+       pinctrl-0 = <&main_uart1_pins_default>;
+       status = "okay";
+};
+
 &main_pmx0 {
        main_uart0_pins_default: main_uart0_pins_default {
                pinctrl-single,pins = <
@@ -126,6 +132,15 @@
                >;
        };
 
+       main_uart1_pins_default: main_uart1_pins_default {
+               pinctrl-single,pins = <
+                       AM65X_IOPAD(0x014c, PIN_OUTPUT | MUX_MODE6)     /* (AD23) PRG1_PRU1GPO7.UART1_TXD */
+                       AM65X_IOPAD(0x0174, PIN_INPUT | MUX_MODE6)      /* (AE23) PRG1_PRU1GPO17.UART1_RXD */
+                       AM65X_IOPAD(0x0178, PIN_INPUT | MUX_MODE6)      /* (AD22) PRG1_PRU1GPO18.UART1_CTS */
+                       AM65X_IOPAD(0x017c, PIN_OUTPUT | MUX_MODE6)     /* (AC21) PRG1_PRU1GPO19.UART1_RTS */
+               >;
+       };
+
        main_i2c2_pins_default: main_i2c2_pins_default {
                pinctrl-single,pins = <
                        AM65X_IOPAD(0x0074, PIN_INPUT | MUX_MODE5) /* (T27) GPMC0_CSn3.I2C2_SCL */
diff --git a/arch/arm/dts/k3-am654-r5-base-board.dts b/arch/arm/dts/k3-am654-r5-base-board.dts
index 859fef3..e011494 100644
--- a/arch/arm/dts/k3-am654-r5-base-board.dts
+++ b/arch/arm/dts/k3-am654-r5-base-board.dts
@@ -80,6 +80,13 @@
        status = "okay";
 };
 
+&main_uart1 {
+       u-boot,dm-spl;
+       pinctrl-names = "default";
+       pinctrl-0 = <&main_uart1_pins_default>;
+       status = "okay";
+};
+
 &wkup_uart0 {
        u-boot,dm-spl;
        pinctrl-names = "default";
@@ -99,6 +106,16 @@
                u-boot,dm-spl;
        };
 
+       main_uart1_pins_default: main_uart1_pins_default {
+               pinctrl-single,pins = <
+                       AM65X_IOPAD(0x014c, PIN_OUTPUT | MUX_MODE6)     /* (AD23) PRG1_PRU1GPO7.UART1_TXD */
+                       AM65X_IOPAD(0x0174, PIN_INPUT | MUX_MODE6)      /* (AE23) PRG1_PRU1GPO17.UART1_RXD */
+                       AM65X_IOPAD(0x0178, PIN_INPUT | MUX_MODE6)      /* (AD22) PRG1_PRU1GPO18.UART1_CTS */
+                       AM65X_IOPAD(0x017c, PIN_OUTPUT | MUX_MODE6)     /* (AC21) PRG1_PRU1GPO19.UART1_RTS */
+               >;
+               u-boot,dm-spl;
+       };
+
        main_mmc0_pins_default: main_mmc0_pins_default {
                u-boot,dm-pre-reloc;
                pinctrl-single,pins = <
diff --git a/include/configs/am65x_evm.h b/include/configs/am65x_evm.h
index 960e08a..b83ec6f 100644
--- a/include/configs/am65x_evm.h
+++ b/include/configs/am65x_evm.h
@@ -67,7 +67,7 @@
        "overlayaddr=0x83000000\0"                                      \
        "name_kern=Image\0"                                             \
        "console=ttyS2,115200n8\0"                                      \
-       "args_all=setenv optargs earlycon=ns16550a,mmio32,0x02800000 "  \
+       "args_all=setenv optargs earlycon=ns16550a,mmio32,0x02810000 "  \
                "${mtdparts}\0"                                         \
        "run_kern=booti ${loadaddr} ${rd_spec} ${fdtaddr}\0"

We then compiled U-Boot, pointing to the correct ATF and OP-TEE binaries. With this configuration, we do get initial output on UART1 on boot but the boot process fails when loading OP-TEE in the same way that it fails on our hardware:

U-Boot SPL 2018.01-00465-gb864b20-dirty (Dec 17 2018 - 08:29:45)
SYSFW ABI: 2.4 (firmware rev 0x0012 '18.8.2-v2018.08b (Curious Crow)')
Trying to boot from MMC2
Starting ATF on ARM64 core...

NOTICE:  BL31: v1.5(release):ti2018.03-rc1-dirty
NOTICE:  BL31: Built : 08:21:36, Dec 17 2018

No further output. Any suggestions?

Thanks,

Matt McKee

Hi Rex,

Tried it without OP-TEE as you suggested. Built ATF with the following command after a distclean:

make CROSS_COMPILE=aarch64-linux-gnu- ARCH=aarch64 PLAT=k3 TARGET_BOARD=generic

Re-built U-Boot pointing to the new ATF and with and without the TEE variable specified. This is the result:

U-Boot SPL 2018.01-00468-g1f974c6-dirty (Dec 21 2018 - 10:58:50)
SYSFW ABI: 2.4 (firmware rev 0x0012 '18.8.2-v2018.08b (Curious Crow)')
Trying to boot from MMC2
Starting ATF on ARM64 core...

NOTICE:  BL31: v1.5(release):ti2018.03-rc1-dirty
NOTICE:  BL31: Built : 10:41:45, Dec 21 2018
ti_sci_get_response: Message receive failed. ret = -110
Mbox send fail -110
Failed to put device 159 (-110)
### ERROR ### Please RESET the board ###
▒
U-Boot SPL 2018.01-00468-g1f974c6-dirty (Dec 21 2018 - 11:02:28)
SYSFW ABI: 2.4 (firmware rev 0x0012 '18.8.2-v2018.08b (Curious Crow)')
Trying to boot from MMC2
Starting ATF on ARM64 core...

NOTICE:  BL31: v1.5(release):ti2018.03-rc1-dirty
NOTICE:  BL31: Built : 10:41:45, Dec 21 2018
ti_sci_get_response: Message receive failed. ret = -110
Mbox send fail -110
Failed to put device 159 (-110)
### ERROR ### Please RESET the board ###

So it looks like building ATF without SPD=opteed in the build line isn't sufficient for skipping OP-TEE.

Thanks,

Matt McKee

Hi, Matt,

Do you mind that I close this thread for now? However, I would like you to create a new thread for uart boot issue with sysfw on AM65x and let's work on the new thread.

Rex