From a4ecb6b15c6ec16e13b14062a47f05589ce01c42 Mon Sep 17 00:00:00 2001 From: "C.S.M" Date: Thu, 27 Feb 2025 14:13:41 +0800 Subject: [PATCH 1/5] fix(i2c_master): Fix i2c master race condition issue, Closes https://github.com/espressif/esp-idf/issues/15444 --- components/esp_driver_i2c/i2c_master.c | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/components/esp_driver_i2c/i2c_master.c b/components/esp_driver_i2c/i2c_master.c index 18ce5c4c38..6d4821d6b5 100644 --- a/components/esp_driver_i2c/i2c_master.c +++ b/components/esp_driver_i2c/i2c_master.c @@ -331,7 +331,7 @@ static bool s_i2c_read_command(i2c_master_bus_handle_t i2c_master, i2c_operation i2c_ll_master_write_cmd_reg(hal->dev, hw_cmd, i2c_master->cmd_idx); i2c_ll_master_write_cmd_reg(hal->dev, hw_end_cmd, i2c_master->cmd_idx + 1); portEXIT_CRITICAL_SAFE(&handle->spinlock); - i2c_master->status = I2C_STATUS_READ; + atomic_store(&i2c_master->status, I2C_STATUS_READ); portENTER_CRITICAL_SAFE(&handle->spinlock); if (i2c_master->async_trans == false) { i2c_hal_master_trans_start(hal); @@ -472,15 +472,15 @@ static void s_i2c_send_commands(i2c_master_bus_handle_t i2c_master, TickType_t t while (i2c_master->i2c_trans.cmd_count) { if (xSemaphoreTake(i2c_master->cmd_semphr, ticks_to_wait) != pdTRUE) { // Software timeout, clear the command link and finish this transaction. + atomic_store(&i2c_master->status, I2C_STATUS_TIMEOUT); i2c_master->cmd_idx = 0; i2c_master->trans_idx = 0; - atomic_store(&i2c_master->status, I2C_STATUS_TIMEOUT); ESP_LOGE(TAG, "I2C software timeout"); xSemaphoreGive(i2c_master->cmd_semphr); return; } - if (i2c_master->status == I2C_STATUS_TIMEOUT) { + if (atomic_load(&i2c_master->status) == I2C_STATUS_TIMEOUT) { s_i2c_hw_fsm_reset(i2c_master); i2c_master->cmd_idx = 0; i2c_master->trans_idx = 0; @@ -489,7 +489,7 @@ static void s_i2c_send_commands(i2c_master_bus_handle_t i2c_master, TickType_t t return; } - if (i2c_master->status == I2C_STATUS_ACK_ERROR) { + if (atomic_load(&i2c_master->status) == I2C_STATUS_ACK_ERROR) { ESP_LOGE(TAG, "I2C hardware NACK detected"); const i2c_ll_hw_cmd_t hw_stop_cmd = { .op_code = I2C_LL_CMD_STOP, @@ -596,7 +596,7 @@ static esp_err_t s_i2c_transaction_start(i2c_master_dev_handle_t i2c_dev, int xf TickType_t ticks_to_wait = (xfer_timeout_ms == -1) ? portMAX_DELAY : pdMS_TO_TICKS(xfer_timeout_ms); // Sometimes when the FSM get stuck, the ACK_ERR interrupt will occur endlessly until we reset the FSM and clear bus. esp_err_t ret = ESP_OK; - if (i2c_master->status == I2C_STATUS_TIMEOUT || i2c_ll_is_bus_busy(hal->dev)) { + if (atomic_load(&i2c_master->status) == I2C_STATUS_TIMEOUT || i2c_ll_is_bus_busy(hal->dev)) { ESP_RETURN_ON_ERROR(s_i2c_hw_fsm_reset(i2c_master), TAG, "reset hardware failed"); } @@ -630,7 +630,7 @@ static esp_err_t s_i2c_transaction_start(i2c_master_dev_handle_t i2c_dev, int xf } else { s_i2c_send_commands(i2c_master, ticks_to_wait); // Wait event bits - if (i2c_master->status != I2C_STATUS_DONE) { + if (atomic_load(&i2c_master->status) != I2C_STATUS_DONE) { ret = ESP_ERR_INVALID_STATE; } // Interrupt can be disabled when on transaction finishes. @@ -650,7 +650,7 @@ IRAM_ATTR static void i2c_isr_receive_handler(i2c_master_bus_t *i2c_master) { i2c_hal_context_t *hal = &i2c_master->base->hal; - if (i2c_master->status == I2C_STATUS_READ) { + if (atomic_load(&i2c_master->status) == I2C_STATUS_READ) { i2c_operation_t *i2c_operation = &i2c_master->i2c_trans.ops[i2c_master->trans_idx]; portENTER_CRITICAL_ISR(&i2c_master->base->spinlock); i2c_ll_read_rxfifo(hal->dev, i2c_operation->data + i2c_operation->bytes_used, i2c_master->rx_cnt); From e2ce4d256afb6a5e2a5275f424719da47e2118d6 Mon Sep 17 00:00:00 2001 From: "C.S.M" Date: Mon, 17 Mar 2025 12:37:08 +0800 Subject: [PATCH 2/5] fix(i2c): Fix the array size of static operation, Closes https://github.com/espressif/esp-idf/issues/15583 --- components/esp_driver_i2c/i2c_private.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/components/esp_driver_i2c/i2c_private.h b/components/esp_driver_i2c/i2c_private.h index 89b763c624..ec0934b25d 100644 --- a/components/esp_driver_i2c/i2c_private.h +++ b/components/esp_driver_i2c/i2c_private.h @@ -61,7 +61,7 @@ extern "C" { #define I2C_ALLOW_INTR_PRIORITY_MASK ESP_INTR_FLAG_LOWMED #define I2C_PM_LOCK_NAME_LEN_MAX 16 -#define I2C_STATIC_OPERATION_ARRAY_MAX 6 +#define I2C_STATIC_OPERATION_ARRAY_MAX SOC_I2C_CMD_REG_NUM #define I2C_TRANS_READ_COMMAND(ack_value) {.ack_val = (ack_value), .op_code = I2C_LL_CMD_READ} #define I2C_TRANS_WRITE_COMMAND(ack_check) {.ack_en = (ack_check), .op_code = I2C_LL_CMD_WRITE} From 785e45cfcb35183022e84e070ee022c63cd66415 Mon Sep 17 00:00:00 2001 From: "C.S.M" Date: Thu, 27 Mar 2025 14:32:49 +0800 Subject: [PATCH 3/5] fix(i2c): Fix i2c slave auto selection issue, Closes https://github.com/espressif/esp-idf/issues/15644 --- components/esp_driver_i2c/i2c_slave.c | 9 ++-- components/esp_driver_i2c/i2c_slave_v2.c | 12 +++-- .../i2c_test_apps/main/test_i2c_slave_v2.c | 46 +++++++++++++++++-- 3 files changed, 57 insertions(+), 10 deletions(-) diff --git a/components/esp_driver_i2c/i2c_slave.c b/components/esp_driver_i2c/i2c_slave.c index f70558e1c0..a4df8d075d 100644 --- a/components/esp_driver_i2c/i2c_slave.c +++ b/components/esp_driver_i2c/i2c_slave.c @@ -1,5 +1,5 @@ /* - * SPDX-FileCopyrightText: 2023-2024 Espressif Systems (Shanghai) CO LTD + * SPDX-FileCopyrightText: 2023-2025 Espressif Systems (Shanghai) CO LTD * * SPDX-License-Identifier: Apache-2.0 */ @@ -202,6 +202,7 @@ esp_err_t i2c_new_slave_device(const i2c_slave_config_t *slave_config, i2c_slave ESP_RETURN_ON_FALSE(i2c_slave, ESP_ERR_NO_MEM, TAG, "no memory for i2c slave bus"); ESP_GOTO_ON_ERROR(i2c_acquire_bus_handle(i2c_port_num, &i2c_slave->base, I2C_BUS_MODE_SLAVE), err, TAG, "I2C bus acquire failed"); + i2c_port_num = i2c_slave->base->port_num; i2c_hal_context_t *hal = &i2c_slave->base->hal; i2c_slave->base->scl_num = slave_config->scl_io_num; @@ -293,8 +294,10 @@ err: static esp_err_t i2c_slave_bus_destroy(i2c_slave_dev_handle_t i2c_slave) { if (i2c_slave) { - i2c_ll_disable_intr_mask(i2c_slave->base->hal.dev, I2C_LL_SLAVE_EVENT_INTR); - i2c_common_deinit_pins(i2c_slave->base); + if (i2c_slave->base) { + i2c_ll_disable_intr_mask(i2c_slave->base->hal.dev, I2C_LL_SLAVE_EVENT_INTR); + i2c_common_deinit_pins(i2c_slave->base); + } if (i2c_slave->slv_rx_mux) { vSemaphoreDeleteWithCaps(i2c_slave->slv_rx_mux); i2c_slave->slv_rx_mux = NULL; diff --git a/components/esp_driver_i2c/i2c_slave_v2.c b/components/esp_driver_i2c/i2c_slave_v2.c index 7a38f0425c..dc851d1254 100644 --- a/components/esp_driver_i2c/i2c_slave_v2.c +++ b/components/esp_driver_i2c/i2c_slave_v2.c @@ -1,5 +1,5 @@ /* - * SPDX-FileCopyrightText: 2024 Espressif Systems (Shanghai) CO LTD + * SPDX-FileCopyrightText: 2024-2025 Espressif Systems (Shanghai) CO LTD * * SPDX-License-Identifier: Apache-2.0 */ @@ -207,7 +207,12 @@ IRAM_ATTR static void i2c_slave_isr_handler(void *arg) static esp_err_t i2c_slave_device_destroy(i2c_slave_dev_handle_t i2c_slave) { - i2c_ll_disable_intr_mask(i2c_slave->base->hal.dev, I2C_LL_SLAVE_EVENT_INTR); + esp_err_t ret = ESP_OK; + if (i2c_slave->base) { + i2c_ll_disable_intr_mask(i2c_slave->base->hal.dev, I2C_LL_SLAVE_EVENT_INTR); + i2c_common_deinit_pins(i2c_slave->base); + ret = i2c_release_bus_handle(i2c_slave->base); + } if (i2c_slave->rx_ring_buf) { vRingbufferDeleteWithCaps(i2c_slave->rx_ring_buf); i2c_slave->rx_ring_buf = NULL; @@ -223,7 +228,6 @@ static esp_err_t i2c_slave_device_destroy(i2c_slave_dev_handle_t i2c_slave) if (i2c_slave->receive_desc.buffer) { free(i2c_slave->receive_desc.buffer); } - esp_err_t ret = i2c_release_bus_handle(i2c_slave->base); free(i2c_slave); return ret; @@ -256,7 +260,7 @@ esp_err_t i2c_new_slave_device(const i2c_slave_config_t *slave_config, i2c_slave i2c_slave->base->sda_num = slave_config->sda_io_num; i2c_slave->base->pull_up_enable = slave_config->flags.enable_internal_pullup; i2c_slave->rx_data_count = 0; - int i2c_port_num = slave_config->i2c_port; + i2c_port_num_t i2c_port_num = i2c_slave->base->port_num; ESP_GOTO_ON_ERROR(i2c_common_set_pins(i2c_slave->base), err, TAG, "i2c slave set pins failed"); i2c_slave->rx_ring_buf = xRingbufferCreateWithCaps(slave_config->receive_buf_depth, RINGBUF_TYPE_BYTEBUF, I2C_MEM_ALLOC_CAPS); diff --git a/components/esp_driver_i2c/test_apps/i2c_test_apps/main/test_i2c_slave_v2.c b/components/esp_driver_i2c/test_apps/i2c_test_apps/main/test_i2c_slave_v2.c index f3f00289cc..d7e9bd2d6c 100644 --- a/components/esp_driver_i2c/test_apps/i2c_test_apps/main/test_i2c_slave_v2.c +++ b/components/esp_driver_i2c/test_apps/i2c_test_apps/main/test_i2c_slave_v2.c @@ -1,5 +1,5 @@ /* - * SPDX-FileCopyrightText: 2024 Espressif Systems (Shanghai) CO LTD + * SPDX-FileCopyrightText: 2024-2025 Espressif Systems (Shanghai) CO LTD * * SPDX-License-Identifier: Unlicense OR CC0-1.0 */ @@ -19,6 +19,47 @@ #if SOC_I2C_SLAVE_CAN_GET_STRETCH_CAUSE +TEST_CASE("I2C peripheral allocate slave all", "[i2c]") +{ + i2c_slave_dev_handle_t dev_handle[SOC_HP_I2C_NUM]; + for (int i = 0; i < SOC_HP_I2C_NUM; i++) { + i2c_slave_config_t i2c_slv_config_1 = { + .clk_source = I2C_CLK_SRC_DEFAULT, + .i2c_port = -1, + .scl_io_num = I2C_SLAVE_SCL_IO, + .sda_io_num = I2C_SLAVE_SDA_IO, + .slave_addr = ESP_SLAVE_ADDR, + .send_buf_depth = DATA_LENGTH, + .receive_buf_depth = DATA_LENGTH, + .flags.enable_internal_pullup = true, + }; + + TEST_ESP_OK(i2c_new_slave_device(&i2c_slv_config_1, &dev_handle[i])); + } + i2c_slave_config_t i2c_slv_config_1 = { + .clk_source = I2C_CLK_SRC_DEFAULT, + .i2c_port = -1, + .scl_io_num = I2C_SLAVE_SCL_IO, + .sda_io_num = I2C_SLAVE_SDA_IO, + .slave_addr = ESP_SLAVE_ADDR, + .send_buf_depth = DATA_LENGTH, + .receive_buf_depth = DATA_LENGTH, + .flags.enable_internal_pullup = true, + }; + i2c_slave_dev_handle_t dev_handle_2; + + TEST_ESP_ERR(ESP_ERR_NOT_FOUND, i2c_new_slave_device(&i2c_slv_config_1, &dev_handle_2)); + + for (int i = 0; i < SOC_HP_I2C_NUM; i++) { + TEST_ESP_OK(i2c_del_slave_device(dev_handle[i])); + } + + // Get another one + + TEST_ESP_OK(i2c_new_slave_device(&i2c_slv_config_1, &dev_handle_2)); + TEST_ESP_OK(i2c_del_slave_device(dev_handle_2)); +} + static QueueHandle_t event_queue; static uint8_t *temp_data; static size_t temp_len = 0; @@ -91,7 +132,7 @@ static void i2c_slave_read_test_v2(void) unity_wait_for_signal("master write"); i2c_slave_event_t evt; - if (xQueueReceive(event_queue, &evt, 1) == pdTRUE) { + if (xQueueReceive(event_queue, &evt, portMAX_DELAY) == pdTRUE) { if (evt == I2C_SLAVE_EVT_RX) { disp_buf(temp_data, temp_len); printf("length is %x\n", temp_len); @@ -286,7 +327,6 @@ static void i2c_master_write_test_with_customize_api(void) TEST_ESP_OK(i2c_del_master_bus(bus_handle)); } - TEST_CASE_MULTIPLE_DEVICES("I2C master write slave with customize api", "[i2c][test_env=generic_multi_device][timeout=150]", i2c_master_write_test_with_customize_api, i2c_slave_read_test_v2); #endif // SOC_I2C_SLAVE_CAN_GET_STRETCH_CAUSE From 0752abf3c7402b68f93bff3a585bd4065fe4f3a8 Mon Sep 17 00:00:00 2001 From: "C.S.M" Date: Mon, 24 Mar 2025 14:01:07 +0800 Subject: [PATCH 4/5] fix(i2c_master): Fix the stretch happen cause timeout in probe, Closes https://github.com/espressif/esp-idf/issues/15589 --- components/esp_driver_i2c/i2c_master.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/components/esp_driver_i2c/i2c_master.c b/components/esp_driver_i2c/i2c_master.c index 6d4821d6b5..bdf85f4791 100644 --- a/components/esp_driver_i2c/i2c_master.c +++ b/components/esp_driver_i2c/i2c_master.c @@ -1278,6 +1278,8 @@ esp_err_t i2c_master_probe(i2c_master_bus_handle_t bus_handle, uint16_t address, } i2c_ll_master_set_fractional_divider(hal->dev, 0, 0); i2c_ll_enable_intr_mask(hal->dev, I2C_LL_MASTER_EVENT_INTR); + // 20ms is sufficient for stretch, since there is no device config on probe operation. + i2c_hal_master_set_scl_timeout_val(hal, 20 * 1000, bus_handle->base->clk_src_freq_hz); i2c_ll_update(hal->dev); s_i2c_send_commands(bus_handle, ticks_to_wait); From c29d6575a1eaf59e9030d16926b960cf51b740be Mon Sep 17 00:00:00 2001 From: "C.S.M" Date: Mon, 31 Mar 2025 16:47:07 +0800 Subject: [PATCH 5/5] fix(i2c): Fix i2c customize only read one byte issue --- components/esp_driver_i2c/i2c_master.c | 13 ++- components/esp_driver_i2c/i2c_slave.c | 2 +- .../i2c_test_apps/main/test_i2c_slave_v2.c | 92 +++++++++++++++++++ 3 files changed, 105 insertions(+), 2 deletions(-) diff --git a/components/esp_driver_i2c/i2c_master.c b/components/esp_driver_i2c/i2c_master.c index bdf85f4791..298a4127cb 100644 --- a/components/esp_driver_i2c/i2c_master.c +++ b/components/esp_driver_i2c/i2c_master.c @@ -294,6 +294,13 @@ static bool s_i2c_read_command(i2c_master_bus_handle_t i2c_master, i2c_operation i2c_master->read_buf_pos = i2c_master->trans_idx; } else { i2c_ll_master_write_cmd_reg(hal->dev, hw_cmd, i2c_master->cmd_idx); + // If the read position has not been marked, that means the transaction doesn't contain read-ack + // operation, then mark the read position in read-nack operation. + // i2c_master->read_buf_pos will never be 0. + if (i2c_master->read_buf_pos == 0) { + i2c_master->read_buf_pos = i2c_master->trans_idx; + i2c_master->read_len_static = i2c_master->rx_cnt; + } i2c_master->cmd_idx++; } i2c_master->trans_idx++; @@ -610,6 +617,7 @@ static esp_err_t s_i2c_transaction_start(i2c_master_dev_handle_t i2c_dev, int xf i2c_master->cmd_idx = 0; i2c_master->rx_cnt = 0; i2c_master->read_len_static = 0; + i2c_master->read_buf_pos = 0; I2C_CLOCK_SRC_ATOMIC() { i2c_hal_set_bus_timing(hal, i2c_dev->scl_speed_hz, i2c_master->base->clk_src, i2c_master->base->clk_src_freq_hz); @@ -673,7 +681,10 @@ IRAM_ATTR static void i2c_isr_receive_handler(i2c_master_bus_t *i2c_master) i2c_operation_t *i2c_operation = &i2c_master->i2c_trans.ops[i2c_master->read_buf_pos]; portENTER_CRITICAL_ISR(&i2c_master->base->spinlock); i2c_ll_read_rxfifo(hal->dev, i2c_operation->data + i2c_operation->bytes_used, i2c_master->read_len_static); - i2c_ll_read_rxfifo(hal->dev, i2c_master->i2c_trans.ops[i2c_master->read_buf_pos + 1].data, 1); + // If the read command only contain nack marker, no read it for the second time. + if (i2c_master->i2c_trans.ops[i2c_master->read_buf_pos + 1].data) { + i2c_ll_read_rxfifo(hal->dev, i2c_master->i2c_trans.ops[i2c_master->read_buf_pos + 1].data, 1); + } i2c_master->w_r_size = i2c_master->read_len_static + 1; i2c_master->contains_read = false; portEXIT_CRITICAL_ISR(&i2c_master->base->spinlock); diff --git a/components/esp_driver_i2c/i2c_slave.c b/components/esp_driver_i2c/i2c_slave.c index a4df8d075d..85c56f4f14 100644 --- a/components/esp_driver_i2c/i2c_slave.c +++ b/components/esp_driver_i2c/i2c_slave.c @@ -297,6 +297,7 @@ static esp_err_t i2c_slave_bus_destroy(i2c_slave_dev_handle_t i2c_slave) if (i2c_slave->base) { i2c_ll_disable_intr_mask(i2c_slave->base->hal.dev, I2C_LL_SLAVE_EVENT_INTR); i2c_common_deinit_pins(i2c_slave->base); + i2c_release_bus_handle(i2c_slave->base); } if (i2c_slave->slv_rx_mux) { vSemaphoreDeleteWithCaps(i2c_slave->slv_rx_mux); @@ -313,7 +314,6 @@ static esp_err_t i2c_slave_bus_destroy(i2c_slave_dev_handle_t i2c_slave) if (i2c_slave->slv_evt_queue) { vQueueDeleteWithCaps(i2c_slave->slv_evt_queue); } - i2c_release_bus_handle(i2c_slave->base); } free(i2c_slave); diff --git a/components/esp_driver_i2c/test_apps/i2c_test_apps/main/test_i2c_slave_v2.c b/components/esp_driver_i2c/test_apps/i2c_test_apps/main/test_i2c_slave_v2.c index d7e9bd2d6c..306e8f0844 100644 --- a/components/esp_driver_i2c/test_apps/i2c_test_apps/main/test_i2c_slave_v2.c +++ b/components/esp_driver_i2c/test_apps/i2c_test_apps/main/test_i2c_slave_v2.c @@ -329,4 +329,96 @@ static void i2c_master_write_test_with_customize_api(void) } TEST_CASE_MULTIPLE_DEVICES("I2C master write slave with customize api", "[i2c][test_env=generic_multi_device][timeout=150]", i2c_master_write_test_with_customize_api, i2c_slave_read_test_v2); +static void master_read_slave_test_v2_single_byte(void) +{ + uint8_t data_rd[DATA_LENGTH] = {0}; + i2c_master_bus_config_t i2c_mst_config = { + .clk_source = I2C_CLK_SRC_DEFAULT, + .i2c_port = TEST_I2C_PORT, + .scl_io_num = I2C_MASTER_SCL_IO, + .sda_io_num = I2C_MASTER_SDA_IO, + .flags.enable_internal_pullup = true, + }; + i2c_master_bus_handle_t bus_handle; + TEST_ESP_OK(i2c_new_master_bus(&i2c_mst_config, &bus_handle)); + + i2c_device_config_t dev_cfg = { + .dev_addr_length = I2C_ADDR_BIT_LEN_7, + .device_address = I2C_DEVICE_ADDRESS_NOT_USED, + .scl_speed_hz = 100000, + .scl_wait_us = 20000, + }; + + i2c_master_dev_handle_t dev_handle; + TEST_ESP_OK(i2c_master_bus_add_device(bus_handle, &dev_cfg, &dev_handle)); + + unity_wait_for_signal("i2c slave init finish"); + + uint8_t read_address = (ESP_SLAVE_ADDR << 1 | 1); + + i2c_operation_job_t i2c_ops[] = { + { .command = I2C_MASTER_CMD_START }, + { .command = I2C_MASTER_CMD_WRITE, .write = { .ack_check = true, .data = (uint8_t *) &read_address, .total_bytes = 1 } }, + { .command = I2C_MASTER_CMD_READ, .read = { .ack_value = I2C_NACK_VAL, .data = data_rd, .total_bytes = 1 }}, + { .command = I2C_MASTER_CMD_STOP }, + }; + + i2c_master_execute_defined_operations(dev_handle, i2c_ops, sizeof(i2c_ops) / sizeof(i2c_operation_job_t), 1000); + vTaskDelay(100 / portTICK_PERIOD_MS); + TEST_ASSERT(data_rd[0] == 6); + unity_send_signal("ready to delete master read test"); + + TEST_ESP_OK(i2c_master_bus_rm_device(dev_handle)); + TEST_ESP_OK(i2c_del_master_bus(bus_handle)); +} + +static void slave_write_buffer_test_v2_single_byte(void) +{ + i2c_slave_dev_handle_t handle; + uint8_t data_wr[DATA_LENGTH]; + event_queue = xQueueCreate(2, sizeof(i2c_slave_event_t)); + assert(event_queue); + + i2c_slave_config_t i2c_slv_config = { + .i2c_port = TEST_I2C_PORT, + .clk_source = I2C_CLK_SRC_DEFAULT, + .scl_io_num = I2C_SLAVE_SCL_IO, + .sda_io_num = I2C_SLAVE_SDA_IO, + .slave_addr = ESP_SLAVE_ADDR, + .send_buf_depth = DATA_LENGTH, + .receive_buf_depth = DATA_LENGTH, + .flags.enable_internal_pullup = true, + }; + + TEST_ESP_OK(i2c_new_slave_device(&i2c_slv_config, &handle)); + + i2c_slave_event_callbacks_t cbs = { + .on_receive = i2c_slave_receive_cb, + .on_request = i2c_slave_request_cb, + }; + + TEST_ESP_OK(i2c_slave_register_event_callbacks(handle, &cbs, NULL)); + + unity_send_signal("i2c slave init finish"); + + data_wr[0] = 6; + + i2c_slave_event_t evt; + uint32_t write_len; + while (true) { + if (xQueueReceive(event_queue, &evt, portMAX_DELAY) == pdTRUE) { + if (evt == I2C_SLAVE_EVT_TX) { + TEST_ESP_OK(i2c_slave_write(handle, data_wr, 1, &write_len, 1000)); + break; + } + } + } + + unity_wait_for_signal("ready to delete master read test"); + vQueueDelete(event_queue); + TEST_ESP_OK(i2c_del_slave_device(handle)); +} + +TEST_CASE_MULTIPLE_DEVICES("I2C master read slave test single byte", "[i2c][test_env=generic_multi_device][timeout=150]", master_read_slave_test_v2_single_byte, slave_write_buffer_test_v2_single_byte); + #endif // SOC_I2C_SLAVE_CAN_GET_STRETCH_CAUSE