From 9ddde5df8942400294eec424d87c78966a3f531d Mon Sep 17 00:00:00 2001 From: Marius Vikhammer Date: Wed, 22 Apr 2026 15:11:01 +0800 Subject: [PATCH] fix(esp_timer): removed leftover TODOs --- components/esp_hal_systimer/esp32s31/include/hal/systimer_ll.h | 2 -- components/esp_hw_support/port/esp32s31/systimer.c | 2 -- components/soc/esp32s31/include/soc/soc_caps.h | 2 +- 3 files changed, 1 insertion(+), 5 deletions(-) diff --git a/components/esp_hal_systimer/esp32s31/include/hal/systimer_ll.h b/components/esp_hal_systimer/esp32s31/include/hal/systimer_ll.h index 3ce56961c0..44a7959883 100644 --- a/components/esp_hal_systimer/esp32s31/include/hal/systimer_ll.h +++ b/components/esp_hal_systimer/esp32s31/include/hal/systimer_ll.h @@ -19,8 +19,6 @@ extern "C" { // All these functions get invoked either from ISR or HAL that linked to IRAM. // Always inline these functions even no gcc optimization is applied. -// TODO: [ESP32S31] IDF-14693 - /******************* SYSTIMER LL CAPS *************************/ #define SYSTIMER_LL_COUNTER_NUM 2 // Number of counter units #define SYSTIMER_LL_ALARM_NUM 3 // Number of alarm units diff --git a/components/esp_hw_support/port/esp32s31/systimer.c b/components/esp_hw_support/port/esp32s31/systimer.c index fd6987f3df..a46839a0dc 100644 --- a/components/esp_hw_support/port/esp32s31/systimer.c +++ b/components/esp_hw_support/port/esp32s31/systimer.c @@ -6,8 +6,6 @@ #include "esp_private/systimer.h" -// TODO: [ESP32S31] IDF-14693 - /** * @brief systimer's clock source is fixed to XTAL (40MHz), and has a fixed fractional divider (2.5). * So the resolution of the systimer is 40MHz/2.5 = 16MHz. diff --git a/components/soc/esp32s31/include/soc/soc_caps.h b/components/soc/esp32s31/include/soc/soc_caps.h index 7093186646..d5369231fd 100644 --- a/components/soc/esp32s31/include/soc/soc_caps.h +++ b/components/soc/esp32s31/include/soc/soc_caps.h @@ -64,7 +64,7 @@ #define SOC_LEDC_SUPPORTED 1 // #define SOC_ISP_SUPPORTED 1 // TODO: [ESP32S31] IDF-14769 #define SOC_I2C_SUPPORTED 1 -#define SOC_SYSTIMER_SUPPORTED 1 // TODO: [ESP32S31] IDF-14693 +#define SOC_SYSTIMER_SUPPORTED 1 #define SOC_AES_SUPPORTED 1 #define SOC_MPI_SUPPORTED 1 #define SOC_SHA_SUPPORTED 1