MCUboot should only timeout after hardware reset

Question

Hi there, 
 I've made MCUboot work, but it always timeouts before loading the application firmware. I only want it to timeout after resetting the SoC via its RESET pin. 
 I added CONFIG_BOOTLOADER_MCUBOOT=y to my prj.conf, and my mcuboot.conf contains the following: 
 CONFIG_PINCTRL=y
CONFIG_PM_PARTITION_SIZE_MCUBOOT=0x10000
CONFIG_LOG=y
CONFIG_SIZE_OPTIMIZATIONS=y
CONFIG_SERIAL=y
CONFIG_UART_LINE_CTRL=y
CONFIG_MCUBOOT_SERIAL=y
CONFIG_BOOT_SERIAL_CDC_ACM=y
CONFIG_BOOT_SERIAL_PIN_RESET=y
CONFIG_BOOT_SERIAL_WAIT_FOR_DFU=y
CONFIG_BOOT_SERIAL_WAIT_FOR_DFU_TIMEOUT=3000
CONFIG_BOOT_SERIAL_NO_APPLICATION=y
CONFIG_LOG_BACKEND_UART=y
CONFIG_LOG_BACKEND_RTT=n
 
 My second issue is that I have to add the `-Dmcuboot_OVERLAY_CONFIG=/home/laci/projects/firmware-uhk/uhk/device/mcuboot.conf` west argument every time I build because the VS Code forgets it: 
 
 How can I persist this setting? The "Save Configuration as Preset" option doesn't make a difference. 
 Thanks in advance! - Laci

Vidar Berg · Accepted Answer

Please try to apply the diff below, then test your project with the following kconfig settings: 
 CONFIG_BOOT_SERIAL_WAIT_FOR_DFU=y
CONFIG_BOOT_SERIAL_WAIT_FOR_DFU_ALWAYS=n
CONFIG_BOOT_SERIAL_WAIT_FOR_DFU_TIMEOUT=3000 
 diff --git a/boot/zephyr/Kconfig.serial_recovery b/boot/zephyr/Kconfig.serial_recovery
index 52ec3b01..3487e37d 100644
--- a/boot/zephyr/Kconfig.serial_recovery
+++ b/boot/zephyr/Kconfig.serial_recovery
@@ -155,13 +155,17 @@ config BOOT_SERIAL_DETECT_DELAY
 the one used to place the device in bootloader mode.
 
 menuconfig BOOT_SERIAL_WAIT_FOR_DFU
-	bool "Wait a prescribed duration to see if DFU is invoked by receiving a MCUmgr comand"
+	bool "Wait a prescribed duration to see if DFU is invoked by receiving a MCUmgr command"
 depends on BOOT_SERIAL_UART || BOOT_SERIAL_CDC_ACM
 help
 If y, MCUboot waits for a prescribed duration of time to allow
 for DFU to be invoked. The serial recovery can be entered by receiving any
 mcumgr command.
 
+config BOOT_SERIAL_WAIT_FOR_DFU_ALWAYS
+	bool "Always enter serial recovery mode upon startup"
+	depends on BOOT_SERIAL_WAIT_FOR_DFU
+
 config BOOT_SERIAL_WAIT_FOR_DFU_TIMEOUT
 int "Duration to wait for the serial DFU timeout in ms"
 default 500
diff --git a/boot/zephyr/main.c b/boot/zephyr/main.c
index 27039700..4364ea67 100644
--- a/boot/zephyr/main.c
+++ b/boot/zephyr/main.c
@@ -559,8 +559,12 @@ static void boot_serial_enter()
 BOOT_LOG_INF("Enter the serial recovery mode");
 rc = boot_console_init();
 __ASSERT(rc == 0, "Error initializing boot console.
");
+#ifdef CONFIG_BOOT_SERIAL_WAIT_FOR_DFU
+ boot_serial_check_start(&boot_funcs, CONFIG_BOOT_SERIAL_WAIT_FOR_DFU_TIMEOUT);
+#else
 boot_serial_start(&boot_funcs);
 __ASSERT(0, "Bootloader serial process was terminated unexpectedly.
");
+#endif 
 }
 #endif
 
@@ -649,7 +653,7 @@ int main(void)
 }
 #endif
 
-#ifdef CONFIG_BOOT_SERIAL_WAIT_FOR_DFU
+#ifdef CONFIG_BOOT_SERIAL_WAIT_FOR_DFU_ALWAYS
 /* Initialize the boot console, so we can already fill up our buffers while
 * waiting for the boot image check to finish. This image check, can take
 * some time, so it's better to reuse thistime to already receive the
@@ -674,7 +678,7 @@ int main(void)
 }
 #endif
 
-#ifdef CONFIG_BOOT_SERIAL_WAIT_FOR_DFU
+#ifdef CONFIG_BOOT_SERIAL_WAIT_FOR_DFU_ALWAYS
 timeout_in_ms -= (k_uptime_get_32() - start);
 if( timeout_in_ms <= 0 ) {
 /* at least one check if time was expired */

MCUboot should only timeout after hardware reset

Top Replies