From c5baceb4b092b86fbb082fe73c0013d9f99de916 Mon Sep 17 00:00:00 2001 From: Gunar Schorcht Date: Mon, 12 May 2025 11:06:19 +0200 Subject: [PATCH] boards/esp32-wrover-kit: fix definition of LED2_PIN - GPIO2 (LED1_PIN) is used as SPI signal, not only if `sdcard_spi` is enabled. herefore, it must not be defined as LED pin if `periph_spi` is used. - GIO4 (LED2_PIN) is used by the SDMMC Host Controller as SD Card DAT1 signal and must not be defined as LED pin in that case. --- boards/esp32-wrover-kit/include/board.h | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/boards/esp32-wrover-kit/include/board.h b/boards/esp32-wrover-kit/include/board.h index f41e76e1a7..2bce290922 100644 --- a/boards/esp32-wrover-kit/include/board.h +++ b/boards/esp32-wrover-kit/include/board.h @@ -57,13 +57,14 @@ #define LED0_ACTIVE (1) /**< LED0 is high active */ #endif -#if !MODULE_SDCARD_SPI || DOXYGEN -#define LED1_PIN GPIO2 /**< LED1 is available when the SD-Card is not used */ +#if !MODULE_PERIPH_SPI || DOXYGEN +#define LED1_PIN GPIO2 /**< LED1 is available when the SPI bus is not used */ #define LED1_ACTIVE (1) /**< LED1 is high active */ #endif -#if !MODULE_ESP32_WROVER_KIT_CAMERA || DOXYGEN -#define LED2_PIN GPIO4 /**< LED2 is available when the camera is not plugged in */ +#if (!MODULE_ESP32_WROVER_KIT_CAMERA && !MODULE_PERIPH_SDMMC) || DOXYGEN +#define LED2_PIN GPIO4 /**< LED2 is available when the camera is not plugged in + and SDMMD is not used */ #define LED2_ACTIVE (1) /**< LED2 is high active */ #endif