espressif: fix Kconfig style issues

Fix Kconfig style issues in Espressif files. Signed-off-by: Sylvio Alves <sylvio.alves@espressif.com>
2025-08-21 09:00:22 -03:00
parent ba4bb91b1b
commit 34eb58c043
9 changed files with 126 additions and 102 deletions
--- a/drivers/adc/Kconfig.esp32
+++ b/drivers/adc/Kconfig.esp32
@@ -13,7 +13,6 @@ if ADC_ESP32

 config ADC_ESP32_DMA
 	bool "ESP32 ADC DMA Support"
-	default n
 	depends on DT_HAS_ESPRESSIF_ESP32_GDMA_ENABLED
 	help
 	  Enable the ADC DMA mode for ADC instances
--- a/drivers/input/Kconfig.esp32
+++ b/drivers/input/Kconfig.esp32
@@ -3,7 +3,6 @@

 config INPUT_ESP32_TOUCH_SENSOR
 	bool "Espressif Touch Sensor input driver"
-	default n
 	depends on DT_HAS_ESPRESSIF_ESP32_TOUCH_ENABLED
 	help
 	  Enable support for Espressif Touch Sensor input driver.
--- a/soc/espressif/common/Kconfig.esptool
+++ b/soc/espressif/common/Kconfig.esptool
@@ -6,7 +6,6 @@ if SOC_FAMILY_ESPRESSIF_ESP32
 config ESPTOOLPY_OCT_FLASH
 	bool "Use Octal Flash"
 	depends on SOC_SERIES_ESP32S3
-	default n

 config ESPTOOLPY_FLASH_MODE_AUTO_DETECT
 	depends on SOC_SERIES_ESP32S3
@@ -38,15 +37,19 @@ choice ESPTOOLPY_FLASHMODE
 config ESPTOOLPY_FLASHMODE_QIO
 	bool "QIO"
 	depends on !ESPTOOLPY_OCT_FLASH
+
 config ESPTOOLPY_FLASHMODE_QOUT
 	bool "QOUT"
 	depends on !ESPTOOLPY_OCT_FLASH
+
 config ESPTOOLPY_FLASHMODE_DIO
 	bool "DIO"
 	depends on !ESPTOOLPY_OCT_FLASH
+
 config ESPTOOLPY_FLASHMODE_DOUT
 	bool "DOUT"
 	depends on !ESPTOOLPY_OCT_FLASH
+
 config ESPTOOLPY_FLASHMODE_OPI
 	bool "OPI"
 	depends on ESPTOOLPY_OCT_FLASH
@@ -60,6 +63,7 @@ choice ESPTOOLPY_FLASH_SAMPLE_MODE

 config ESPTOOLPY_FLASH_SAMPLE_MODE_STR
 	bool "STR Mode"
+
 config ESPTOOLPY_FLASH_SAMPLE_MODE_DTR
 	bool "DTR Mode"
 	depends on ESPTOOLPY_OCT_FLASH
@@ -89,24 +93,28 @@ config ESPTOOLPY_FLASHFREQ_120M
 	select SPI_FLASH_HPM_ENABLE
 	depends on SOC_SERIES_ESP32S3 && ESPTOOLPY_FLASH_SAMPLE_MODE_STR
 	help
-		- Flash 120 MHz SDR mode is stable.
-		- Flash 120 MHz DDR mode is an experimental feature, it works when
-		the temperature is stable.
+	  - Flash 120 MHz SDR mode is stable.
+	  - Flash 120 MHz DDR mode is an experimental feature, it works when
+	  the temperature is stable.

-		Risks:
-		If your chip powers on at a certain temperature, then after the temperature
-		increases or decreases by approximately 20 Celsius degrees (depending on the
-		chip), the program will crash randomly.
+	  Risks:
+	  If your chip powers on at a certain temperature, then after the temperature
+	  increases or decreases by approximately 20 Celsius degrees (depending on the
+	  chip), the program will crash randomly.

 config ESPTOOLPY_FLASHFREQ_80M
 	bool "80 MHz"
+
 config ESPTOOLPY_FLASHFREQ_60M
 	bool "60 MHz"
+
 config ESPTOOLPY_FLASHFREQ_40M
 	bool "40 MHz"
