commit 6902abba270c0fbcbe8ee3bb56fe39bc9acc2774
author Jamie McCrae <jamie.mccrae@nordicsemi.no> Tue Apr 04 13:50:43 2023 +0100
committer Dominik Ermel <56024351+de-nordic@users.noreply.github.com> Thu Apr 13 20:07:55 2023 +0200
tree 9158de946082661986891fac37134d38ac3b903d
parent 35941feacd0650bc3e7dc72e7490d589367d553c

zephyr: Create common boot serial enter function

Creates a common function for enterial serial recovery mode.

Signed-off-by: Jamie McCrae <jamie.mccrae@nordicsemi.no>

boot/zephyr/main.c

diff --git a/boot/zephyr/main.c b/boot/zephyr/main.c
index 3f6a87b..2deef1e 100644
--- a/boot/zephyr/main.c
+++ b/boot/zephyr/main.c
@@ -476,6 +476,25 @@
 }
 #endif
 
+#ifdef CONFIG_MCUBOOT_SERIAL
+static void boot_serial_enter()
+{
+    int rc;
+
+#ifdef CONFIG_MCUBOOT_INDICATION_LED
+    gpio_pin_set_dt(&led0, 1);
+#endif
+
+    mcuboot_status_change(MCUBOOT_STATUS_SERIAL_DFU_ENTERED);
+
+    BOOT_LOG_INF("Enter the serial recovery mode");
+    rc = boot_console_init();
+    __ASSERT(rc == 0, "Error initializing boot console.\n");
+    boot_serial_start(&boot_funcs);
+    __ASSERT(0, "Bootloader serial process was terminated unexpectedly.\n");
+}
+#endif
+
 void main(void)
 {
     struct boot_rsp rsp;
@@ -514,17 +533,7 @@
 #ifdef CONFIG_BOOT_SERIAL_ENTRANCE_GPIO
     if (detect_pin() &&
             !boot_skip_serial_recovery()) {
-#ifdef CONFIG_MCUBOOT_INDICATION_LED
-        gpio_pin_set_dt(&led0, 1);
-#endif
-
-        mcuboot_status_change(MCUBOOT_STATUS_SERIAL_DFU_ENTERED);
-
-        BOOT_LOG_INF("Enter the serial recovery mode");
-        rc = boot_console_init();
-        __ASSERT(rc == 0, "Error initializing boot console.\n");
-        boot_serial_start(&boot_funcs);
-        __ASSERT(0, "Bootloader serial process was terminated unexpectedly.\n");
+        boot_serial_enter();
     }
 #endif
 
@@ -532,18 +541,8 @@
     rc = hwinfo_get_reset_cause(&reset_cause);
 
     if (rc == 0 && reset_cause == RESET_PIN) {
-#ifdef CONFIG_MCUBOOT_INDICATION_LED
-        gpio_pin_set_dt(&led0, 1);
-#endif
-
-        mcuboot_status_change(MCUBOOT_STATUS_SERIAL_DFU_ENTERED);
         (void)hwinfo_clear_reset_cause();
-
-        BOOT_LOG_INF("Enter the serial recovery mode");
-        rc = boot_console_init();
-        __ASSERT(rc == 0, "Error initializing boot console.\n");
-        boot_serial_start(&boot_funcs);
-        __ASSERT(0, "Bootloader serial process was terminated unexpectedly.\n");
+        boot_serial_enter();
     }
 #endif
 
@@ -600,14 +599,8 @@
         /* Boot mode to stay in bootloader, clear status and enter serial
          * recovery mode
          */
-#ifdef CONFIG_MCUBOOT_INDICATION_LED
-        gpio_pin_set_dt(&led0, 1);
-#endif
-
-        mcuboot_status_change(MCUBOOT_STATUS_SERIAL_DFU_ENTERED);
-        rc = boot_console_init();
         bootmode_clear();
-        boot_serial_start(&boot_funcs);
+        boot_serial_enter();
     }
 #endif
 
@@ -629,13 +622,7 @@
         /* No bootable image and configuration set to remain in serial
          * recovery mode
          */
-#ifdef CONFIG_MCUBOOT_INDICATION_LED
-        gpio_pin_set_dt(&led0, 1);
-#endif
-
-        mcuboot_status_change(MCUBOOT_STATUS_SERIAL_DFU_ENTERED);
-        rc = boot_console_init();
-        boot_serial_start(&boot_funcs);
+        boot_serial_enter();
 #endif
 
         FIH_PANIC;