Message ID | 20230626143136.2260037-1-festevam@gmail.com |
---|---|
State | Accepted |
Commit | 4347dec6b27a373517e25596c8d7b767fe20b922 |
Delegated to: | Tom Rini |
Headers | show |
Series | smegw01: Fix duplicate bootcmd | expand |
On Mon, Jun 26, 2023 at 11:31:36AM -0300, Fabio Estevam wrote: > From: Eduard Strehlau <eduard@lionizers.com> > > Two conflicting bootcmds were included in the environment. > Streamline to defining the bootcmd only in the env file. > > Signed-off-by: Eduard Strehlau <eduard@lionizers.com> > Signed-off-by: Fabio Estevam <festevam@denx.de> Applied to u-boot/master, thanks!
diff --git a/board/storopack/smegw01/smegw01.env b/board/storopack/smegw01/smegw01.env index 25bc7cdbd28b..1263ddac8ed0 100644 --- a/board/storopack/smegw01/smegw01.env +++ b/board/storopack/smegw01/smegw01.env @@ -29,7 +29,16 @@ altbootcmd= run bootcmd; boot_emmc=setenv mmcdev_wanted 1; run persist_mmcdev; run bootcmd; boot_sd=setenv mmcdev_wanted 0; run persist_mmcdev; run bootcmd; -bootcmd=run finduuid; run distro_bootcmd +bootcmd= + if test "${bootcount}" -gt "${bootlimit}"; then + run altbootcmd; + else + if test "${ustate}" = 1; then + setenv upgrade_available 1; + saveenv; + fi; + run mmcboot; + fi; bootdelay=2 bootlimit=3 bootm_size=0x10000000 diff --git a/configs/smegw01_defconfig b/configs/smegw01_defconfig index 3d43c9b996e0..7f1b2bee8e16 100644 --- a/configs/smegw01_defconfig +++ b/configs/smegw01_defconfig @@ -21,8 +21,6 @@ CONFIG_FIT_VERBOSE=y # CONFIG_BOOTSTD is not set CONFIG_AUTOBOOT_MENU_SHOW=y CONFIG_BOOTMENU_DISABLE_UBOOT_CONSOLE=y -CONFIG_USE_BOOTCOMMAND=y -CONFIG_BOOTCOMMAND="if test \"${bootcount}\" -gt \"${bootlimit}\"; then run altbootcmd; else if test \"${ustate}\" = 1; then setenv upgrade_available 1; saveenv; fi; run mmcboot; fi;" CONFIG_USE_PREBOOT=y CONFIG_PREBOOT="run setup_boot_menu;" CONFIG_HUSH_PARSER=y