Mirrors/esp-idf
1
0
Fork 0
mirror of https://github.com/espressif/esp-idf.git synced 2026-09-22 13:01:16 +03:00
Code Issues Packages Projects Releases Wiki Activity

fix(mcpwm): stabilize mcpwm_timer_various_clk_src test

Browse Source 
Use count-up mode and account for counter wrap when measuring timer
resolution. Extend the stop wait to cover the full timer period.
This commit is contained in:
Steven Yang
2026-07-30 17:07:23 +08:00
parent 4065b2ed8f
commit f54adcfa93
2 changed files with 10 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
components/esp_driver_mcpwm/test_apps/mcpwm/main/CMakeLists.txt
View File

@@ -24,5 +24,6 @@ endif()
# In order for the cases defined by `TEST_CASE` to be linked into the final elf,
# the component can be registered as WHOLE_ARCHIVE
idf_component_register(SRCS ${srcs}
PRIV_REQUIRES unity esp_driver_mcpwm esp_driver_gpio esp_driver_gptimer esp_psram esp_timer esp_rom
PRIV_REQUIRES unity esp_driver_mcpwm esp_driver_gpio esp_driver_gptimer esp_psram
esp_timer esp_rom
WHOLE_ARCHIVE)

11
components/esp_driver_mcpwm/test_apps/mcpwm/main/test_mcpwm_timer.c
View File

@@ -103,7 +103,7 @@ TEST_CASE("mcpwm_timer_various_clk_src", "[mcpwm]")
.clk_src = clk_src,
.resolution_hz = 1000000, // 1MHz
.period_ticks = 12000,
.count_mode = MCPWM_TIMER_COUNT_MODE_UP_DOWN,
.count_mode = MCPWM_TIMER_COUNT_MODE_UP,
};
printf("create MCPWM timers with clock source: %d\r\n", clk_src);
@@ -136,7 +136,12 @@ TEST_CASE("mcpwm_timer_various_clk_src", "[mcpwm]")
TEST_ESP_OK(mcpwm_timer_get_phase(timers[i], &end_count, &direction));
int64_t elapsed_time_us = esp_timer_get_time() - start_time_us;
uint32_t expected_ticks = (uint64_t)config.resolution_hz * elapsed_time_us / 1000000;
uint32_t actual_ticks = end_count - start_count;
uint32_t actual_ticks;
if (end_count >= start_count) {
actual_ticks = end_count - start_count;
} else {
actual_ticks = config.period_ticks - start_count + end_count;
}
uint32_t tolerance_ticks = expected_ticks * tolerance_percent / 100;
TEST_ASSERT_EQUAL(MCPWM_TIMER_DIRECTION_UP, direction);
@@ -144,7 +149,7 @@ TEST_CASE("mcpwm_timer_various_clk_src", "[mcpwm]")
// make sure the timer has stopped
TEST_ESP_OK(mcpwm_timer_start_stop(timers[i], MCPWM_TIMER_STOP_EMPTY));
vTaskDelay(pdMS_TO_TICKS(10));
vTaskDelay(pdMS_TO_TICKS(20));
check_mcpwm_timer_phase(&timers[i], 1, 0, MCPWM_TIMER_DIRECTION_UP);
}