diff --git a/components/esp_pm/linker.lf b/components/esp_pm/linker.lf index 0b4c6636e3..381faa4b3b 100644 --- a/components/esp_pm/linker.lf +++ b/components/esp_pm/linker.lf @@ -47,13 +47,11 @@ entries: archive: libesp_timer.a entries: if PM_SLP_IRAM_OPT = y: + esp_timer_impl_common:esp_timer_impl_lock (noflash) + esp_timer_impl_common:esp_timer_impl_unlock (noflash) if ESP_TIMER_IMPL_TG0_LAC = y: - esp_timer_impl_lac:esp_timer_impl_lock (noflash) - esp_timer_impl_lac:esp_timer_impl_unlock (noflash) esp_timer_impl_lac:esp_timer_impl_advance (noflash) elif ESP_TIMER_IMPL_SYSTIMER = y: - esp_timer_impl_systimer:esp_timer_impl_lock (noflash) - esp_timer_impl_systimer:esp_timer_impl_unlock (noflash) esp_timer_impl_systimer:esp_timer_impl_advance (noflash) [mapping:newlib_pm] diff --git a/components/esp_timer/CMakeLists.txt b/components/esp_timer/CMakeLists.txt index 02a93c13dc..f2a1816840 100644 --- a/components/esp_timer/CMakeLists.txt +++ b/components/esp_timer/CMakeLists.txt @@ -2,7 +2,8 @@ idf_build_get_property(target IDF_TARGET) set(srcs "src/esp_timer.c" "src/ets_timer_legacy.c" - "src/system_time.c") + "src/system_time.c" + "src/esp_timer_impl_common.c") if(CONFIG_ESP_TIMER_IMPL_TG0_LAC) list(APPEND srcs "src/esp_timer_impl_lac.c") diff --git a/components/esp_timer/private_include/esp_timer_impl.h b/components/esp_timer/private_include/esp_timer_impl.h index d59a13eb13..3977d6719f 100644 --- a/components/esp_timer/private_include/esp_timer_impl.h +++ b/components/esp_timer/private_include/esp_timer_impl.h @@ -1,5 +1,5 @@ /* - * SPDX-FileCopyrightText: 2017-2021 Espressif Systems (Shanghai) CO LTD + * SPDX-FileCopyrightText: 2017-2023 Espressif Systems (Shanghai) CO LTD * * SPDX-License-Identifier: Apache-2.0 */ @@ -142,3 +142,12 @@ uint64_t esp_timer_impl_get_alarm_reg(void); */ void esp_timer_impl_init_system_time(void); #endif + +#if CONFIG_ESP_TIMER_SUPPORTS_ISR_DISPATCH_METHOD +/** + * @brief Set the next alarm if there is such an alarm in the cached array. + * + * @note Available only when CONFIG_ESP_TIMER_SUPPORTS_ISR_DISPATCH_METHOD is enabled. + */ +void esp_timer_impl_try_to_set_next_alarm(void); +#endif diff --git a/components/esp_timer/src/esp_timer.c b/components/esp_timer/src/esp_timer.c index 41d7fa34b2..29190f9f82 100644 --- a/components/esp_timer/src/esp_timer.c +++ b/components/esp_timer/src/esp_timer.c @@ -489,6 +489,7 @@ static void IRAM_ATTR timer_alarm_handler(void* arg) bool isr_timers_processed = false; #ifdef CONFIG_ESP_TIMER_SUPPORTS_ISR_DISPATCH_METHOD + esp_timer_impl_try_to_set_next_alarm(); // process timers with ISR dispatch method isr_timers_processed = timer_process_alarm(ESP_TIMER_ISR); xHigherPriorityTaskWoken = s_isr_dispatch_need_yield; diff --git a/components/esp_timer/src/esp_timer_impl_common.c b/components/esp_timer/src/esp_timer_impl_common.c new file mode 100644 index 0000000000..d0c86f685d --- /dev/null +++ b/components/esp_timer/src/esp_timer_impl_common.c @@ -0,0 +1,70 @@ +/* + * SPDX-FileCopyrightText: 2023 Espressif Systems (Shanghai) CO LTD + * + * SPDX-License-Identifier: Apache-2.0 + */ + +#include "esp_timer_impl.h" +#include "esp_timer.h" +#include "esp_err.h" +#include "esp_task.h" +#include "esp_attr.h" + +/* Spinlock used to protect access to the hardware registers. */ +portMUX_TYPE s_time_update_lock = portMUX_INITIALIZER_UNLOCKED; + +/* Alarm values to generate interrupt on match + * [0] - for ESP_TIMER_TASK alarms, + * [1] - for ESP_TIMER_ISR alarms. +*/ +uint64_t timestamp_id[2] = { UINT64_MAX, UINT64_MAX }; + +void esp_timer_impl_lock(void) +{ + portENTER_CRITICAL(&s_time_update_lock); +} + +void esp_timer_impl_unlock(void) +{ + portEXIT_CRITICAL(&s_time_update_lock); +} + +void esp_timer_private_lock(void) __attribute__((alias("esp_timer_impl_lock"))); +void esp_timer_private_unlock(void) __attribute__((alias("esp_timer_impl_unlock"))); + +void IRAM_ATTR esp_timer_impl_set_alarm(uint64_t timestamp) +{ + esp_timer_impl_set_alarm_id(timestamp, 0); +} + +#ifdef CONFIG_ESP_TIMER_SUPPORTS_ISR_DISPATCH_METHOD +void IRAM_ATTR esp_timer_impl_try_to_set_next_alarm(void) { + portENTER_CRITICAL_ISR(&s_time_update_lock); + unsigned now_alarm_idx; // ISR is called due to this current alarm + unsigned next_alarm_idx; // The following alarm after now_alarm_idx + if (timestamp_id[0] < timestamp_id[1]) { + now_alarm_idx = 0; + next_alarm_idx = 1; + } else { + now_alarm_idx = 1; + next_alarm_idx = 0; + } + + if (timestamp_id[next_alarm_idx] != UINT64_MAX) { + // The following alarm is valid and can be used. + // Remove the current alarm from consideration. + esp_timer_impl_set_alarm_id(UINT64_MAX, now_alarm_idx); + } else { + // There is no the following alarm. + // Remove the current alarm from consideration as well. + timestamp_id[now_alarm_idx] = UINT64_MAX; + } + portEXIT_CRITICAL_ISR(&s_time_update_lock); +} +#endif + +/* FIXME: This value is safe for 80MHz APB frequency, should be modified to depend on clock frequency. */ +uint64_t IRAM_ATTR esp_timer_impl_get_min_period_us(void) +{ + return 50; +} diff --git a/components/esp_timer/src/esp_timer_impl_lac.c b/components/esp_timer/src/esp_timer_impl_lac.c index c7f400e3ad..7a30833b22 100644 --- a/components/esp_timer/src/esp_timer_impl_lac.c +++ b/components/esp_timer/src/esp_timer_impl_lac.c @@ -92,20 +92,10 @@ static intr_handle_t s_timer_interrupt_handle; static intr_handler_t s_alarm_handler = NULL; /* Spinlock used to protect access to the hardware registers. */ -portMUX_TYPE s_time_update_lock = portMUX_INITIALIZER_UNLOCKED; +extern portMUX_TYPE s_time_update_lock; /* Alarm values to generate interrupt on match */ -static uint64_t timestamp_id[2] = { UINT64_MAX, UINT64_MAX }; - -void esp_timer_impl_lock(void) -{ - portENTER_CRITICAL(&s_time_update_lock); -} - -void esp_timer_impl_unlock(void) -{ - portEXIT_CRITICAL(&s_time_update_lock); -} +extern uint64_t timestamp_id[2]; uint64_t IRAM_ATTR esp_timer_impl_get_counter_reg(void) { @@ -175,46 +165,11 @@ void IRAM_ATTR esp_timer_impl_set_alarm_id(uint64_t timestamp, unsigned alarm_id portEXIT_CRITICAL_SAFE(&s_time_update_lock); } -void IRAM_ATTR esp_timer_impl_set_alarm(uint64_t timestamp) -{ - esp_timer_impl_set_alarm_id(timestamp, 0); -} - -#ifdef CONFIG_ESP_TIMER_SUPPORTS_ISR_DISPATCH_METHOD -static void IRAM_ATTR try_to_set_next_alarm(void) { - portENTER_CRITICAL_ISR(&s_time_update_lock); - unsigned now_alarm_idx; // ISR is called due to this current alarm - unsigned next_alarm_idx; // The following alarm after now_alarm_idx - if (timestamp_id[0] < timestamp_id[1]) { - now_alarm_idx = 0; - next_alarm_idx = 1; - } else { - now_alarm_idx = 1; - next_alarm_idx = 0; - } - - if (timestamp_id[next_alarm_idx] != UINT64_MAX) { - // The following alarm is valid and can be used. - // Remove the current alarm from consideration. - esp_timer_impl_set_alarm_id(UINT64_MAX, now_alarm_idx); - } else { - // There is no the following alarm. - // Remove the current alarm from consideration as well. - timestamp_id[now_alarm_idx] = UINT64_MAX; - } - portEXIT_CRITICAL_ISR(&s_time_update_lock); -} -#else -#define try_to_set_next_alarm() -#endif - static void IRAM_ATTR timer_alarm_isr(void *arg) { /* Clear interrupt status */ REG_WRITE(INT_CLR_REG, TIMG_LACT_INT_CLR); - try_to_set_next_alarm(); - /* Call the upper layer handler */ (*s_alarm_handler)(arg); } @@ -306,12 +261,6 @@ void esp_timer_impl_deinit(void) s_timer_interrupt_handle = NULL; } -/* FIXME: This value is safe for 80MHz APB frequency, should be modified to depend on clock frequency. */ -uint64_t IRAM_ATTR esp_timer_impl_get_min_period_us(void) -{ - return 50; -} - uint64_t esp_timer_impl_get_alarm_reg(void) { portENTER_CRITICAL_SAFE(&s_time_update_lock); @@ -326,5 +275,3 @@ uint64_t esp_timer_impl_get_alarm_reg(void) void esp_timer_private_update_apb_freq(uint32_t apb_ticks_per_us) __attribute__((alias("esp_timer_impl_update_apb_freq"))); void esp_timer_private_set(uint64_t new_us) __attribute__((alias("esp_timer_impl_set"))); void esp_timer_private_advance(int64_t time_diff_us) __attribute__((alias("esp_timer_impl_advance"))); -void esp_timer_private_lock(void) __attribute__((alias("esp_timer_impl_lock"))); -void esp_timer_private_unlock(void) __attribute__((alias("esp_timer_impl_unlock"))); diff --git a/components/esp_timer/src/esp_timer_impl_systimer.c b/components/esp_timer/src/esp_timer_impl_systimer.c index 4a06bf9255..07adbe4dee 100644 --- a/components/esp_timer/src/esp_timer_impl_systimer.c +++ b/components/esp_timer/src/esp_timer_impl_systimer.c @@ -48,20 +48,10 @@ static intr_handler_t s_alarm_handler = NULL; static systimer_hal_context_t systimer_hal; /* Spinlock used to protect access to the hardware registers. */ -static portMUX_TYPE s_time_update_lock = portMUX_INITIALIZER_UNLOCKED; +extern portMUX_TYPE s_time_update_lock; /* Alarm values to generate interrupt on match */ -static uint64_t timestamp_id[2] = { UINT64_MAX, UINT64_MAX }; - -void esp_timer_impl_lock(void) -{ - portENTER_CRITICAL(&s_time_update_lock); -} - -void esp_timer_impl_unlock(void) -{ - portEXIT_CRITICAL(&s_time_update_lock); -} +extern uint64_t timestamp_id[2]; uint64_t IRAM_ATTR esp_timer_impl_get_counter_reg(void) { @@ -89,46 +79,10 @@ void IRAM_ATTR esp_timer_impl_set_alarm_id(uint64_t timestamp, unsigned alarm_id portEXIT_CRITICAL_SAFE(&s_time_update_lock); } -void IRAM_ATTR esp_timer_impl_set_alarm(uint64_t timestamp) -{ - esp_timer_impl_set_alarm_id(timestamp, 0); -} - -#ifdef CONFIG_ESP_TIMER_SUPPORTS_ISR_DISPATCH_METHOD -static void IRAM_ATTR try_to_set_next_alarm(void) { - portENTER_CRITICAL_ISR(&s_time_update_lock); - unsigned now_alarm_idx; // ISR is called due to this current alarm - unsigned next_alarm_idx; // The following alarm after now_alarm_idx - if (timestamp_id[0] < timestamp_id[1]) { - now_alarm_idx = 0; - next_alarm_idx = 1; - } else { - now_alarm_idx = 1; - next_alarm_idx = 0; - } - - if (timestamp_id[next_alarm_idx] != UINT64_MAX) { - // The following alarm is valid and can be used. - // Remove the current alarm from consideration. - esp_timer_impl_set_alarm_id(UINT64_MAX, now_alarm_idx); - } else { - // There is no the following alarm. - // Remove the current alarm from consideration as well. - timestamp_id[now_alarm_idx] = UINT64_MAX; - } - portEXIT_CRITICAL_ISR(&s_time_update_lock); -} -#else -#define try_to_set_next_alarm() -#endif - static void IRAM_ATTR timer_alarm_isr(void *arg) { // clear the interrupt systimer_ll_clear_alarm_int(systimer_hal.dev, SYSTIMER_LL_ALARM_CLOCK); - - try_to_set_next_alarm(); - /* Call the upper layer handler */ (*s_alarm_handler)(arg); } @@ -234,11 +188,6 @@ void esp_timer_impl_deinit(void) s_alarm_handler = NULL; } -uint64_t IRAM_ATTR esp_timer_impl_get_min_period_us(void) -{ - return 50; -} - uint64_t esp_timer_impl_get_alarm_reg(void) { portENTER_CRITICAL_SAFE(&s_time_update_lock); @@ -250,5 +199,3 @@ uint64_t esp_timer_impl_get_alarm_reg(void) void esp_timer_private_update_apb_freq(uint32_t apb_ticks_per_us) __attribute__((alias("esp_timer_impl_update_apb_freq"))); void esp_timer_private_set(uint64_t new_us) __attribute__((alias("esp_timer_impl_set"))); void esp_timer_private_advance(int64_t time_diff_us) __attribute__((alias("esp_timer_impl_advance"))); -void esp_timer_private_lock(void) __attribute__((alias("esp_timer_impl_lock"))); -void esp_timer_private_unlock(void) __attribute__((alias("esp_timer_impl_unlock")));