boards/xtensa/esp32: Add LED support for

boards/xtensa/esp32/esp32-devkitc/include/board.h

@@ -48,6 +48,7 @@
 /* Define how many LEDs this board has (needed by userleds) */
 
 #define BOARD_NLEDS       1
+#define GPIO_LED1         2  /* Typical On-board LED is on GPIO 2 */
 
 /* GPIO pins used by the GPIO Subsystem */
 

boards/xtensa/esp32/esp32-devkitc/src/esp32_userleds.c

@@ -23,16 +23,10 @@
 /****************************************************************************
  * Included Files
  ****************************************************************************/
-
 #include <nuttx/config.h>
-
-#include <stdint.h>
 #include <stdbool.h>
-#include <nuttx/debug.h>
-
 #include <nuttx/board.h>
 #include <arch/board/board.h>
-
 #include "espressif/esp_gpio.h"
 #include "esp32-devkitc.h"
 
@@ -42,11 +36,7 @@
 
 /* This array maps an LED number to GPIO pin configuration */
 
-static const uint32_t g_ledcfg[BOARD_NLEDS] =
-{
-  GPIO_LED1,
-};
-
+static const uint32_t g_ledcfg[BOARD_NLEDS] = { GPIO_LED1 };
 /****************************************************************************
  * Public Functions
  ****************************************************************************/
@@ -55,42 +45,28 @@
  * Name: board_userled_initialize
  ****************************************************************************/
 
-uint32_t board_userled_initialize(void)
-{
-  uint8_t i;
-
-  for (i = 0; i < BOARD_NLEDS; i++)
-    {
-      esp_configgpio(g_ledcfg[i], OUTPUT);
-    }
-
+uint32_t board_userled_initialize(void) {
+  for (int i = 0; i < BOARD_NLEDS; i++) {
+    esp_configgpio(g_ledcfg[i], OUTPUT);
+  }
   return BOARD_NLEDS;
 }
-
 /****************************************************************************
  * Name: board_userled
  ****************************************************************************/
 
-void board_userled(int led, bool ledon)
-{
-  if ((unsigned)led < BOARD_NLEDS)
-    {
-      esp_gpiowrite(g_ledcfg[led], ledon);
-    }
-}
 
+void board_userled(int led, bool ledon) {
+  if ((unsigned)led < BOARD_NLEDS) {
+    esp_gpiowrite(g_ledcfg[led], ledon);
+  }
+}
 /****************************************************************************
  * Name: board_userled_all
  ****************************************************************************/
 
-void board_userled_all(uint32_t ledset)
-{
-  uint8_t i;
-
-  /* Configure LED1-8 GPIOs for output */
-
-  for (i = 0; i < BOARD_NLEDS; i++)
-    {
-      esp_gpiowrite(g_ledcfg[i], (ledset & (1 << i)) != 0);
-    }
-}
+void board_userled_all(uint32_t ledset) {
+  for (int i = 0; i < BOARD_NLEDS; i++) {
+    esp_gpiowrite(g_ledcfg[i], (ledset & (1 << i)) != 0);
+  }
+}

boards/xtensa/esp32/heltec_wifi_lora32/include/board.h

@@ -48,6 +48,7 @@
 /* Define how many LEDs this board has (needed by userleds) */
 
 #define BOARD_NLEDS       1
+#define GPIO_LED1         25  /* White LED on Heltec WiFi LoRa 32 */
 
 /* GPIO pins used by the GPIO Subsystem */
 

boards/xtensa/esp32/heltec_wifi_lora32/src/Make.defs

@@ -34,3 +34,8 @@ DEPPATH += --dep-path board
 VPATH += :board
 CFLAGS += ${INCDIR_PREFIX}$(TOPDIR)$(DELIM)arch$(DELIM)$(CONFIG_ARCH)$(DELIM)src$(DELIM)board$(DELIM)board
 
+
+ifeq ($(CONFIG_USERLED),y)
+CSRCS += esp32_userleds.c
+endif
+

boards/xtensa/esp32/heltec_wifi_lora32/src/esp32_bringup.c

@@ -38,6 +38,12 @@
 
 #include "esp32_start.h"
 
+#ifdef CONFIG_USERLED
+#  include <nuttx/leds/userled.h>
+#endif
+
+
+
 #ifdef CONFIG_ESPRESSIF_HR_TIMER
 #  include "espressif/esp_hr_timer.h"
 #endif
@@ -110,6 +116,16 @@ int esp32_bringup(void)
    * capabilities.
    */
 
+
+#ifdef CONFIG_USERLED
+  ret = userled_lower_initialize("/dev/userleds");
+  if (ret < 0)
+    {
+      syslog(LOG_ERR, "ERROR: userled_lower_initialize() failed: %d\n", ret);
+    }
+#endif
+
+
   UNUSED(ret);
   return OK;
 }

boards/xtensa/esp32/heltec_wifi_lora32/src/esp32_userleds.c

@@ -0,0 +1,90 @@
+/****************************************************************************
+ * boards/xtensa/esp32/heltec_wifi_lora32/src/esp32_userleds.c
+ *
+ * SPDX-License-Identifier: Apache-2.0
+ *
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements.  See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.  The
+ * ASF licenses this file to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance with the
+ * License.  You may obtain a copy of the License at
+ *
+ *   http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
+ * WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.  See the
+ * License for the specific language governing permissions and limitations
+ * under the License.
+ *
+ ****************************************************************************/
+
+/****************************************************************************
+ * Included Files
+ ****************************************************************************/
+
+#include <nuttx/config.h>
+
+#include <stdbool.h>
+
+#include <nuttx/board.h>
+#include <arch/board/board.h>
+
+#include "espressif/esp_gpio.h"
+#include "heltec_wifi_lora32.h"
+
+/****************************************************************************
+ * Private Data
+ ****************************************************************************/
+
+static const uint32_t g_ledcfg[BOARD_NLEDS] =
+{
+  GPIO_LED1,
+};
+
+/****************************************************************************
+ * Public Functions
+ ****************************************************************************/
+
+/****************************************************************************
+ * Name: board_userled_initialize
+ ****************************************************************************/
+
+uint32_t board_userled_initialize(void)
+{
+  int i;
+
+  for (i = 0; i < BOARD_NLEDS; i++)
+    {
+      esp_configgpio(g_ledcfg[i], OUTPUT);
+    }
+
+  return BOARD_NLEDS;
+}
+
+/****************************************************************************
+ * Name: board_userled
+ ****************************************************************************/
+
+void board_userled(int led, bool ledon)
+{
+  if ((unsigned int)led < BOARD_NLEDS)
+    {
+      esp_gpiowrite(g_ledcfg[led], ledon);
+    }
+}
+
+/****************************************************************************
+ * Name: board_userled_all
+ ****************************************************************************/
+
+void board_userled_all(uint32_t ledset)
+{
+  int i;
+
+  for (i = 0; i < BOARD_NLEDS; i++)
+    {
+      esp_gpiowrite(g_ledcfg[i], (ledset & (1 << i)) != 0);
+    }
+}