+
 config ESPTOOLPY_FLASHFREQ_26M
 	bool "26 MHz"
 	depends on !SOC_SERIES_ESP32S3
+
 config ESPTOOLPY_FLASHFREQ_20M
 	bool "20 MHz"

@@ -136,22 +144,30 @@ choice ESPTOOLPY_FLASHSIZE
 	help
 	  SPI flash size, in megabytes

-	config ESPTOOLPY_FLASHSIZE_1MB
-		bool "1 MB"
-	config ESPTOOLPY_FLASHSIZE_2MB
-		bool "2 MB"
-	config ESPTOOLPY_FLASHSIZE_4MB
-		bool "4 MB"
-	config ESPTOOLPY_FLASHSIZE_8MB
-		bool "8 MB"
-	config ESPTOOLPY_FLASHSIZE_16MB
-		bool "16 MB"
-	config ESPTOOLPY_FLASHSIZE_32MB
-		bool "32 MB"
-	config ESPTOOLPY_FLASHSIZE_64MB
-		bool "64 MB"
-	config ESPTOOLPY_FLASHSIZE_128MB
-		bool "128 MB"
+config ESPTOOLPY_FLASHSIZE_1MB
+	bool "1 MB"
+
+config ESPTOOLPY_FLASHSIZE_2MB
+	bool "2 MB"
+
+config ESPTOOLPY_FLASHSIZE_4MB
+	bool "4 MB"
+
+config ESPTOOLPY_FLASHSIZE_8MB
+	bool "8 MB"
+
+config ESPTOOLPY_FLASHSIZE_16MB
+	bool "16 MB"
+
+config ESPTOOLPY_FLASHSIZE_32MB
+	bool "32 MB"
+
+config ESPTOOLPY_FLASHSIZE_64MB
+	bool "64 MB"
+
+config ESPTOOLPY_FLASHSIZE_128MB
+	bool "128 MB"
+
 endchoice

 config ESPTOOLPY_FLASHSIZE
@@ -167,7 +183,6 @@ config ESPTOOLPY_FLASHSIZE

 config ESPTOOLPY_HEADER_FLASHSIZE_UPDATE
 	bool "Detect flash size when flashing bootloader"
-	default n
 	help
 	  If this option is set, flashing the project will automatically detect
 	  the flash size of the target chip and update the bootloader image
@@ -187,10 +202,12 @@ choice ESPTOOLPY_BEFORE
 	  wired from the serial port to the ESP32. Most USB development
 	  boards do this internally.

-	config ESPTOOLPY_BEFORE_RESET
-	  bool "Reset to bootloader"
-	config ESPTOOLPY_BEFORE_NORESET
-	  bool "No reset"
+config ESPTOOLPY_BEFORE_RESET
+	bool "Reset to bootloader"
+
+config ESPTOOLPY_BEFORE_NORESET
+	bool "No reset"
+
 endchoice # ESPTOOLPY_BEFORE

 config ESPTOOLPY_BEFORE
@@ -203,15 +220,16 @@ choice ESPTOOLPY_AFTER
 	default ESPTOOLPY_AFTER_RESET
 	help
 	  Configure whether esptool.py should reset the ESP32 after flashing.
-
 	  Automatic resetting depends on the RTS & DTR signals being
 	  wired from the serial port to the ESP32. Most USB development
 	  boards do this internally.

-	config ESPTOOLPY_AFTER_RESET
-	  bool "Reset after flashing"
-	config ESPTOOLPY_AFTER_NORESET
-	  bool "Stay in bootloader"
+config ESPTOOLPY_AFTER_RESET
+	bool "Reset after flashing"
+
+config ESPTOOLPY_AFTER_NORESET
+	bool "Stay in bootloader"
+
 endchoice

 config ESPTOOLPY_AFTER
--- a/soc/espressif/common/Kconfig.flash
+++ b/soc/espressif/common/Kconfig.flash
@@ -122,7 +122,6 @@ config BOOTLOADER_REGION_PROTECTION_ENABLE
 config SPI_FLASH_HPM_ENABLE
 	bool
 	depends on SOC_SERIES_ESP32S3
-	default n
 	help
 	  This option is invisible, and will be selected automatically
 	  when ``ESPTOOLPY_FLASHFREQ_120M`` is selected.
