From 2a74a2b580035e767f4a4ead2b7c8b9253872276 Mon Sep 17 00:00:00 2001
From: Piotr Dymacz <pepe2k@gmail.com>
Date: Tue, 5 Dec 2023 14:26:22 +0100
Subject: [PATCH] zephyr: io: add 'io_led_set()'

The static declaration of 'led0' was moved to 'io.c' which broke
building with the 'MCUBOOT_INDICATION_LED' enabled:

  mcuboot/boot/zephyr/main.c:380:22: error:
    'led0' undeclared (first use in this function)
      380 |     gpio_pin_set_dt(&led0, 1);
          |                      ^~~~

This adds simple function 'io_led_set()' for changing LED's value.

Fixes: 433b8480 ("zephyr: Move IO functions out of main to separate file")
Signed-off-by: Piotr Dymacz <pepe2k@gmail.com>
---
 boot/zephyr/include/io/io.h | 5 +++++
 boot/zephyr/io.c            | 5 +++++
 boot/zephyr/main.c          | 8 ++++----
 3 files changed, 14 insertions(+), 4 deletions(-)

diff --git a/boot/zephyr/include/io/io.h b/boot/zephyr/include/io/io.h
index 332eefbd..145530bb 100644
--- a/boot/zephyr/include/io/io.h
+++ b/boot/zephyr/include/io/io.h
@@ -34,6 +34,11 @@ extern "C" {
  */
 void io_led_init(void);
 
+/*
+ * Sets value of the configured LED.
+ */
+void io_led_set(int value);
+
 /*
  * Checks if GPIO is set in the required way to remain in serial recovery mode
  *
diff --git a/boot/zephyr/io.c b/boot/zephyr/io.c
index e6d54d3c..309f1ab9 100644
--- a/boot/zephyr/io.c
+++ b/boot/zephyr/io.c
@@ -91,6 +91,11 @@ void io_led_init(void)
     gpio_pin_configure_dt(&led0, GPIO_OUTPUT);
     gpio_pin_set_dt(&led0, 0);
 }
+
+void io_led_set(int value)
+{
+    gpio_pin_set_dt(&led0, value);
+}
 #endif /* CONFIG_MCUBOOT_INDICATION_LED */
 
 #if defined(CONFIG_BOOT_SERIAL_ENTRANCE_GPIO) || defined(CONFIG_BOOT_USB_DFU_GPIO) || \
diff --git a/boot/zephyr/main.c b/boot/zephyr/main.c
index 91c9b9b9..c6a0f74a 100644
--- a/boot/zephyr/main.c
+++ b/boot/zephyr/main.c
@@ -377,7 +377,7 @@ static void boot_serial_enter()
     int rc;
 
 #ifdef CONFIG_MCUBOOT_INDICATION_LED
-    gpio_pin_set_dt(&led0, 1);
+    io_led_set(1);
 #endif
 
     mcuboot_status_change(MCUBOOT_STATUS_SERIAL_DFU_ENTERED);
@@ -434,7 +434,7 @@ int main(void)
 #if defined(CONFIG_BOOT_USB_DFU_GPIO)
     if (io_detect_pin()) {
 #ifdef CONFIG_MCUBOOT_INDICATION_LED
-        gpio_pin_set_dt(&led0, 1);
+        io_led_set(1);
 #endif
 
         mcuboot_status_change(MCUBOOT_STATUS_USB_DFU_ENTERED);
@@ -475,7 +475,7 @@ int main(void)
     uint32_t start = k_uptime_get_32();
 
 #ifdef CONFIG_MCUBOOT_INDICATION_LED
-    gpio_pin_set_dt(&led0, 1);
+    io_led_set(1);
 #endif
 #endif
 
@@ -499,7 +499,7 @@ int main(void)
     boot_serial_check_start(&boot_funcs,timeout_in_ms);
 
 #ifdef CONFIG_MCUBOOT_INDICATION_LED
-    gpio_pin_set_dt(&led0, 0);
+    io_led_set(0);
 #endif
 #endif