Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

boards: add ESP32-S3-USB-OTG support #19808

Merged
merged 2 commits into from
Jul 7, 2023

Conversation

gschorcht
Copy link
Contributor

@gschorcht gschorcht commented Jul 7, 2023

Contribution description

This PR provides the support for the ESP32-S3-USB-OTG board.

The SD card can only be used in SPI mode at the moment. As soon as PR #19786 is merged, the SD/MMC support can be enabled and the SD card can then be used in SD mode with 4-bit data bus width.

The display uses uses the ST7789 driver IC that is compatible with the ST7735. For that purpose the ST7735 driver is extended by a pseudomodule definition st7789 for the ST7789 which is enabled by the board and enables automatically the st7789 (e57c48a). Vise versa, board's Makefile.dep enables automatically the st7789 pseudomodule if the st7735 is used. The pseudomodule st7789 is just used to increase the upper limit for supported lines. This change is also part of PR #19807.

The PR includes a very small documentation fix for ESP32-S3-DevKit board that was noticed during the development of this board definition (1155b6a).

Testing procedure

The board has been tested with all basic tests for supported hardware including tests/driver/st3375:

IMG_20230707_105556

  • tests/drivers/st3375
  • tests/periph/gpio
  • tests/periph/pwm
  • tests/periph/spi
  • tests/periph/uart
  • tests/sys/usbus_cdc_ecm

Issues/PRs references

@github-actions github-actions bot added Area: doc Area: Documentation Area: drivers Area: Device drivers Area: boards Area: Board ports Area: Kconfig Area: Kconfig integration labels Jul 7, 2023
@gschorcht gschorcht added Type: new feature The issue requests / The PR implemements a new feature for RIOT CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR labels Jul 7, 2023
ST7735 driver can also be used for ST7789. st7789 is simply defined as pseudomodule that pulls in automatically module st7735.
@gschorcht gschorcht force-pushed the boards/esp32s3-usb-otg branch from edcfcae to 2bf4155 Compare July 7, 2023 09:17
@riot-ci
Copy link

riot-ci commented Jul 7, 2023

Murdock results

✔️ PASSED

d68c826 boards: add support for ESP32-S3-USB-OTG

Success Failures Total Runtime
6936 0 6936 10m:11s

Artifacts

Copy link
Contributor

@benpicco benpicco left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please squash

@gschorcht gschorcht force-pushed the boards/esp32s3-usb-otg branch from 51fd859 to d2b9efd Compare July 7, 2023 12:56
@benpicco
Copy link
Contributor

benpicco commented Jul 7, 2023

There is still a fixup commit 👀

@gschorcht gschorcht force-pushed the boards/esp32s3-usb-otg branch from d2b9efd to d68c826 Compare July 7, 2023 13:34
@gschorcht
Copy link
Contributor Author

There is still a fixup commit eyes

Yes, I had to fix a further Kconfig issue.

@benpicco
Copy link
Contributor

benpicco commented Jul 7, 2023

bors merge

@bors
Copy link
Contributor

bors bot commented Jul 7, 2023

Build succeeded!

The publicly hosted instance of bors-ng is deprecated and will go away soon.

If you want to self-host your own instance, instructions are here.
For more help, visit the forum.

If you want to switch to GitHub's built-in merge queue, visit their help page.

@bors bors bot merged commit 72ca81b into RIOT-OS:master Jul 7, 2023
@benpicco benpicco added this to the Release 2023.07 milestone Aug 2, 2023
@gschorcht gschorcht deleted the boards/esp32s3-usb-otg branch September 3, 2023 09:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: boards Area: Board ports Area: doc Area: Documentation Area: drivers Area: Device drivers Area: Kconfig Area: Kconfig integration CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Type: new feature The issue requests / The PR implemements a new feature for RIOT
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants