[PATCH 0/2] board: ti: j721s2: Switch to standard boot

This series switches J721S2 to standard boot flow.
Boot logs: https://gist.github.com/nehamalcom/127019254d49057b5d342f66841bfd3c
Neha Malcom Francis (2): board: ti: j721s2: j721s2.env: Add explicit boot_targets configs: j721e_evm_a72_defconfig: Switch to bootstd
board/ti/j721s2/j721s2.env | 1 + configs/j721s2_evm_a72_defconfig | 5 +++-- include/configs/j721s2_evm.h | 1 - 3 files changed, 4 insertions(+), 3 deletions(-)

Add explicit boot_targets to indicate the specific boot sequence to follow.
Signed-off-by: Neha Malcom Francis n-francis@ti.com --- board/ti/j721s2/j721s2.env | 1 + 1 file changed, 1 insertion(+)
diff --git a/board/ti/j721s2/j721s2.env b/board/ti/j721s2/j721s2.env index 64e3d9da85..e76cafe9ae 100644 --- a/board/ti/j721s2/j721s2.env +++ b/board/ti/j721s2/j721s2.env @@ -21,6 +21,7 @@ args_all=setenv optargs earlycon=ns16550a,mmio32,0x02880000 ${mtdparts} run_kern=booti ${loadaddr} ${rd_spec} ${fdtaddr}
+boot_targets=mmc1 mmc0 usb pxe dhcp boot=mmc mmcdev=1 bootpart=1:2

Switch to using bootstd. Note that with this change, we will stop using distro_bootcmd and instead depend entirely on bootflow method of starting the system up.
Also config_distro_bootcmd.h header file that is no longer needed in j721s2_evm.h.
Signed-off-by: Neha Malcom Francis n-francis@ti.com --- configs/j721s2_evm_a72_defconfig | 5 +++-- include/configs/j721s2_evm.h | 1 - 2 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/configs/j721s2_evm_a72_defconfig b/configs/j721s2_evm_a72_defconfig index a7adb9282b..04bc6a4449 100644 --- a/configs/j721s2_evm_a72_defconfig +++ b/configs/j721s2_evm_a72_defconfig @@ -29,9 +29,10 @@ CONFIG_SPL_SPI=y # CONFIG_SYS_MALLOC_CLEAR_ON_INIT is not set CONFIG_SPL_LOAD_FIT=y CONFIG_SPL_LOAD_FIT_ADDRESS=0x81000000 -CONFIG_DISTRO_DEFAULTS=y CONFIG_OF_SYSTEM_SETUP=y -CONFIG_BOOTCOMMAND="run envboot; run distro_bootcmd;" +CONFIG_BOOTSTD_FULL=y +CONFIG_BOOTSTD_DEFAULTS=y +CONFIG_BOOTCOMMAND="run envboot; bootflow scan -lb" CONFIG_LOGLEVEL=7 CONFIG_SPL_MAX_SIZE=0xc0000 CONFIG_SPL_HAS_BSS_LINKER_SECTION=y diff --git a/include/configs/j721s2_evm.h b/include/configs/j721s2_evm.h index 692c6bb5e4..02673ae798 100644 --- a/include/configs/j721s2_evm.h +++ b/include/configs/j721s2_evm.h @@ -10,7 +10,6 @@ #define __CONFIG_J721S2_EVM_H
#include <linux/sizes.h> -#include <config_distro_bootcmd.h>
/* DDR Configuration */ #define CFG_SYS_SDRAM_BASE1 0x880000000

Hi Neha,
$sub: s/j721e/j721s2/
On 12:50-20231128, Neha Malcom Francis wrote:
Switch to using bootstd. Note that with this change, we will stop using distro_bootcmd and instead depend entirely on bootflow method of starting the system up.
Also config_distro_bootcmd.h header file that is no longer needed in j721s2_evm.h.
Signed-off-by: Neha Malcom Francis n-francis@ti.com
configs/j721s2_evm_a72_defconfig | 5 +++-- include/configs/j721s2_evm.h | 1 - 2 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/configs/j721s2_evm_a72_defconfig b/configs/j721s2_evm_a72_defconfig index a7adb9282b..04bc6a4449 100644 --- a/configs/j721s2_evm_a72_defconfig +++ b/configs/j721s2_evm_a72_defconfig @@ -29,9 +29,10 @@ CONFIG_SPL_SPI=y # CONFIG_SYS_MALLOC_CLEAR_ON_INIT is not set CONFIG_SPL_LOAD_FIT=y CONFIG_SPL_LOAD_FIT_ADDRESS=0x81000000 -CONFIG_DISTRO_DEFAULTS=y CONFIG_OF_SYSTEM_SETUP=y -CONFIG_BOOTCOMMAND="run envboot; run distro_bootcmd;" +CONFIG_BOOTSTD_FULL=y +CONFIG_BOOTSTD_DEFAULTS=y +CONFIG_BOOTCOMMAND="run envboot; bootflow scan -lb"
Since we haven't yet figured out how to run remoteprocs with stdboot [0] can we keep `run boot_rprocs` also in this till that remoteproc stuff is figured out? Don't want to break existing compatibility that we have with this.
[0]: https://lore.kernel.org/u-boot/20231106152205.GG496310@bill-the-cat/
Regards, Manorit
CONFIG_LOGLEVEL=7 CONFIG_SPL_MAX_SIZE=0xc0000 CONFIG_SPL_HAS_BSS_LINKER_SECTION=y diff --git a/include/configs/j721s2_evm.h b/include/configs/j721s2_evm.h index 692c6bb5e4..02673ae798 100644 --- a/include/configs/j721s2_evm.h +++ b/include/configs/j721s2_evm.h @@ -10,7 +10,6 @@ #define __CONFIG_J721S2_EVM_H
#include <linux/sizes.h> -#include <config_distro_bootcmd.h>
/* DDR Configuration */
#define CFG_SYS_SDRAM_BASE1 0x880000000
2.34.1
participants (2)
-
Manorit Chawdhry
-
Neha Malcom Francis