--- a/soc/espressif/common/Kconfig.spiram
+++ b/soc/espressif/common/Kconfig.spiram
@@ -34,12 +34,12 @@ choice SPIRAM_MODE
 	prompt "Mode (QUAD/OCT) of SPI RAM chip in use"
 	default SPIRAM_MODE_QUAD

-	config SPIRAM_MODE_QUAD
-		bool "Quad Mode PSRAM"
+config SPIRAM_MODE_QUAD
+	bool "Quad Mode PSRAM"

-	config SPIRAM_MODE_OCT
-		bool "Octal Mode PSRAM"
-		depends on SOC_SERIES_ESP32S3
+config SPIRAM_MODE_OCT
+	bool "Octal Mode PSRAM"
+	depends on SOC_SERIES_ESP32S3

 endchoice # SPIRAM_MODE

@@ -117,7 +117,6 @@ config SPIRAM_CS_IO
 config SPIRAM_FETCH_INSTRUCTIONS
 	bool "Move Instructions in Flash to PSRAM"
 	depends on SOC_SERIES_ESP32S2 || SOC_SERIES_ESP32S3
-	default n
 	help
 	  If enabled, instructions in flash will be moved into PSRAM on startup.
 	  If SPIRAM_RODATA is also enabled, code that requires execution during an SPI1 Flash operation
@@ -127,7 +126,6 @@ config SPIRAM_FETCH_INSTRUCTIONS
 config SPIRAM_RODATA
 	bool "Move Read-Only Data in Flash to PSRAM"
 	depends on SOC_SERIES_ESP32S2 || SOC_SERIES_ESP32S3
-	default n
 	help
 	  If enabled, rodata in flash will be moved into PSRAM on startup.
 	  If SPIRAM_FETCH_INSTRUCTIONS is also enabled, code that requires execution during an SPI1 Flash operation
@@ -136,7 +134,6 @@ config SPIRAM_RODATA

 config SPIRAM_ECC_ENABLE
 	bool "Allow enabling SPI RAM ECC"
-	default n
 	depends on SPIRAM_MODE_OCT && SOC_SERIES_ESP32S3
 	help
 	  Enable MSPI Error-Correcting Code function when accessing SPIRAM.
@@ -202,7 +199,6 @@ endmenu # PSRAM clock and cs IO for ESP32-PICO
 config SPIRAM_CUSTOM_SPIWP_SD3_PIN
 	bool "Use custom SPI PSRAM WP(SD3) Pin when flash pins set in eFuse (read help)"
 	depends on ESPTOOLPY_FLASHMODE_DIO || ESPTOOLPY_FLASHMODE_DOUT
-	default n
 	help
 	  This setting is only used if the SPI flash pins have been overridden by setting the eFuses
 	  SPI_PAD_CONFIG_xxx, and the SPI flash mode is DIO or DOUT.
--- a/soc/espressif/esp32/Kconfig
+++ b/soc/espressif/esp32/Kconfig
@@ -22,7 +22,6 @@ config ESP32_BT_RESERVE_DRAM

 config ESP32_REGION_1_NOINIT
 	bool "Use DRAM region 1 to spill noinit section"
-	default n
 	help
 	  ESP32 has two banks of size 192K and 128K which can be used
 	  as DRAM. Enabling this option would allocate .noinit sections
@@ -50,7 +49,6 @@ config ESP32_EMAC
 	bool
 	default y if ETH_ESP32
 	default y if MDIO_ESP32
-	default n
 	help
 	  Hidden option to enable the ESP32 Ethernet MAC driver.
 	  Both Ethernet and MDIO depend on this driver.
--- a/soc/espressif/esp32c3/Kconfig
+++ b/soc/espressif/esp32c3/Kconfig
@@ -29,7 +29,6 @@ config SOC_ESP32C3_REV_1_1
 config MAC_BB_PD
 	bool "Power down MAC and baseband of Wi-Fi and Bluetooth when PHY is disabled"
 	depends on SOC_SERIES_ESP32C3 && TICKLESS_KERNEL
-	default n
 	help
 	  If enabled, the MAC and baseband of Wi-Fi and Bluetooth will be powered
 	  down when PHY is disabled. Enabling this setting reduces power consumption
