feat(ch422g): support enter/exit sleep

.github/workflows/arduino_lint.yml

@@ -4,6 +4,9 @@ on:
   workflow_dispatch:
   pull_request:
     types: [opened, reopened, synchronize]
+  push:
+    branches:
+      - master
 
 jobs:
   lint:

.github/workflows/build_test.yml

@@ -4,6 +4,9 @@ on:
   workflow_dispatch:
   pull_request:
     types: [opened, reopened, synchronize]
+  push:
+    branches:
+      - master
 
 jobs:
   build:

.github/workflows/pre-commit.yml

@@ -11,4 +11,4 @@ jobs:
     steps:
     - uses: actions/checkout@v2
     - uses: actions/setup-python@v2
-    - uses: pre-commit/action@v2.0.3
+    - uses: pre-commit/action@v3.0.1

.github/workflows/upload_component.yml

@@ -15,6 +15,6 @@ jobs:
       - name: Upload components to component service
         uses: espressif/upload-components-ci-action@v1
         with:
-          name: "ESP32_IO_Expander"
+          name: "esp32_io_expander"
           namespace: "espressif"
           api_token: ${{ secrets.IDF_COMPONENT_API_TOKEN }}

CHANGELOG.md

@@ -1,5 +1,15 @@
 # ChangeLog
 
+## v1.1.1 - 2025-07-07
+
+### Enhancements:
+
+* feat(ch422g): support enter/exit sleep
+
+### Bug Fixes:
+
+* fix(port): fix discarded qualifiers warning
+
 ## v1.1.0 - 2025-02-07
 
 ### Enhancements:

idf_component.yml

@@ -1,4 +1,4 @@
-version: "1.1.0"
+version: "1.1.1"
 description: ESP32_IO_Expander is a library designed for driving IO expander chips using ESP SoCs
 url: https://github.com/esp-arduino-libs/ESP32_IO_Expander
 repository: https://github.com/esp-arduino-libs/ESP32_IO_Expander.git

library.properties

@@ -1,5 +1,5 @@
 name=ESP32_IO_Expander
-version=1.1.0
+version=1.1.1
 author=espressif
 maintainer=espressif
 sentence=ESP32_IO_Expander is a library designed for driving IO expander chips using ESP SoCs

src/chip/esp_expander_ch422g.cpp

@@ -104,4 +104,34 @@ bool CH422G::enableAllIO_Output(void)
     return true;
 }
 
+bool CH422G::enterSleep(void)
+{
+    ESP_UTILS_LOG_TRACE_ENTER_WITH_THIS();
+
+    ESP_UTILS_CHECK_FALSE_RETURN(isOverState(State::BEGIN), false, "Not begun");
+
+    ESP_UTILS_CHECK_ERROR_RETURN(
+        esp_io_expander_ch422g_enter_sleep(device_handle), false, "Enter sleep failed"
+    );
+
+    ESP_UTILS_LOG_TRACE_EXIT_WITH_THIS();
+
+    return true;
+}
+
+bool CH422G::exitSleep(void)
+{
+    ESP_UTILS_LOG_TRACE_ENTER_WITH_THIS();
+
+    ESP_UTILS_CHECK_FALSE_RETURN(isOverState(State::BEGIN), false, "Not begun");
+
+    ESP_UTILS_CHECK_ERROR_RETURN(
+        esp_io_expander_ch422g_exit_sleep(device_handle), false, "Exit sleep failed"
+    );
+
+    ESP_UTILS_LOG_TRACE_EXIT_WITH_THIS();
+
+    return true;
+}
+
 } // namespace esp_expander

src/chip/esp_expander_ch422g.hpp

@@ -99,6 +99,20 @@ class CH422G: public Base {
      * @return true if success, otherwise false
      */
     bool enableAllIO_Output(void);
+
+    /**
+     * @brief Enter sleep mode
+     *
+     * @return true if success, otherwise false
+     */
+    bool enterSleep(void);
+
+    /**
+     * @brief Exit sleep mode
+     *
+     * @return true if success, otherwise false
+     */
+    bool exitSleep(void);
 };
 
 } // namespace esp_expander

src/port/esp_io_expander.c

@@ -26,7 +26,7 @@ typedef enum {
     REG_DIRECTION,
 } reg_type_t;
 
-static char *TAG = "io_expander";
+static const char *TAG = "io_expander";
 
 static esp_err_t write_reg(esp_io_expander_handle_t handle, reg_type_t reg, uint32_t value);
 static esp_err_t read_reg(esp_io_expander_handle_t handle, reg_type_t reg, uint32_t *value);

src/port/esp_io_expander_ch422g.c

@@ -38,13 +38,14 @@
                                             // Default:    |  0  |  0  |  0  | 0   |    0    |    0    |    0     |    1    |
 
 // *INDENT-OFF*
