Mirrors/esp-idf
1
0
Fork 0
mirror of https://github.com/espressif/esp-idf.git synced 2026-10-02 11:10:54 +03:00
Code Issues Packages Projects Releases Wiki Activity

fix(lp_core): fix LP UART init parameter validation

Browse Source 
Align data_bits, flow_ctrl, and rx_flow_ctrl_thresh checks with uart_param_config().

Made-with: Cursor
This commit is contained in:
Sudeep Mohanty
2026-04-28 11:51:55 +02:00
parent 783e86d9a1
commit ffd41dc3e1
2 changed files with 24 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files
components/ulp/lp_core/lp_core_uart.c
+3 -3
View File
@@ -34,9 +34,9 @@ static esp_err_t lp_core_uart_param_config(const lp_core_uart_cfg_t *cfg)
esp_err_t ret = ESP_OK; esp_err_t ret = ESP_OK;
/* Argument sanity check */ /* Argument sanity check */
if ((cfg->uart_proto_cfg.rx_flow_ctrl_thresh > SOC_LP_UART_FIFO_LEN) || if ((cfg->uart_proto_cfg.rx_flow_ctrl_thresh >= SOC_LP_UART_FIFO_LEN) ||
(cfg->uart_proto_cfg.flow_ctrl > UART_HW_FLOWCTRL_MAX) || (cfg->uart_proto_cfg.flow_ctrl >= UART_HW_FLOWCTRL_MAX) ||
(cfg->uart_proto_cfg.data_bits > UART_DATA_BITS_MAX)) { (cfg->uart_proto_cfg.data_bits >= UART_DATA_BITS_MAX)) {
// Invalid config // Invalid config
return ESP_ERR_INVALID_ARG; return ESP_ERR_INVALID_ARG;
} }
components/ulp/test_apps/lp_core/lp_core_basic_tests/main/test_lp_core_uart.c
+21 -1
View File
@@ -1,5 +1,5 @@
/* /*
* SPDX-FileCopyrightText: 2024-2025 Espressif Systems (Shanghai) CO LTD * SPDX-FileCopyrightText: 2024-2026 Espressif Systems (Shanghai) CO LTD
* *
* SPDX-License-Identifier: Apache-2.0 * SPDX-License-Identifier: Apache-2.0
*/ */
@@ -79,6 +79,26 @@ TEST_CASE("LP-Core LP-UART initialization test", "[lp_core]")
#endif /* !SOC_LP_GPIO_MATRIX_SUPPORTED */ #endif /* !SOC_LP_GPIO_MATRIX_SUPPORTED */
} }
TEST_CASE("LP-Core LP-UART data_bits parameter validation", "[lp_core]")
{
/* Valid data_bits values (UART_DATA_5_BITS=0 through UART_DATA_8_BITS=3) must succeed */
ESP_LOGI(TAG, "Verifying LP UART data_bits valid range");
const uart_word_length_t valid_bits[] = {
UART_DATA_5_BITS, UART_DATA_6_BITS, UART_DATA_7_BITS, UART_DATA_8_BITS,
};
for (int i = 0; i < (int)(sizeof(valid_bits) / sizeof(valid_bits[0])); i++) {
lp_core_uart_cfg_t cfg = LP_CORE_UART_DEFAULT_CONFIG();
cfg.uart_proto_cfg.data_bits = valid_bits[i];
TEST_ASSERT_EQUAL(ESP_OK, lp_core_uart_init(&cfg));
}
/* UART_DATA_BITS_MAX (=4) is not a valid word length and must be rejected */
ESP_LOGI(TAG, "Verifying LP UART data_bits = UART_DATA_BITS_MAX is rejected");
lp_core_uart_cfg_t cfg_max = LP_CORE_UART_DEFAULT_CONFIG();
cfg_max.uart_proto_cfg.data_bits = UART_DATA_BITS_MAX;
TEST_ASSERT_NOT_EQUAL(ESP_OK, lp_core_uart_init(&cfg_max));
}
/* LP UART default config */ /* LP UART default config */
static lp_core_uart_cfg_t lp_uart_cfg = LP_CORE_UART_DEFAULT_CONFIG(); static lp_core_uart_cfg_t lp_uart_cfg = LP_CORE_UART_DEFAULT_CONFIG();