--- a/soc/espressif/esp32s2/Kconfig
+++ b/soc/espressif/esp32s2/Kconfig
@@ -16,27 +16,27 @@ if SOC_SERIES_ESP32S2

 menu "Cache config"

-choice
+choice ESP32S2_INSTRUCTION_CACHE_LINE_SIZE
 	prompt "Instruction cache line size"
 	default ESP32S2_INSTRUCTION_CACHE_LINE_32B

-	config ESP32S2_INSTRUCTION_CACHE_LINE_16B
-		bool "16 Bytes"
+config ESP32S2_INSTRUCTION_CACHE_LINE_16B
+	bool "16 Bytes"

-	config ESP32S2_INSTRUCTION_CACHE_LINE_32B
-		bool "32 Bytes"
+config ESP32S2_INSTRUCTION_CACHE_LINE_32B
+	bool "32 Bytes"

 endchoice

-choice
+choice ESP32S2_INSTRUCTION_CACHE_SIZE
 	prompt "Instruction cache size"
 	default ESP32S2_INSTRUCTION_CACHE_8KB

-	config ESP32S2_INSTRUCTION_CACHE_8KB
-		bool "8KB instruction cache size"
+config ESP32S2_INSTRUCTION_CACHE_8KB
+	bool "8KB instruction cache size"

-	config ESP32S2_INSTRUCTION_CACHE_16KB
-		bool "16KB instruction cache size"
+config ESP32S2_INSTRUCTION_CACHE_16KB
+	bool "16KB instruction cache size"

 endchoice

@@ -44,14 +44,17 @@ choice ESP32S2_DATA_CACHE_SIZE
 	prompt "Data cache size"
 	default ESP32S2_DATA_CACHE_8KB
 	help
-		Data cache size to be set on application startup.
-	config ESP32S2_DATA_CACHE_0KB
-		depends on !ESP_SPIRAM
-		bool "0KB"
-	config ESP32S2_DATA_CACHE_8KB
-		bool "8KB"
-	config ESP32S2_DATA_CACHE_16KB
-		bool "16KB"
+	  Data cache size to be set on application startup.
+
+config ESP32S2_DATA_CACHE_0KB
+	depends on !ESP_SPIRAM
+	bool "0KB"
+
+config ESP32S2_DATA_CACHE_8KB
+	bool "8KB"
+
+config ESP32S2_DATA_CACHE_16KB
+	bool "16KB"

 endchoice

@@ -59,11 +62,11 @@ choice
 	prompt "Data cache line size"
 	default ESP32S2_DATA_CACHE_LINE_32B

-	config ESP32S2_DATA_CACHE_LINE_16B
-		bool "16 Bytes"
+config ESP32S2_DATA_CACHE_LINE_16B
+	bool "16 Bytes"

-	config ESP32S2_DATA_CACHE_LINE_32B
-		bool "32 Bytes"
+config ESP32S2_DATA_CACHE_LINE_32B
+	bool "32 Bytes"

 endchoice

--- a/soc/espressif/esp32s3/Kconfig
+++ b/soc/espressif/esp32s3/Kconfig
@@ -24,10 +24,12 @@ choice ESP32S3_INSTRUCTION_CACHE_SIZE
 	  If you use 16KB instruction cache rather than 32KB instruction cache,
 	  then the other 16KB will be managed by heap allocator.

-	config ESP32S3_INSTRUCTION_CACHE_16KB
-		bool "16KB"
-	config ESP32S3_INSTRUCTION_CACHE_32KB
-		bool "32KB"
+config ESP32S3_INSTRUCTION_CACHE_16KB
+	bool "16KB"
+
+config ESP32S3_INSTRUCTION_CACHE_32KB
+	bool "32KB"
+
 endchoice

 config ESP32S3_INSTRUCTION_CACHE_SIZE
@@ -41,10 +43,12 @@ choice ESP32S3_ICACHE_ASSOCIATED_WAYS
 	help
 	  Instruction cache associated ways to be set on application startup.

