mirror of
				https://github.com/espressif/esp-idf.git
				synced 2025-10-31 04:59:55 +00:00 
			
		
		
		
	fix(i2s): add check for the tdm frame bits num
This commit is contained in:
		| @@ -100,7 +100,10 @@ static esp_err_t i2s_tdm_set_slot(i2s_chan_handle_t handle, const i2s_tdm_slot_c | |||||||
|     handle->total_slot = slot_cfg->total_slot < max_slot_num ? max_slot_num : slot_cfg->total_slot; |     handle->total_slot = slot_cfg->total_slot < max_slot_num ? max_slot_num : slot_cfg->total_slot; | ||||||
|     // At least two slots in a frame if not using PCM short format |     // At least two slots in a frame if not using PCM short format | ||||||
|     handle->total_slot = ((handle->total_slot < 2) && (slot_cfg->ws_width != 1)) ? 2 : handle->total_slot; |     handle->total_slot = ((handle->total_slot < 2) && (slot_cfg->ws_width != 1)) ? 2 : handle->total_slot; | ||||||
|  |     uint32_t slot_bits = slot_cfg->slot_bit_width == I2S_SLOT_BIT_WIDTH_AUTO ? slot_cfg->data_bit_width : slot_cfg->slot_bit_width; | ||||||
|  |     ESP_RETURN_ON_FALSE(handle->total_slot * slot_bits <= I2S_LL_SLOT_FRAME_BIT_MAX, ESP_ERR_INVALID_ARG, TAG, | ||||||
|  |                         "total slots(%"PRIu32") * slot_bit_width(%"PRIu32") exceeds the maximum %d", | ||||||
|  |                         handle->total_slot, slot_bits, (int)I2S_LL_SLOT_FRAME_BIT_MAX); | ||||||
|     uint32_t buf_size = i2s_get_buf_size(handle, slot_cfg->data_bit_width, handle->dma.frame_num); |     uint32_t buf_size = i2s_get_buf_size(handle, slot_cfg->data_bit_width, handle->dma.frame_num); | ||||||
|     /* The DMA buffer need to re-allocate if the buffer size changed */ |     /* The DMA buffer need to re-allocate if the buffer size changed */ | ||||||
|     if (handle->dma.buf_size != buf_size) { |     if (handle->dma.buf_size != buf_size) { | ||||||
|   | |||||||
| @@ -31,6 +31,7 @@ extern "C" { | |||||||
|  |  | ||||||
| #define I2S_LL_MCLK_DIVIDER_BIT_WIDTH  (9) | #define I2S_LL_MCLK_DIVIDER_BIT_WIDTH  (9) | ||||||
| #define I2S_LL_MCLK_DIVIDER_MAX        ((1 << I2S_LL_MCLK_DIVIDER_BIT_WIDTH) - 1) | #define I2S_LL_MCLK_DIVIDER_MAX        ((1 << I2S_LL_MCLK_DIVIDER_BIT_WIDTH) - 1) | ||||||
|  | #define I2S_LL_SLOT_FRAME_BIT_MAX  128 // Up-to 128 bits in one frame, determined by MAX(half_sample_bits) * 2 | ||||||
|  |  | ||||||
| #define I2S_LL_PLL_F160M_CLK_FREQ      (160 * 1000000) // PLL_F160M_CLK: 160MHz | #define I2S_LL_PLL_F160M_CLK_FREQ      (160 * 1000000) // PLL_F160M_CLK: 160MHz | ||||||
| #define I2S_LL_DEFAULT_PLL_CLK_FREQ     I2S_LL_PLL_F160M_CLK_FREQ    // The default PLL clock frequency while using I2S_CLK_SRC_DEFAULT | #define I2S_LL_DEFAULT_PLL_CLK_FREQ     I2S_LL_PLL_F160M_CLK_FREQ    // The default PLL clock frequency while using I2S_CLK_SRC_DEFAULT | ||||||
|   | |||||||
| @@ -32,6 +32,7 @@ extern "C" { | |||||||
|  |  | ||||||
| #define I2S_LL_MCLK_DIVIDER_BIT_WIDTH  (9) | #define I2S_LL_MCLK_DIVIDER_BIT_WIDTH  (9) | ||||||
| #define I2S_LL_MCLK_DIVIDER_MAX        ((1 << I2S_LL_MCLK_DIVIDER_BIT_WIDTH) - 1) | #define I2S_LL_MCLK_DIVIDER_MAX        ((1 << I2S_LL_MCLK_DIVIDER_BIT_WIDTH) - 1) | ||||||
|  | #define I2S_LL_SLOT_FRAME_BIT_MAX  128 // Up-to 128 bits in one frame, determined by MAX(half_sample_bits) * 2 | ||||||
|  |  | ||||||
| #define I2S_LL_PLL_F160M_CLK_FREQ      (160 * 1000000) // PLL_F160M_CLK: 160MHz | #define I2S_LL_PLL_F160M_CLK_FREQ      (160 * 1000000) // PLL_F160M_CLK: 160MHz | ||||||
| #define I2S_LL_DEFAULT_PLL_CLK_FREQ     I2S_LL_PLL_F160M_CLK_FREQ    // The default PLL clock frequency while using I2S_CLK_SRC_DEFAULT | #define I2S_LL_DEFAULT_PLL_CLK_FREQ     I2S_LL_PLL_F160M_CLK_FREQ    // The default PLL clock frequency while using I2S_CLK_SRC_DEFAULT | ||||||
|   | |||||||
| @@ -32,6 +32,7 @@ extern "C" { | |||||||
|  |  | ||||||
| #define I2S_LL_MCLK_DIVIDER_BIT_WIDTH  (9) | #define I2S_LL_MCLK_DIVIDER_BIT_WIDTH  (9) | ||||||
| #define I2S_LL_MCLK_DIVIDER_MAX        ((1 << I2S_LL_MCLK_DIVIDER_BIT_WIDTH) - 1) | #define I2S_LL_MCLK_DIVIDER_MAX        ((1 << I2S_LL_MCLK_DIVIDER_BIT_WIDTH) - 1) | ||||||
|  | #define I2S_LL_SLOT_FRAME_BIT_MAX  512 // Up-to 512 bits in one frame, determined by MAX(half_sample_bits) * 2 | ||||||
|  |  | ||||||
| #define I2S_LL_PLL_F96M_CLK_FREQ      (96 * 1000000) // PLL_F96M_CLK: 96MHz | #define I2S_LL_PLL_F96M_CLK_FREQ      (96 * 1000000) // PLL_F96M_CLK: 96MHz | ||||||
| #define I2S_LL_PLL_F64M_CLK_FREQ      (64 * 1000000) // PLL_F64M_CLK: 64MHz | #define I2S_LL_PLL_F64M_CLK_FREQ      (64 * 1000000) // PLL_F64M_CLK: 64MHz | ||||||
|   | |||||||
| @@ -32,6 +32,7 @@ extern "C" { | |||||||
|  |  | ||||||
| #define I2S_LL_MCLK_DIVIDER_BIT_WIDTH  (9) | #define I2S_LL_MCLK_DIVIDER_BIT_WIDTH  (9) | ||||||
| #define I2S_LL_MCLK_DIVIDER_MAX        ((1 << I2S_LL_MCLK_DIVIDER_BIT_WIDTH) - 1) | #define I2S_LL_MCLK_DIVIDER_MAX        ((1 << I2S_LL_MCLK_DIVIDER_BIT_WIDTH) - 1) | ||||||
|  | #define I2S_LL_SLOT_FRAME_BIT_MAX  128 // Up-to 128 bits in one frame, determined by MAX(half_sample_bits) * 2 | ||||||
|  |  | ||||||
| #define I2S_LL_PLL_F160M_CLK_FREQ      (160 * 1000000) // PLL_F160M_CLK: 160MHz | #define I2S_LL_PLL_F160M_CLK_FREQ      (160 * 1000000) // PLL_F160M_CLK: 160MHz | ||||||
| #define I2S_LL_DEFAULT_PLL_CLK_FREQ     I2S_LL_PLL_F160M_CLK_FREQ    // The default PLL clock frequency while using I2S_CLK_SRC_DEFAULT | #define I2S_LL_DEFAULT_PLL_CLK_FREQ     I2S_LL_PLL_F160M_CLK_FREQ    // The default PLL clock frequency while using I2S_CLK_SRC_DEFAULT | ||||||
|   | |||||||
		Reference in New Issue
	
	Block a user
	 laokaiyao
					laokaiyao