Merge pull request #217 from espressif/chore/esp_idf_pre_commit_all_files

chore: run esp-idf pre-commit on all files
    - astyle_py: max-continuation-indent
    - kconfig check pre-commit
    - cmake list linter

Author: peter-marcisovsky

.pre-commit-config.yaml

@@ -3,7 +3,7 @@ repos:
     rev: v1.0.5
     hooks:
     -   id: astyle_py
-        args: ['--style=otbs', '--attach-namespaces', '--attach-classes', '--indent=spaces=4', '--convert-tabs', '--align-pointer=name', '--align-reference=name', '--keep-one-line-statements', '--pad-header', '--pad-oper']
+        args: ['--style=otbs', '--attach-namespaces', '--attach-classes', '--indent=spaces=4', '--convert-tabs', '--align-pointer=name', '--align-reference=name', '--keep-one-line-statements', '--pad-header', '--pad-oper', '--max-continuation-indent=120']
 
 -   repo: https://github.com/codespell-project/codespell
     rev: v2.2.6
@@ -29,3 +29,21 @@ repos:
         args: ['--fix=lf']
         description: Forces to replace line ending by the UNIX 'lf' character
         exclude: '.*\.uxf'
+    -   id: check-yaml
+
+- repo: https://github.com/espressif/esp-idf-kconfig.git
+  rev: v2.5.0
+  hooks:
+    - id: check-kconfig-files
+    - id: check-deprecated-kconfig-options
+
+- repo: local
+  hooks:
+      - id: cmake-lint
+        name: Check CMake Files Format
+        entry: cmakelint --linelength=120 --spaces=4 --filter=-whitespace/indent
+        language: python
+        additional_dependencies:
+          - cmakelint==1.4.1
+        files: 'CMakeLists.txt$|\.cmake$'
+        exclude: '\/third_party\/'

device/esp_tinyusb/Kconfig

@@ -188,22 +188,22 @@ menu "TinyUSB Stack"
 
         menu "TinyUSB FAT Format Options"
             choice TINYUSB_FAT_FORMAT_TYPE
-               prompt "FatFS Format Type"
+                prompt "FatFS Format Type"
                 default TINYUSB_FAT_FORMAT_ANY
                 help
                     Select the FAT filesystem type used when formatting storage.
 
-            config TINYUSB_FAT_FORMAT_ANY
-                bool "FM_ANY - Automatically select from FAT12/FAT16/FAT32"
+                config TINYUSB_FAT_FORMAT_ANY
+                    bool "FM_ANY - Automatically select from FAT12/FAT16/FAT32"
 
-            config TINYUSB_FAT_FORMAT_FAT
-                bool "FM_FAT - Allow only FAT12/FAT16"
+                config TINYUSB_FAT_FORMAT_FAT
+                    bool "FM_FAT - Allow only FAT12/FAT16"
 
-            config TINYUSB_FAT_FORMAT_FAT32
-                bool "FM_FAT32 - Force FAT32 only"
+                config TINYUSB_FAT_FORMAT_FAT32
+                    bool "FM_FAT32 - Force FAT32 only"
 
-            config TINYUSB_FAT_FORMAT_EXFAT
-                bool "FM_EXFAT - Force exFAT (requires exFAT enabled)"
+                config TINYUSB_FAT_FORMAT_EXFAT
+                    bool "FM_EXFAT - Force exFAT (requires exFAT enabled)"
 
             endchoice
 
@@ -213,7 +213,7 @@ menu "TinyUSB Stack"
                 help
                     Format as a Super Floppy Disk (no partition table).
                     This is typical for USB flash drives and small volumes.
-            endmenu
+        endmenu
 
     endmenu # "Massive Storage Class"
 
@@ -348,23 +348,23 @@ menu "TinyUSB Stack"
             default 3200
             range 1600 10240
             help
-              Size of NTB buffers on the reception side. The minimum size used by Linux is 2048 bytes.
-              NTB buffer size must be significantly larger than the MTU (Maximum Transmission Unit).
-              The typical default MTU size for Ethernet is 1500 bytes, plus an additional packet overhead.
-              To improve performance, the NTB buffer size should be large enough to fit multiple MTU-sized
-              frames in a single NTB buffer and it's length should be multiple of 4.
+                Size of NTB buffers on the reception side. The minimum size used by Linux is 2048 bytes.
+                NTB buffer size must be significantly larger than the MTU (Maximum Transmission Unit).
+                The typical default MTU size for Ethernet is 1500 bytes, plus an additional packet overhead.
+                To improve performance, the NTB buffer size should be large enough to fit multiple MTU-sized
+                frames in a single NTB buffer and it's length should be multiple of 4.
 
         config TINYUSB_NCM_IN_NTB_BUFF_MAX_SIZE
             int "NCM NTB Buffer size for transmission size"
             depends on TINYUSB_NET_MODE_NCM
             default 3200
             range 1600 10240
             help
-              Size of NTB buffers on the transmission side. The minimum size used by Linux is 2048 bytes.
-              NTB buffer size must be significantly larger than the MTU (Maximum Transmission Unit).
-              The typical default MTU size for Ethernet is 1500 bytes, plus an additional packet overhead.
-              To improve performance, the NTB buffer size should be large enough to fit multiple MTU-sized
-              frames in a single NTB buffer and it's length should be multiple of 4.
+                Size of NTB buffers on the transmission side. The minimum size used by Linux is 2048 bytes.
+                NTB buffer size must be significantly larger than the MTU (Maximum Transmission Unit).
+                The typical default MTU size for Ethernet is 1500 bytes, plus an additional packet overhead.
+                To improve performance, the NTB buffer size should be large enough to fit multiple MTU-sized
+                frames in a single NTB buffer and it's length should be multiple of 4.
 
     endmenu # "Network driver (ECM/NCM/RNDIS)"
 

