Merge branch 'refactor/remove_i2c_slave_v1' into 'master'

refactor(i2c): Remove i2c slave v1 driver

See merge request espressif/esp-idf!39329

Author: mythbuster5

components/esp_driver_i2c/CMakeLists.txt

@@ -10,11 +10,7 @@ if(CONFIG_SOC_I2C_SUPPORTED)
                      "i2c_common.c"
         )
     if(CONFIG_SOC_I2C_SUPPORT_SLAVE)
-        if(CONFIG_I2C_ENABLE_SLAVE_DRIVER_VERSION_2)
-            list(APPEND srcs "i2c_slave_v2.c")
-        else()
-            list(APPEND srcs "i2c_slave.c")
-        endif()
+        list(APPEND srcs "i2c_slave.c")
     endif()
 
 endif()

components/esp_driver_i2c/Kconfig

@@ -18,12 +18,6 @@ menu "ESP-Driver:I2C Configurations"
 
             note: This cannot be used in the I2C legacy driver.
 
-    config I2C_ENABLE_SLAVE_DRIVER_VERSION_2
-        bool "Enable I2C slave driver version 2"
-        default n
-        help
-            I2C slave version 2 solves some existing known issues. Such as write/read workflow, stretch handling, etc.
-
     config I2C_MASTER_ISR_HANDLER_IN_IRAM
         bool "Place I2C master ISR handler into IRAM"
         default y

components/esp_driver_i2c/i2c_private.h

@@ -193,27 +193,6 @@ typedef struct {
     uint32_t rcv_fifo_cnt;      // receive fifo count.
 } i2c_slave_receive_t;
 
-#if !CONFIG_I2C_ENABLE_SLAVE_DRIVER_VERSION_2
-
-struct i2c_slave_dev_t {
-    i2c_bus_t *base;                            // bus base class
-    SemaphoreHandle_t slv_rx_mux;               // Mutex for slave rx direction
-    SemaphoreHandle_t slv_tx_mux;               // Mutex for slave tx direction
-    RingbufHandle_t rx_ring_buf;                // Handle for rx ringbuffer
-    RingbufHandle_t tx_ring_buf;                // Handle for tx ringbuffer
-    uint8_t data_buf[SOC_I2C_FIFO_LEN];         // Data buffer for slave
-    uint32_t trans_data_length;                 // Send data length
-    i2c_slave_event_callbacks_t callbacks;      // I2C slave callbacks
-    void *user_ctx;                             // Callback user context
-    i2c_slave_fifo_mode_t fifo_mode;            // Slave fifo mode.
-    QueueHandle_t slv_evt_queue;                // Event Queue used in slave nonfifo mode.
-    i2c_slave_evt_t slave_evt;                  // Slave event structure.
-    i2c_slave_receive_t receive_desc;           // Slave receive descriptor
-    uint32_t already_receive_len;               // Data length already received in ISR.
-};
-
-#else // CONFIG_I2C_ENABLE_SLAVE_DRIVER_VERSION_2
-
 struct i2c_slave_dev_t {
     i2c_bus_t *base;                                  // bus base class
     SemaphoreHandle_t operation_mux;                  // Mux for i2c slave operation
@@ -226,8 +205,6 @@ struct i2c_slave_dev_t {
     i2c_slave_receive_t receive_desc;                 // slave receive descriptor
 };
 
-#endif // CONFIG_I2C_ENABLE_SLAVE_DRIVER_VERSION_2
-
 /**
  * @brief Acquire I2C bus handle
  *