[v3,13/13] smegw01: Fix fallback to altbootcmd

Message ID	20230424111206.1438983-13-festevam@gmail.com
State	Changes Requested
Delegated to:	Tom Rini
Headers	show Return-Path: <u-boot-bounces@lists.denx.de> From: Fabio Estevam <festevam@gmail.com> To: sbabic@denx.de Cc: eduard@lionizers.com, u-boot@lists.denx.de, Fabio Estevam <festevam@denx.de> Subject: [PATCH v3 13/13] smegw01: Fix fallback to altbootcmd Date: Mon, 24 Apr 2023 08:12:06 -0300 Message-Id: <20230424111206.1438983-13-festevam@gmail.com> In-Reply-To: <20230424111206.1438983-1-festevam@gmail.com> References: <20230424111206.1438983-1-festevam@gmail.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Precedence: list Errors-To: u-boot-bounces@lists.denx.de Sender: "U-Boot" <u-boot-bounces@lists.denx.de>
Series	[v3,01/13] smegw01: Enable setting additional boot params \| expand [v3,01/13] smegw01: Enable setting additional boot params [v3,02/13] smegw01: Select CONFIG_CMD_SQUASHFS [v3,03/13] smegw01: Select bootcount support [v3,04/13] smegw01: Add altbootcmd [v3,05/13] smegw01: Run altbootcmd in the case of failure [v3,06/13] smegw01: Only commit to new partition if update was successful [v3,07/13] smegw01: Enable EMMC boot from multiple partitions [v3,08/13] smegw01: Change default boot device to eMMC [v3,09/13] smegw01: Switch to fitImage [v3,10/13] smegw01: Add lockdown U-Boot env support [v3,11/13] smegw01: Read the second MAC address [v3,12/13] smegw01: Disable additional boot menu options [v3,13/13] smegw01: Fix fallback to altbootcmd

Message ID

20230424111206.1438983-13-festevam@gmail.com

State

Changes Requested

Delegated to:

Tom Rini

Headers

From: Fabio Estevam <festevam@gmail.com>
To: sbabic@denx.de
Cc: eduard@lionizers.com, u-boot@lists.denx.de,
 Fabio Estevam <festevam@denx.de>
Subject: [PATCH v3 13/13] smegw01: Fix fallback to altbootcmd
Date: Mon, 24 Apr 2023 08:12:06 -0300
Message-Id: <20230424111206.1438983-13-festevam@gmail.com>
In-Reply-To: <20230424111206.1438983-1-festevam@gmail.com>
References: <20230424111206.1438983-1-festevam@gmail.com>
MIME-Version: 1.0
Content-Transfer-Encoding: 8bit
Precedence: list
Errors-To: u-boot-bounces@lists.denx.de
Sender: "U-Boot" <u-boot-bounces@lists.denx.de>

Series

[v3,01/13] smegw01: Enable setting additional boot params | expand

Commit Message

Fabio Estevam April 24, 2023, 11:12 a.m. UTC

From: Eduard Strehlau <eduard@lionizers.com>

The bootcmd/altbootcmd mechanism is not invoked for bootmenus.
Manually compare bootcount and bootlimit to implement fallback on all
codepaths.

Signed-off-by: Eduard Strehlau <eduard@lionizers.com>
Signed-off-by: Fabio Estevam <festevam@denx.de>
---
Changes since v2:
- None

 configs/smegw01_defconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/configs/smegw01_defconfig b/configs/smegw01_defconfig
index fe8152f33e..3d2713f528 100644
--- a/configs/smegw01_defconfig
+++ b/configs/smegw01_defconfig
@@ -23,7 +23,7 @@  CONFIG_AUTOBOOT_MENU_SHOW=y
 # CONFIG_CMD_BOOTEFI_BOOTMGR is not set
 CONFIG_BOOTMENU_DISABLE_UBOOT_CONSOLE=y
 CONFIG_USE_BOOTCOMMAND=y
-CONFIG_BOOTCOMMAND="if test \"${ustate}\" = 1; then setenv upgrade_available 1; saveenv; fi; run mmcboot; "
+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

[v3,13/13] smegw01: Fix fallback to altbootcmd

Commit Message

Patch