diff --git a/components/driver/i2s.c b/components/driver/i2s.c index 20ee16a93e..a8a0cb87eb 100644 --- a/components/driver/i2s.c +++ b/components/driver/i2s.c @@ -107,15 +107,6 @@ static int _i2s_adc_channel = -1; static i2s_dma_t *i2s_create_dma_queue(i2s_port_t i2s_num, int dma_buf_count, int dma_buf_len); static esp_err_t i2s_destroy_dma_queue(i2s_port_t i2s_num, i2s_dma_t *dma); -static esp_err_t i2s_reset_fifo(i2s_port_t i2s_num) -{ - I2S_CHECK((i2s_num < I2S_NUM_MAX), "i2s_num error", ESP_ERR_INVALID_ARG); - I2S_ENTER_CRITICAL(); - i2s_hal_reset_fifo(&(p_i2s_obj[i2s_num]->hal)); - I2S_EXIT_CRITICAL(); - return ESP_OK; -} - static inline void gpio_matrix_out_check(uint32_t gpio, uint32_t signal_idx, bool out_inv, bool oen_inv) { //if pin = -1, do not need to configure @@ -469,6 +460,14 @@ esp_err_t i2s_set_clk(i2s_port_t i2s_num, uint32_t rate, i2s_bits_per_sample_t b ESP_LOGI(I2S_TAG, "PLL_D2: Req RATE: %d, real rate: %0.3f, BITS: %u, CLKM: %u, BCK: %u, MCLK: %0.3f, SCLK: %f, diva: %d, divb: %d", rate, real_rate, bits, clkmInteger, bck, (double)I2S_BASE_CLK / mclk, real_rate*bits*channel, 64, clkmDecimals); } + if (p_i2s_obj[i2s_num]->mode & I2S_MODE_TX) { + p_i2s_obj[i2s_num]->tx->curr_ptr = NULL; + p_i2s_obj[i2s_num]->tx->rw_pos = 0; + } + if (p_i2s_obj[i2s_num]->mode & I2S_MODE_RX) { + p_i2s_obj[i2s_num]->rx->curr_ptr = NULL; + p_i2s_obj[i2s_num]->rx->rw_pos = 0; + } i2s_hal_set_tx_bits_mod(&(p_i2s_obj[i2s_num]->hal), bits); i2s_hal_set_rx_bits_mod(&(p_i2s_obj[i2s_num]->hal), bits); @@ -643,9 +642,7 @@ static i2s_dma_t *i2s_create_dma_queue(i2s_port_t i2s_num, int dma_buf_count, in } dma->queue = xQueueCreate(dma_buf_count - 1, sizeof(char*)); dma->mux = xSemaphoreCreateMutex(); - dma->rw_pos = 0; dma->buf_size = dma_buf_len * sample_size; - dma->curr_ptr = NULL; ESP_LOGI(I2S_TAG, "DMA Malloc info, datalen=blocksize=%d, dma_buf_count=%d", dma_buf_len * sample_size, dma_buf_count); return dma; } @@ -655,8 +652,6 @@ esp_err_t i2s_start(i2s_port_t i2s_num) I2S_CHECK((i2s_num < I2S_NUM_MAX), "i2s_num error", ESP_ERR_INVALID_ARG); //start DMA link I2S_ENTER_CRITICAL(); - i2s_reset_fifo(i2s_num); - i2s_hal_reset(&(p_i2s_obj[i2s_num]->hal)); esp_intr_disable(p_i2s_obj[i2s_num]->i2s_isr_handle); @@ -865,7 +860,6 @@ static esp_err_t i2s_param_config(i2s_port_t i2s_num, const i2s_config_t *i2s_co } #endif // configure I2S data port interface. - i2s_reset_fifo(i2s_num); i2s_hal_config_param(&(p_i2s_obj[i2s_num]->hal), i2s_config); if ((p_i2s_obj[i2s_num]->mode & I2S_MODE_RX) && (p_i2s_obj[i2s_num]->mode & I2S_MODE_TX)) { i2s_hal_enable_sig_loopback(&(p_i2s_obj[i2s_num]->hal)); diff --git a/components/soc/include/hal/i2s_hal.h b/components/soc/include/hal/i2s_hal.h index aa03922616..b3118fe89b 100644 --- a/components/soc/include/hal/i2s_hal.h +++ b/components/soc/include/hal/i2s_hal.h @@ -40,13 +40,6 @@ typedef struct { uint32_t version; } i2s_hal_context_t; -/** - * @brief Reset I2S fifo - * - * @param hal Context of the HAL layer - */ -void i2s_hal_reset_fifo(i2s_hal_context_t *hal); - /** * @brief Get I2S interrupt status * @@ -214,7 +207,7 @@ void i2s_hal_set_tx_bits_mod(i2s_hal_context_t *hal, i2s_bits_per_sample_t bits) void i2s_hal_set_rx_bits_mod(i2s_hal_context_t *hal, i2s_bits_per_sample_t bits); /** - * @brief Reset I2S tx + * @brief Reset I2S TX & RX module, including DMA and FIFO * * @param hal Context of the HAL layer */ diff --git a/components/soc/src/hal/i2s_hal.c b/components/soc/src/hal/i2s_hal.c index 25da23aec2..cf388425aa 100644 --- a/components/soc/src/hal/i2s_hal.c +++ b/components/soc/src/hal/i2s_hal.c @@ -17,12 +17,6 @@ #include "soc/soc.h" #include "hal/i2s_hal.h" -void i2s_hal_reset_fifo(i2s_hal_context_t *hal) -{ - i2s_ll_reset_rx_fifo(hal->dev); - i2s_ll_reset_tx_fifo(hal->dev); -} - void i2s_hal_set_tx_mode(i2s_hal_context_t *hal, i2s_channel_t ch, i2s_bits_per_sample_t bits) { if (bits <= I2S_BITS_PER_SAMPLE_16BIT) { @@ -84,10 +78,13 @@ void i2s_hal_set_rx_bits_mod(i2s_hal_context_t *hal, i2s_bits_per_sample_t bits) void i2s_hal_reset(i2s_hal_context_t *hal) { - i2s_ll_reset_dma_in(hal->dev); - i2s_ll_reset_dma_out(hal->dev); + // Reset I2S TX/RX module first, and then, reset DMA and FIFO. i2s_ll_reset_tx(hal->dev); i2s_ll_reset_rx(hal->dev); + i2s_ll_reset_dma_in(hal->dev); + i2s_ll_reset_dma_out(hal->dev); + i2s_ll_reset_rx_fifo(hal->dev); + i2s_ll_reset_tx_fifo(hal->dev); } void i2s_hal_start_tx(i2s_hal_context_t *hal)