Skip to content

Comments

boards/xtensa/esp32: add three missing defconfigs#18406

Merged
simbit18 merged 5 commits intoapache:masterfrom
fdcavalcanti:feature/add-esp32-defconfigs
Feb 20, 2026
Merged

boards/xtensa/esp32: add three missing defconfigs#18406
simbit18 merged 5 commits intoapache:masterfrom
fdcavalcanti:feature/add-esp32-defconfigs

Conversation

@fdcavalcanti
Copy link
Contributor

Summary

This PR adds three new defconfigs to esp32-devkitc: gpio, i2c and spi. Those are available on other boards but were missing from the esp32-devkitc.

  • documentation: update ESP32 docs for spi, gpio and i2c

Adds description for the GPIO, SPI and I2C defconfigs for the
esp32-devkitc board.

  • board/xtensa/esp32: add SPI defconfig

Add SPI defconfig for ESP32-DevKit-C.

  • board/xtensa/esp32: add GPIO defconfig

Add a GPIO defconfig for ESP32-DevKit-C.

  • boards/xtensa/esp32: add I2C defconfig and update board source

Add I2C defconfig to esp32-devkitc board and update some board
ifdefs.

  • arch/xtensa/esp32: select I2C_DRIVER on I2C

Automatically select I2C_DRIVER when ESP32_I2C is selected.

Impact

Impact on user: New defconfigs available: gpio, spi, i2c.

Impact on build: No.

Impact on hardware: Only ESP32 devices.

Impact on documentation: Adds defconfig description on the esp32-devkitc page.

Impact on security: No.

Impact on compatibility: No.

Testing

Building

Build each of the new defconfigs and flash the board:

./tools/configure.sh esp32-devkitc:spi
./tools/configure.sh esp32-devkitc:i2c
./tools/configure.sh esp32-devkitc:gpio

Running

Merge and flash as usual.

Results

  • SPI:
nsh> ls /dev
/dev:
 console
 null
 spi2
 ttyS0
 zero

nsh> spi bus
 BUS   EXISTS?
Bus 2: YES
Bus 3: NO
  • GPIO:
nsh> ls /dev
/dev:
 console
 gpio0
 gpio1
 gpio2
 null
 ttyS0
 zero
nsh> gpio -o 1 /dev/gpio0
Driver: /dev/gpio0
  Output pin:    Value=0
  Writing:       Value=1
  Verify:        Value=1
  • I2C:
nsh> ls /dev
/dev:
 console
 i2c0
 null
 ttyS0
 zero
nsh> i2c bus
 BUS   EXISTS?
Bus 0: YES
Bus 1: NO
Bus 2: NO
Bus 3: NO

Automatically select I2C_DRIVER when ESP32_I2C is selected.

Signed-off-by: Filipe Cavalcanti <filipe.cavalcanti@espressif.com>
@github-actions github-actions bot added Area: Documentation Improvements or additions to documentation Arch: xtensa Issues related to the Xtensa architecture Size: M The size of the change in this PR is medium Board: xtensa labels Feb 18, 2026
simbit18
simbit18 previously approved these changes Feb 18, 2026
acassis
acassis previously approved these changes Feb 18, 2026
Add I2C defconfig to esp32-devkitc board and update some board
ifdefs.

Signed-off-by: Filipe Cavalcanti <filipe.cavalcanti@espressif.com>
Add a GPIO defconfig for ESP32-DevKit-C.

Signed-off-by: Filipe Cavalcanti <filipe.cavalcanti@espressif.com>
Add SPI defconfig for ESP32-DevKit-C.

Signed-off-by: Filipe Cavalcanti <filipe.cavalcanti@espressif.com>
Adds description for the GPIO, SPI and I2C defconfigs for the
esp32-devkitc board.

Signed-off-by: Filipe Cavalcanti <filipe.cavalcanti@espressif.com>
@linguini1
Copy link
Contributor

Hi @fdcavalcanti , try running ./tools/refresh.sh --defaults on the modified boards (I forget the syntax exactly but ./tools/refresh.sh --help will tell you) to normalize the configs. That should fix the CI errors

linguini1
linguini1 previously approved these changes Feb 19, 2026
@fdcavalcanti fdcavalcanti dismissed stale reviews from linguini1 and acassis via b571a26 February 19, 2026 11:42
@fdcavalcanti fdcavalcanti force-pushed the feature/add-esp32-defconfigs branch from c60939e to b571a26 Compare February 19, 2026 11:42
@github-actions github-actions bot added Area: Audio and removed Area: Documentation Improvements or additions to documentation labels Feb 19, 2026
@fdcavalcanti
Copy link
Contributor Author

Hi @fdcavalcanti , try running ./tools/refresh.sh --defaults on the modified boards (I forget the syntax exactly but ./tools/refresh.sh --help will tell you) to normalize the configs. That should fix the CI errors

Yes I did it but pushed to the wrong fork...
Should work now!

@simbit18 simbit18 merged commit 29858d3 into apache:master Feb 20, 2026
50 of 55 checks passed
@acassis
Copy link
Contributor

acassis commented Feb 20, 2026

@fdcavalcanti nevermind, I found your Documentation :-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Arch: xtensa Issues related to the Xtensa architecture Area: Audio Board: xtensa Size: M The size of the change in this PR is medium

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants