espressif/esp-idf
1
0
Fork 1
mirror of https://github.com/espressif/esp-idf.git synced 2025-07-30 18:57:19 +02:00
Code Issues Packages Projects Releases Wiki Activity

Merge branch 'bugfix/ledc_fade_stop_race_condition_v5.3' into 'release/v5.3'

Browse Source 
fix(ledc): fix race condition in ledc_fade_stop causing assert failure (v5.3)

See merge request espressif/esp-idf!38081
This commit is contained in:
morris
2025-04-15 10:32:17 +08:00
parent ca42f3e1a2 7b4ac061db
commit 949b0451d7
1 changed files with 9 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

15
components/esp_driver_ledc/src/ledc.c
View File

@ -1227,15 +1227,17 @@ esp_err_t ledc_fade_stop(ledc_mode_t speed_mode, ledc_channel_t channel)
LEDC_CHECK(p_ledc_obj[speed_mode] != NULL, LEDC_NOT_INIT, ESP_ERR_INVALID_STATE); LEDC_CHECK(p_ledc_obj[speed_mode] != NULL, LEDC_NOT_INIT, ESP_ERR_INVALID_STATE);
LEDC_CHECK(ledc_fade_channel_init_check(speed_mode, channel) == ESP_OK, LEDC_FADE_INIT_ERROR_STR, ESP_FAIL); LEDC_CHECK(ledc_fade_channel_init_check(speed_mode, channel) == ESP_OK, LEDC_FADE_INIT_ERROR_STR, ESP_FAIL);
ledc_fade_t *fade = s_ledc_fade_rec[speed_mode][channel]; ledc_fade_t *fade = s_ledc_fade_rec[speed_mode][channel];
ledc_fade_fsm_t state = fade->fsm; bool skip = false;
bool wait_for_idle = false; bool wait_for_idle = false;
portENTER_CRITICAL(&ledc_spinlock);
ledc_fade_fsm_t state = fade->fsm;
assert(state != LEDC_FSM_KILLED_PENDING); assert(state != LEDC_FSM_KILLED_PENDING);
if (state == LEDC_FSM_IDLE) { if (state == LEDC_FSM_IDLE) {
// if there is no fade going on, do nothing // if there is no fade going on, do nothing
return ESP_OK; skip = true;
goto exit;
} }
// Fade state is either HW_FADE or ISR_CAL (there is a fade in process) // Fade state is either HW_FADE or ISR_CAL (there is a fade in process)
portENTER_CRITICAL(&ledc_spinlock);
// Disable ledc channel interrupt first // Disable ledc channel interrupt first
ledc_enable_intr_type(speed_mode, channel, LEDC_INTR_DISABLE); ledc_enable_intr_type(speed_mode, channel, LEDC_INTR_DISABLE);
// Config duty to the duty cycle at this moment // Config duty to the duty cycle at this moment
@ -1250,21 +1252,22 @@ esp_err_t ledc_fade_stop(ledc_mode_t speed_mode, ledc_channel_t channel)
0 //uint32_t duty_scale 0 //uint32_t duty_scale
); );
_ledc_update_duty(speed_mode, channel); _ledc_update_duty(speed_mode, channel);
state = fade->fsm;
assert(state != LEDC_FSM_IDLE && state != LEDC_FSM_KILLED_PENDING);
if (state == LEDC_FSM_HW_FADE) { if (state == LEDC_FSM_HW_FADE) {
fade->fsm = LEDC_FSM_IDLE; fade->fsm = LEDC_FSM_IDLE;
} else if (state == LEDC_FSM_ISR_CAL) { } else if (state == LEDC_FSM_ISR_CAL) {
fade->fsm = LEDC_FSM_KILLED_PENDING; fade->fsm = LEDC_FSM_KILLED_PENDING;
wait_for_idle = true; wait_for_idle = true;
} }
exit:
portEXIT_CRITICAL(&ledc_spinlock); portEXIT_CRITICAL(&ledc_spinlock);
if (wait_for_idle) { if (wait_for_idle) {
// Wait for ISR return, which gives the semaphore and switches state to IDLE // Wait for ISR return, which gives the semaphore and switches state to IDLE
_ledc_fade_hw_acquire(speed_mode, channel); _ledc_fade_hw_acquire(speed_mode, channel);
assert(fade->fsm == LEDC_FSM_IDLE); assert(fade->fsm == LEDC_FSM_IDLE);
} }
_ledc_fade_hw_release(speed_mode, channel); if (!skip) {
_ledc_fade_hw_release(speed_mode, channel);
}
return ESP_OK; return ESP_OK;
} }
#endif #endif