Fixed bug in ledc_set_fade_with_step where returned while holding critical section.

Merges PR #515 https://github.com/espressif/esp-idf/pull/515

diff --git a/components/driver/ledc.c b/components/driver/ledc.c
index 7da353869f5bab1475d9442919950edfcbb579d0..239f8f33118de1cd5353ebed1071a33c31d3e46a 100644 (file)
--- a/components/driver/ledc.c
+++ b/components/driver/ledc.c
@@ -541,6 +541,7 @@ esp_err_t ledc_set_fade_with_step(ledc_mode_t speed_mode, ledc_channel_t channel
     int duty_cur = LEDC.channel_group[speed_mode].channel[channel].duty_rd.duty_read >> LEDC_DUTY_DECIMAL_BIT_NUM;
     int duty_delta = target_duty > duty_cur ? target_duty - duty_cur : duty_cur - target_duty;
     if (duty_delta == 0) {
+        portEXIT_CRITICAL(&ledc_spinlock);
         return ESP_OK;
     }
     s_ledc_fade_rec[speed_mode][channel]->speed_mode = speed_mode;