icm42670: Support I2C NG driver

components/icm42670/icm42670.c

@@ -1,18 +1,22 @@
 /*
- * SPDX-FileCopyrightText: 2023 Espressif Systems (Shanghai) CO LTD
+ * SPDX-FileCopyrightText: 2023-2024 Espressif Systems (Shanghai) CO LTD
  *
  * SPDX-License-Identifier: Apache-2.0
  */
 
+#include <string.h>
 #include <stdio.h>
 #include <math.h>
 #include <time.h>
 #include <sys/time.h>
 #include "esp_system.h"
 #include "esp_check.h"
-#include "driver/i2c.h"
+#include "driver/i2c_master.h"
+#include "esp_heap_caps.h"
 #include "icm42670.h"
 
+#define I2C_CLK_SPEED 400000
+
 #define ALPHA                       0.99f        /*!< Weight of gyroscope */
 #define RAD_TO_DEG                  57.27272727f /*!< Radians to degrees */
 
@@ -46,8 +50,7 @@
 *******************************************************************************/
 
 typedef struct {
-    i2c_port_t bus;
-    uint8_t dev_addr;
+    i2c_master_dev_handle_t i2c_handle;
     uint32_t counter;
     float dt;  /*!< delay time between two measurements, dt should be small (ms level) */
     struct timeval *timer;
@@ -70,15 +73,21 @@ static const char *TAG = "ICM42670";
 * Public API functions
 *******************************************************************************/
 
-icm42670_handle_t icm42670_create(i2c_port_t port, const uint8_t dev_addr)
+icm42670_handle_t icm42670_create(i2c_master_bus_handle_t i2c_bus, const uint8_t dev_addr)
 {
+    esp_err_t ret = ESP_OK;
+    ESP_RETURN_ON_FALSE(heap_caps_check_integrity_addr((intptr_t)i2c_bus, false), NULL, TAG, "Please use I2C driver NG and put I2C bus handle as a first argument.");
     icm42670_dev_t *sensor = (icm42670_dev_t *) heap_caps_calloc(1, sizeof(icm42670_dev_t), MALLOC_CAP_DEFAULT);
-    sensor->bus = port;
-    sensor->dev_addr = dev_addr;
     sensor->counter = 0;
     sensor->dt = 0;
     sensor->timer = (struct timeval *) calloc(1, sizeof(struct timeval));
 
+    i2c_device_config_t i2c_dev_cfg = {
+        .device_address = dev_addr,
+        .scl_speed_hz = I2C_CLK_SPEED,
+    };
+    ESP_GOTO_ON_ERROR(i2c_master_bus_add_device(i2c_bus, &i2c_dev_cfg, &sensor->i2c_handle), err, TAG, "i2c add device fail");
+
     uint8_t dev_id = 0;
     icm42670_get_deviceid(sensor, &dev_id);
     if (dev_id != ICM42607_ID && dev_id != ICM42670_ID) {
@@ -87,14 +96,22 @@ icm42670_handle_t icm42670_create(i2c_port_t port, const uint8_t dev_addr)
     }
 
     ESP_LOGI(TAG, "Found device %s, ID: 0x%02x", (dev_id == ICM42607_ID ? "ICM42607" : "ICM42670"), dev_id);
-
     return (icm42670_handle_t) sensor;
+err:
+    if (ret != ESP_OK) {
+        icm42670_delete(sensor);
+    }
+    return NULL;
 }
 
 void icm42670_delete(icm42670_handle_t sensor)
 {
     icm42670_dev_t *sens = (icm42670_dev_t *) sensor;
 
+    if (sens->i2c_handle) {
+        i2c_master_bus_rm_device(sens->i2c_handle);
+    }
+
     if (sens->timer) {
         free(sens->timer);
     }
@@ -343,36 +360,24 @@ static esp_err_t icm42670_get_raw_value(icm42670_handle_t sensor, uint8_t reg, i
 static esp_err_t icm42670_write(icm42670_handle_t sensor, const uint8_t reg_start_addr, const uint8_t *data_buf, const uint8_t data_len)
 {
     icm42670_dev_t *sens = (icm42670_dev_t *) sensor;
-    esp_err_t  ret;
-
     assert(sens);
 
-    i2c_cmd_handle_t cmd = i2c_cmd_link_create();
-    ret = i2c_master_start(cmd);
-    assert(ESP_OK == ret);
-    ret = i2c_master_write_byte(cmd, (sens->dev_addr << 1) | I2C_MASTER_WRITE, true);
-    assert(ESP_OK == ret);
-    ret = i2c_master_write_byte(cmd, reg_start_addr, true);
-    assert(ESP_OK == ret);
-    ret = i2c_master_write(cmd, data_buf, data_len, true);
-    assert(ESP_OK == ret);
-    ret = i2c_master_stop(cmd);
-    assert(ESP_OK == ret);
-    ret = i2c_master_cmd_begin(sens->bus, cmd, 1000 / portTICK_PERIOD_MS);
-    i2c_cmd_link_delete(cmd);
-
-    return ret;
+    uint8_t reg_buff[] = {reg_start_addr};
+    i2c_master_transmit_multi_buffer_info_t i2c_buffer[2] = {
+        {.write_buffer = reg_buff, .buffer_size = sizeof(reg_buff)},
+        {.write_buffer = (uint8_t *)data_buf, .buffer_size = data_len},
+    };
+    return i2c_master_multi_buffer_transmit(sens->i2c_handle, i2c_buffer, sizeof(i2c_buffer) / sizeof(i2c_master_transmit_multi_buffer_info_t), -1);
 }
 
 static esp_err_t icm42670_read(icm42670_handle_t sensor, const uint8_t reg_start_addr, uint8_t *data_buf, const uint8_t data_len)
 {
     icm42670_dev_t *sens = (icm42670_dev_t *) sensor;
-    uint8_t reg_buff[] = {reg_start_addr};
-
     assert(sens);
 
     /* Write register number and read data */
-    return i2c_master_write_read_device(sens->bus, sens->dev_addr, reg_buff, sizeof(reg_buff), data_buf, data_len, 1000 / portTICK_PERIOD_MS);
+    uint8_t reg_buff[] = {reg_start_addr};
+    return i2c_master_transmit_receive(sens->i2c_handle, reg_buff, sizeof(reg_buff), data_buf, data_len, -1);
 }
 
 esp_err_t icm42670_complimentory_filter(icm42670_handle_t sensor, const icm42670_value_t *const acce_value,

components/icm42670/idf_component.yml

@@ -1,4 +1,4 @@
-version: "1.0.0"
+version: "1.1.0"
 description: I2C driver for ICM 42670 6-Axis MotionTracking
 url: https://github.com/espressif/esp-bsp/tree/master/components/icm42670
 dependencies:

components/icm42670/include/icm42670.h

@@ -1,5 +1,5 @@
 /*
- * SPDX-FileCopyrightText: 2023 Espressif Systems (Shanghai) CO LTD
+ * SPDX-FileCopyrightText: 2023-2024 Espressif Systems (Shanghai) CO LTD
  *
  * SPDX-License-Identifier: Apache-2.0
  */
@@ -10,7 +10,7 @@
 extern "C" {
 #endif
 
-#include "driver/i2c.h"
+#include "driver/i2c_master.h"
 
 #define ICM42670_I2C_ADDRESS         0x68 /*!< I2C address with AD0 pin low */
 #define ICM42670_I2C_ADDRESS_1       0x69 /*!< I2C address with AD0 pin high */
@@ -96,14 +96,14 @@ typedef void *icm42670_handle_t;
 /**
  * @brief Create and init sensor object and return a sensor handle
  *
- * @param port I2C port number
+ * @param i2c_bus I2C bus
  * @param dev_addr I2C device address of sensor
  *
  * @return
  *     - NULL Fail
  *     - Others Success
  */
-icm42670_handle_t icm42670_create(i2c_port_t port, const uint8_t dev_addr);
+icm42670_handle_t icm42670_create(i2c_master_bus_handle_t i2c_bus, const uint8_t dev_addr);
 
 /**
  * @brief Delete and release a sensor object