device/esp_tinyusb/include/tusb_cdc_acm.h

@@ -1,5 +1,5 @@
 /*
- * SPDX-FileCopyrightText: 2020-2023 Espressif Systems (Shanghai) CO LTD
+ * SPDX-FileCopyrightText: 2020-2025 Espressif Systems (Shanghai) CO LTD
  *
  * SPDX-License-Identifier: Apache-2.0
  */
@@ -130,8 +130,8 @@ esp_err_t tusb_cdc_acm_deinit(int itf);
  * @return esp_err_t - ESP_OK or ESP_ERR_INVALID_ARG
  */
 esp_err_t tinyusb_cdcacm_register_callback(tinyusb_cdcacm_itf_t itf,
-        cdcacm_event_type_t event_type,
-        tusb_cdcacm_callback_t callback);
+                                           cdcacm_event_type_t event_type,
+                                           tusb_cdcacm_callback_t callback);
 
 /**
  * @brief Unregister a callback invoking on CDC event

device/esp_tinyusb/test/local/libusb_test.c

@@ -1,5 +1,5 @@
 /*
- * SPDX-FileCopyrightText: 2024 Espressif Systems (Shanghai) CO LTD
+ * SPDX-FileCopyrightText: 2024-2025 Espressif Systems (Shanghai) CO LTD
  *
  * SPDX-License-Identifier: Apache-2.0
  */
@@ -62,8 +62,8 @@ int main()
     rc = libusb_init(&context);
     assert(rc == 0);
     libusb_device_handle *dev_handle = libusb_open_device_with_vid_pid(context,
-                                       TINYUSB_VENDOR,
-                                       TINYUSB_PRODUCT);
+                                                                       TINYUSB_VENDOR,
+                                                                       TINYUSB_PRODUCT);
 
     if (dev_handle != NULL) {
         printf("TinyUSB Device has been found\n");

device/esp_tinyusb/test_apps/cdc/sdkconfig.defaults

@@ -5,8 +5,7 @@ CONFIG_TINYUSB_CDC_COUNT=2
 CONFIG_TINYUSB_HID_COUNT=0
 
 # Disable watchdogs, they'd get triggered during unity interactive menu
-CONFIG_ESP_INT_WDT=n
-CONFIG_ESP_TASK_WDT=n
+# CONFIG_ESP_TASK_WDT_INIT is not set
 
 # Run-time checks of Heap and Stack
 CONFIG_HEAP_POISONING_COMPREHENSIVE=y

device/esp_tinyusb/test_apps/configuration_desc/sdkconfig.defaults

@@ -3,8 +3,7 @@ CONFIG_TINYUSB_CDC_ENABLED=y
 CONFIG_TINYUSB_CDC_COUNT=2
 
 # Disable watchdogs, they'd get triggered during unity interactive menu
-CONFIG_ESP_INT_WDT=n
-CONFIG_ESP_TASK_WDT=n
+# CONFIG_ESP_TASK_WDT_INIT is not set
 
 # Run-time checks of Heap and Stack
 CONFIG_HEAP_POISONING_COMPREHENSIVE=y

device/esp_tinyusb/test_apps/dconn_detection/sdkconfig.defaults

@@ -5,8 +5,7 @@ CONFIG_TINYUSB_CDC_COUNT=0
 CONFIG_TINYUSB_HID_COUNT=0
 
 # Disable watchdogs, they'd get triggered during unity interactive menu
-CONFIG_ESP_INT_WDT=n
-CONFIG_ESP_TASK_WDT=n
+# CONFIG_ESP_TASK_WDT_INIT is not set
 
 # Run-time checks of Heap and Stack
 CONFIG_HEAP_POISONING_COMPREHENSIVE=y

device/esp_tinyusb/test_apps/default_task/sdkconfig.defaults

@@ -3,8 +3,7 @@ CONFIG_TINYUSB_NO_DEFAULT_TASK=n
 CONFIG_TINYUSB_INIT_IN_DEFAULT_TASK=n
 
 # Disable watchdogs, they'd get triggered during unity interactive menu
-CONFIG_ESP_INT_WDT=n
-CONFIG_ESP_TASK_WDT=n
+# CONFIG_ESP_TASK_WDT_INIT is not set
 
 # Run-time checks of Heap and Stack
 CONFIG_HEAP_POISONING_COMPREHENSIVE=y

device/esp_tinyusb/test_apps/default_task_with_init/sdkconfig.defaults

@@ -3,8 +3,7 @@ CONFIG_TINYUSB_NO_DEFAULT_TASK=n
 CONFIG_TINYUSB_INIT_IN_DEFAULT_TASK=y
 
 # Disable watchdogs, they'd get triggered during unity interactive menu
-CONFIG_ESP_INT_WDT=n
-CONFIG_ESP_TASK_WDT=n
+# CONFIG_ESP_TASK_WDT_INIT is not set
 
 # Run-time checks of Heap and Stack
 CONFIG_HEAP_POISONING_COMPREHENSIVE=y

device/esp_tinyusb/test_apps/external_task/sdkconfig.defaults

@@ -3,8 +3,7 @@ CONFIG_TINYUSB_NO_DEFAULT_TASK=y
 CONFIG_TINYUSB_INIT_IN_DEFAULT_TASK=n
 
 # Disable watchdogs, they'd get triggered during unity interactive menu
-CONFIG_ESP_INT_WDT=n
-CONFIG_ESP_TASK_WDT=n
+# CONFIG_ESP_TASK_WDT_INIT is not set
 
 # Run-time checks of Heap and Stack
 CONFIG_HEAP_POISONING_COMPREHENSIVE=y