From bff0335ac2dbe94cc21bd6fdc16296533577b299 Mon Sep 17 00:00:00 2001 From: Alexey Zholtikov Date: Sat, 12 Sep 2026 12:27:18 +0300 Subject: [PATCH] style: updated comments --- README.md | 4 ---- zh_ttp229.c | 12 ++++++------ 2 files changed, 6 insertions(+), 10 deletions(-) diff --git a/README.md b/README.md index 0f5ff2d..e36d805 100644 --- a/README.md +++ b/README.md @@ -1,9 +1,5 @@ # ESP32 ESP-IDF component for TTP229 touch pad (via RMT driver) -## SAST Tools - -[PVS-Studio](https://pvs-studio.com/pvs-studio/?utm_source=website&utm_medium=github&utm_campaign=open_source) - static analyzer for C, C++, C#, and Java code. - ## Features 1. Support for 8-pad and 16-pad TTP229 configurations. diff --git a/zh_ttp229.c b/zh_ttp229.c index 9ab5d28..6b65b86 100644 --- a/zh_ttp229.c +++ b/zh_ttp229.c @@ -154,7 +154,7 @@ static bool _zh_ttp229_rmt_rx_done_callback(rmt_channel_handle_t channel, const ESP_EVENT_DEFINE_BASE(ZH_TTP229); -esp_err_t zh_ttp229_init(const zh_ttp229_init_config_t *config, zh_ttp229_handle_t **handle) // -V2008 +esp_err_t zh_ttp229_init(const zh_ttp229_init_config_t *config, zh_ttp229_handle_t **handle) { ZH_LOGI("Touch pad initialization started."); ZH_ERROR_CHECK(config != NULL && handle != NULL, ESP_ERR_INVALID_ARG, NULL, "Touch pad initialization failed. Invalid argument."); @@ -189,7 +189,7 @@ esp_err_t zh_ttp229_init(const zh_ttp229_init_config_t *config, zh_ttp229_handle return ESP_OK; } -esp_err_t zh_ttp229_deinit(zh_ttp229_handle_t **handle) // -V2008 +esp_err_t zh_ttp229_deinit(zh_ttp229_handle_t **handle) { ZH_LOGI("Touch pad deinitialization started."); ZH_ERROR_CHECK(handle != NULL && *handle != NULL, ESP_ERR_INVALID_ARG, NULL, "Touch pad deinitialization failed. Invalid argument."); @@ -234,7 +234,7 @@ void zh_ttp229_reset_stats(void) ZH_LOGI("Error statistic reset successfully."); } -static esp_err_t _zh_ttp229_validate_config(const zh_ttp229_init_config_t *config, zh_ttp229_handle_t *handle) // -V2008 +static esp_err_t _zh_ttp229_validate_config(const zh_ttp229_init_config_t *config, zh_ttp229_handle_t *handle) { ZH_ERROR_CHECK(config->task_priority >= 1 && config->stack_size >= configMINIMAL_STACK_SIZE, ESP_ERR_INVALID_ARG, NULL, "Invalid task settings."); ZH_ERROR_CHECK(config->queue_size >= 1, ESP_ERR_INVALID_ARG, NULL, "Invalid queue size."); @@ -253,7 +253,7 @@ static esp_err_t _zh_ttp229_validate_config(const zh_ttp229_init_config_t *confi return ESP_OK; } -static esp_err_t _zh_ttp229_gpio_init(const zh_ttp229_init_config_t *config, zh_ttp229_handle_t *handle) // -V2008 +static esp_err_t _zh_ttp229_gpio_init(const zh_ttp229_init_config_t *config, zh_ttp229_handle_t *handle) { ZH_ERROR_CHECK(config->scl_gpio < GPIO_NUM_MAX && config->sdo_gpio < GPIO_NUM_MAX, ESP_ERR_INVALID_ARG, NULL, "Invalid GPIO number.") ZH_ERROR_CHECK(config->scl_gpio != config->sdo_gpio, ESP_ERR_INVALID_ARG, NULL, "SCL GPIO and SDO GPIO is same.") @@ -271,7 +271,7 @@ static esp_err_t _zh_ttp229_gpio_init(const zh_ttp229_init_config_t *config, zh_ return ESP_OK; } -static esp_err_t _zh_ttp229_rmt_init(const zh_ttp229_init_config_t *config, zh_ttp229_handle_t *handle) // -V2008 +static esp_err_t _zh_ttp229_rmt_init(const zh_ttp229_init_config_t *config, zh_ttp229_handle_t *handle) { rmt_tx_channel_config_t tx_chan_config = { .clk_src = RMT_CLK_SRC_DEFAULT, @@ -376,7 +376,7 @@ static void IRAM_ATTR _zh_ttp229_isr_handler(void *arg) portYIELD_FROM_ISR(); } -static void IRAM_ATTR _zh_ttp229_isr_processing_task(void *pvParameter) // -V2008 +static void IRAM_ATTR _zh_ttp229_isr_processing_task(void *pvParameter) { (void)pvParameter; zh_ttp229_queue_t ttp229_queue = {0};