-	config ESP32S3_INSTRUCTION_CACHE_4WAYS
-		bool "4 ways"
-	config ESP32S3_INSTRUCTION_CACHE_8WAYS
-		bool "8 ways"
+config ESP32S3_INSTRUCTION_CACHE_4WAYS
+	bool "4 ways"
+
+config ESP32S3_INSTRUCTION_CACHE_8WAYS
+	bool "8 ways"
+
 endchoice

 config ESP32S3_ICACHE_ASSOCIATED_WAYS
@@ -58,11 +62,13 @@ choice ESP32S3_INSTRUCTION_CACHE_LINE_SIZE
 	help
 	  Instruction cache line size to be set on application startup.

-	config ESP32S3_INSTRUCTION_CACHE_LINE_16B
-		bool "16 Bytes"
-		depends on ESP32S3_INSTRUCTION_CACHE_16KB
-	config ESP32S3_INSTRUCTION_CACHE_LINE_32B
-		bool "32 Bytes"
+config ESP32S3_INSTRUCTION_CACHE_LINE_16B
+	bool "16 Bytes"
+	depends on ESP32S3_INSTRUCTION_CACHE_16KB
+
+config ESP32S3_INSTRUCTION_CACHE_LINE_32B
+	bool "32 Bytes"
+
 endchoice

 config ESP32S3_INSTRUCTION_CACHE_LINE_SIZE
@@ -85,12 +91,15 @@ choice ESP32S3_DATA_CACHE_SIZE
 	  If you use 32KB data cache rather than 64KB data cache,
 	  the other 32KB will be added to the heap.

-	config ESP32S3_DATA_CACHE_16KB
-		bool "16KB"
-	config ESP32S3_DATA_CACHE_32KB
-		bool "32KB"
-	config ESP32S3_DATA_CACHE_64KB
-		bool "64KB"
+config ESP32S3_DATA_CACHE_16KB
+	bool "16KB"
+
+config ESP32S3_DATA_CACHE_32KB
+	bool "32KB"
+
+config ESP32S3_DATA_CACHE_64KB
+	bool "64KB"
+
 endchoice

 config ESP32S3_DATA_CACHE_SIZE
@@ -106,10 +115,12 @@ choice ESP32S3_DCACHE_ASSOCIATED_WAYS
 	help
 	  Data cache associated ways to be set on application startup.

-	config ESP32S3_DATA_CACHE_4WAYS
-		bool "4 ways"
-	config ESP32S3_DATA_CACHE_8WAYS
-		bool "8 ways"
+config ESP32S3_DATA_CACHE_4WAYS
+	bool "4 ways"
+
+config ESP32S3_DATA_CACHE_8WAYS
+	bool "8 ways"
+
 endchoice

 config ESP32S3_DCACHE_ASSOCIATED_WAYS
@@ -123,13 +134,16 @@ choice ESP32S3_DATA_CACHE_LINE_SIZE
 	help
 	  Data cache line size to be set on application startup.

-	config ESP32S3_DATA_CACHE_LINE_16B
-		bool "16 Bytes"
-		depends on ESP32S3_DATA_CACHE_16KB || ESP32S3_DATA_CACHE_32KB
-	config ESP32S3_DATA_CACHE_LINE_32B
-		bool "32 Bytes"
-	config ESP32S3_DATA_CACHE_LINE_64B
-		bool "64 Bytes"
+config ESP32S3_DATA_CACHE_LINE_16B
+	bool "16 Bytes"
+	depends on ESP32S3_DATA_CACHE_16KB || ESP32S3_DATA_CACHE_32KB
+
+config ESP32S3_DATA_CACHE_LINE_32B
+	bool "32 Bytes"
+
+config ESP32S3_DATA_CACHE_LINE_64B
+	bool "64 Bytes"
+
 endchoice

 config ESP32S3_DATA_CACHE_LINE_SIZE
@@ -148,7 +162,6 @@ config ESP32S3_DATA_CACHE_WRAP
 config MAC_BB_PD
 	bool "Power down MAC and baseband of Wi-Fi and Bluetooth when PHY is disabled"
 	depends on SOC_SERIES_ESP32S3 && TICKLESS_KERNEL
-	default n
 	help
 	  If enabled, the MAC and baseband of Wi-Fi and Bluetooth will be powered
 	  down when PHY is disabled. Enabling this setting reduces power consumption