esp32: Update machine_i2c.c.

Signed-off-by: Vincent1-python <pywei201209@163.com>

Author: Vincent1-python

ports/esp32/machine_i2c.c

@@ -54,9 +54,9 @@ typedef struct _machine_hw_i2c_obj_t {
     mp_obj_base_t base;
     i2c_master_bus_handle_t bus_handle;
     i2c_master_dev_handle_t dev_handle;
-    uint8_t port;
-    gpio_num_t scl;
-    gpio_num_t sda;
+    uint8_t port : 8;
+    gpio_num_t scl : 8;
+    gpio_num_t sda : 8;
 } machine_hw_i2c_obj_t;
 
 static machine_hw_i2c_obj_t machine_hw_i2c_obj[I2C_NUM_MAX];
@@ -103,8 +103,8 @@ int machine_hw_i2c_transfer(mp_obj_base_t *self_in, uint16_t addr, size_t n, mp_
     /* 1. Create a temporary device handle for this transaction */
     i2c_device_config_t dev_cfg = {
         .dev_addr_length = I2C_ADDR_BIT_LEN_7,
-        .device_address  = addr,
-        .scl_speed_hz    = 100000,   /* Use bus frequency */
+        .device_address = addr,
+        .scl_speed_hz = 100000,   /* Use bus frequency */
     };
     i2c_master_dev_handle_t dev_handle;
     err = i2c_master_bus_add_device(self->bus_handle, &dev_cfg, &dev_handle);
@@ -117,8 +117,10 @@ int machine_hw_i2c_transfer(mp_obj_base_t *self_in, uint16_t addr, size_t n, mp_
     /* 2. If WRITE1 segment exists, perform the write first */
     if (flags & MP_MACHINE_I2C_FLAG_WRITE1) {
         if (bufs->len) {
-            err = i2c_master_transmit(dev_handle,bufs->buf,bufs->len,1000);                 /* Block with 1 s timeout */
-            if (err != ESP_OK) goto cleanup;
+            err = i2c_master_transmit(dev_handle, bufs->buf, bufs->len, 1000);                 /* Block with 1 s timeout */
+            if (err != ESP_OK) {
+                goto cleanup;
+            }
         }
         data_len += bufs->len;
         --n;
@@ -127,9 +129,13 @@ int machine_hw_i2c_transfer(mp_obj_base_t *self_in, uint16_t addr, size_t n, mp_
     if (flags & MP_MACHINE_I2C_FLAG_READ) {
         /* 3. Main loop: remaining segments */
         for (; n--; ++bufs) {
-            if (bufs->len == 0) continue;
-            err = i2c_master_receive(dev_handle,bufs->buf,bufs->len,1000);
-            if (err != ESP_OK) break;
+            if (bufs->len == 0) {
+                continue;
+            }
+            err = i2c_master_receive(dev_handle, bufs->buf, bufs->len, 1000);
+            if (err != ESP_OK) {
+                break;
+            }
 
             data_len += bufs->len;
         }
@@ -150,7 +156,9 @@ int machine_hw_i2c_transfer(mp_obj_base_t *self_in, uint16_t addr, size_t n, mp_
         n = yuann;
         // Dynamically allocate write_buf
         uint8_t *write_buf = (uint8_t *)malloc(total_len);
-        if (write_buf == NULL) return -MP_ENOMEM;
+        if (write_buf == NULL) {
+            return -MP_ENOMEM;
+        }
 
         // Copy data into write_buf
         size_t index = 0;
@@ -161,7 +169,9 @@ int machine_hw_i2c_transfer(mp_obj_base_t *self_in, uint16_t addr, size_t n, mp_
 
         // Transmit data
         err = i2c_master_transmit(dev_handle, write_buf, total_len, 1000);
-        if (err != ESP_OK) goto cleanup;
+        if (err != ESP_OK) {
+            goto cleanup;
+        }
 
         // Free dynamically allocated memory
         free(write_buf);
@@ -172,18 +182,23 @@ int machine_hw_i2c_transfer(mp_obj_base_t *self_in, uint16_t addr, size_t n, mp_
     i2c_master_bus_rm_device(dev_handle);
 
     /* 5. Map errors */
-    if (err == ESP_FAIL)        return -MP_ENODEV;
-    if (err == ESP_ERR_TIMEOUT) return -MP_ETIMEDOUT;
-    if (err != ESP_OK)          return -abs(err);
+    if (err == ESP_FAIL) {
+        return -MP_ENODEV;
+    }
+    if (err == ESP_ERR_TIMEOUT) {
+        return -MP_ETIMEDOUT;
+    }
+    if (err != ESP_OK) {
+        return -abs(err);
+    }
 
     return data_len;
 }
 
 // ---------------- Print ----------------
 static void machine_hw_i2c_print(const mp_print_t *print, mp_obj_t self_in, mp_print_kind_t kind) {
     machine_hw_i2c_obj_t *self = MP_OBJ_TO_PTR(self_in);
-    mp_printf(print, "I2C(%u, scl=%u, sda=%u)",
-              self->port, self->scl, self->sda);
+    mp_printf(print, "I2C(%u, scl=%u, sda=%u)", self->port, self->scl, self->sda);
 }
 
 // ---------------- Constructor ----------------
@@ -203,11 +218,11 @@ mp_obj_t machine_hw_i2c_make_new(const mp_obj_type_t *type, size_t n_args, size_
         { MP_QSTR_timeout, MP_ARG_KW_ONLY | MP_ARG_INT, {.u_int = I2C_DEFAULT_TIMEOUT_US} },
     };
     mp_arg_val_t args[MP_ARRAY_SIZE(allowed_args)];
-    mp_arg_parse_all_kw_array(n_args, n_kw, all_args,MP_ARRAY_SIZE(allowed_args), allowed_args, args);
+    mp_arg_parse_all_kw_array(n_args, n_kw, all_args, MP_ARRAY_SIZE(allowed_args), allowed_args, args);
 
     mp_int_t i2c_id = args[ARG_id].u_int;
     if (!(I2C_NUM_0 <= i2c_id && i2c_id < I2C_NUM_MAX)) {
-        mp_raise_msg_varg(&mp_type_ValueError,MP_ERROR_TEXT("I2C(%d) doesn't exist"), i2c_id);
+        mp_raise_msg_varg(&mp_type_ValueError, MP_ERROR_TEXT("I2C(%d) doesn't exist"), i2c_id);
     }
 
     machine_hw_i2c_obj_t *self = &machine_hw_i2c_obj[i2c_id];
@@ -227,7 +242,7 @@ mp_obj_t machine_hw_i2c_make_new(const mp_obj_type_t *type, size_t n_args, size_
         self->sda = machine_pin_get_id(args[ARG_sda].u_obj);
     }
 
-    machine_hw_i2c_init(self,args[ARG_freq].u_int,args[ARG_timeout].u_int,first_init);
+    machine_hw_i2c_init(self, args[ARG_freq].u_int, args[ARG_timeout].u_int, first_init);
     return MP_OBJ_FROM_PTR(self);
 }