-#define REG_WR_OC_DEFAULT_VAL   (0x0FUL)
-#define REG_WR_IO_DEFAULT_VAL   (0xFFUL)
+#define REG_WR_OC_DEFAULT_VAL   (0x0FU)
+#define REG_WR_IO_DEFAULT_VAL   (0xFFU)
 #define REG_OUT_DEFAULT_VAL     ((REG_WR_OC_DEFAULT_VAL << 8) | REG_WR_IO_DEFAULT_VAL)
-#define REG_DIR_DEFAULT_VAL     (0xFFFUL)
+#define REG_DIR_DEFAULT_VAL     (0xFFFU)
 
-#define REG_WR_SET_BIT_IO_OE    (1 << 0)
-#define REG_WR_SET_BIT_OD_EN    (1 << 2)
+#define REG_WR_SET_BIT_IO_OE    (1U << 0)
+#define REG_WR_SET_BIT_OD_EN    (1U << 2)
+#define REG_WR_SET_BIT_SLEEP    (1U << 3)
 
 /**
  * @brief Device Structure Type
@@ -171,6 +172,38 @@ esp_err_t esp_io_expander_ch422g_set_all_output(esp_io_expander_handle_t handle)
     return ESP_OK;
 }
 
+esp_err_t esp_io_expander_ch422g_enter_sleep(esp_io_expander_handle_t handle)
+{
+    esp_io_expander_ch422g_t *ch422g = (esp_io_expander_ch422g_t *)__containerof(handle, esp_io_expander_ch422g_t, base);
+    uint8_t data = (uint8_t)(ch422g->regs.wr_set | REG_WR_SET_BIT_SLEEP);
+
+    // WR-SET
+    ESP_RETURN_ON_ERROR(
+        i2c_master_write_to_device(
+            ch422g->i2c_num, CH422G_REG_WR_SET, &data, sizeof(data), pdMS_TO_TICKS(I2C_TIMEOUT_MS)
+        ), TAG, "Write WR_SET reg failed"
+    );
+    ch422g->regs.wr_set = data;
+
+    return ESP_OK;
+}
+
+esp_err_t esp_io_expander_ch422g_exit_sleep(esp_io_expander_handle_t handle)
+{
+    esp_io_expander_ch422g_t *ch422g = (esp_io_expander_ch422g_t *)__containerof(handle, esp_io_expander_ch422g_t, base);
+    uint8_t data = (uint8_t)(ch422g->regs.wr_set & ~REG_WR_SET_BIT_SLEEP);
+
+    // WR-SET
+    ESP_RETURN_ON_ERROR(
+        i2c_master_write_to_device(
+            ch422g->i2c_num, CH422G_REG_WR_SET, &data, sizeof(data), pdMS_TO_TICKS(I2C_TIMEOUT_MS)
+        ), TAG, "Write WR_SET reg failed"
+    );
+    ch422g->regs.wr_set = data;
+
+    return ESP_OK;
+}
+
 static esp_err_t read_input_reg(esp_io_expander_handle_t handle, uint32_t *value)
 {
     esp_io_expander_ch422g_t *ch422g = (esp_io_expander_ch422g_t *)__containerof(handle, esp_io_expander_ch422g_t, base);

src/port/esp_io_expander_ch422g.h

@@ -48,6 +48,10 @@ esp_err_t esp_io_expander_ch422g_set_all_input(esp_io_expander_handle_t handle);
 
 esp_err_t esp_io_expander_ch422g_set_all_output(esp_io_expander_handle_t handle);
 
+esp_err_t esp_io_expander_ch422g_enter_sleep(esp_io_expander_handle_t handle);
+
+esp_err_t esp_io_expander_ch422g_exit_sleep(esp_io_expander_handle_t handle);
+
 #ifdef __cplusplus
 }
 #endif

src/port/esp_io_expander_ht8574.c

@@ -40,7 +40,7 @@ typedef struct {
     } regs;
 } esp_io_expander_ht8574_t;
 
-static char *TAG = "ht8574";
+static const char *TAG = "ht8574";
 
 static esp_err_t read_input_reg(esp_io_expander_handle_t handle, uint32_t *value);
 static esp_err_t write_output_reg(esp_io_expander_handle_t handle, uint32_t value);

src/port/esp_io_expander_tca9554.c

@@ -45,7 +45,7 @@ typedef struct {
     } regs;
 } esp_io_expander_tca9554_t;
 
-static char *TAG = "tca9554";
+static const char *TAG = "tca9554";
 
 static esp_err_t read_input_reg(esp_io_expander_handle_t handle, uint32_t *value);
 static esp_err_t write_output_reg(esp_io_expander_handle_t handle, uint32_t value);

src/port/esp_io_expander_tca95xx_16bit.c

@@ -45,7 +45,7 @@ typedef struct {
     } regs;
 } esp_io_expander_tca95xx_16bit_t;
 
-static char *TAG = "tca95xx_16";
+static const char *TAG = "tca95xx_16";
 
 static esp_err_t read_input_reg(esp_io_expander_handle_t handle, uint32_t *value);
 static esp_err_t write_output_reg(esp_io_expander_handle_t handle, uint32_t value);