Skip to content

Conversation

@xingrz
Copy link
Member

@xingrz xingrz commented Dec 5, 2025

0a096381c116280012d51e49192a4176

This PR adds support for the Waveshare ESP32-S3-GEEK USB dongle.

The board features an ESP32-S3R2 with 2 MB PSRAM, 16 MB on-board flash, a 1.14" LCD, and a microSD slot.

It also introduces an overlay for the cdc_acm_bridge sample, allowing the dongle to be used as a USB-to-serial adapter.

@xingrz xingrz force-pushed the esp32s-s3-geek/pr branch from d9ef400 to 2b26f26 Compare December 5, 2025 14:44
@xingrz xingrz marked this pull request as ready for review December 5, 2025 15:21
status = "okay";
};

zephyr_udc0: &usb_otg {};
Copy link
Contributor

Choose a reason for hiding this comment

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

Just curious: No need for status = "okay"; on this board?
It will not be initialized unless Kconfig also enables USB.
It seems status = "okay"; on i.e. boards/espressif/esp32s3_devkitc/esp32s3_devkitc_procpu.dts.

Copy link
Member Author

Choose a reason for hiding this comment

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

Actually, only one of usb_serial or usb_otg can be enabled at a time because they share the same USB port. On the DevKitC, the USB-to-serial function is provided by the on-board CP2102, so the SoC's internal usb_serial is not used, and the USB-C receptacle is intended for usb_otg.

On this dongle, however, the USB-A plug is the only way to program or access the console unless an external USB-to-serial adapter is attached. For this reason, the USB port is by default used for usb_serial.

Copy link
Contributor

Choose a reason for hiding this comment

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

Very insightful, thank you! I am leaving the thread open for other reviewers.

@josuah
Copy link
Contributor

josuah commented Dec 5, 2025

CI is currently stuck on this:

west build -p -b esp32s3_geek/esp32s3/appcpu tests/subsys/secure_storage/psa/its -T secure_storage.psa.its.secure_storage.store.settings.64-bit_uids

@xingrz
Copy link
Member Author

xingrz commented Dec 5, 2025

@josuah Yes, it's seems broken since #98836. The same issue occurs on esp32s3_devkitc/esp32s3/appcpu too.

@xingrz xingrz force-pushed the esp32s-s3-geek/pr branch 2 times, most recently from e221a9d to 3d2713b Compare December 6, 2025 03:28
Add support for the Waveshare ESP32-S3-GEEK USB dongle. It features an
ESP32-S3R2 with 2 MB PSRAM, a 16 MB on-board Flash, a 1.14" LCD, and a
microSD slot.

Signed-off-by: Chen Xingyu <[email protected]>
Add an overlay to use the Waveshare ESP32-S3-GEEK dongle as a USB-to-serial
adapter.

Signed-off-by: Chen Xingyu <[email protected]>
@xingrz xingrz force-pushed the esp32s-s3-geek/pr branch from 3d2713b to 01d3f21 Compare December 6, 2025 06:42
@sonarqubecloud
Copy link

sonarqubecloud bot commented Dec 6, 2025

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants