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

CIRCUITPY_USB_DEVICE_INSTANCE should default to 0 on most Espressif boards #9889

Merged

Conversation

dhalbert
Copy link
Collaborator

@dhalbert dhalbert commented Dec 15, 2024

#9841 set CIRCUITPY_USB_DEVICE_INSTANCE to 1 as the default for all Espressif boards. However, on nearly all boards, it should be 0, not 1. The ESP32-P4 dev board uses 1 to use the second interface by default.

@tannewt should be it 1 for all ESP32-P4 boards, or does it depend on the board wiring?

Tested that it's fixed on a Feather ESP32-S3 4/2 board.

@dhalbert dhalbert requested a review from tannewt December 15, 2024 23:11
@RetiredWizard
Copy link

I quickly loaded this artifact onto my MatrixPortal S3 and it does bring back the CIRCUITPY drive that didn't appear using the "absolute newest" build, however I ran into another apparently unrelated safemode issue. I'll open a new issue on that shortly.

@tannewt
Copy link
Member

tannewt commented Dec 16, 2024

P4 boards will have the choice to use full speed or high speed USB. Instance 1 is the high speed (480) usb. I was thinking full speed would be usb host.

Copy link
Member

@tannewt tannewt left a comment

Choose a reason for hiding this comment

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

Thank you for the fix!

@tannewt tannewt merged commit 79fd8b7 into adafruit:main Dec 16, 2024
223 checks passed
@dhalbert dhalbert deleted the espressif-CIRCUITPY_USB_DEVICE_INSTANCE branch December 16, 2024 20:32
@dbiersach
Copy link

dbiersach commented Jan 13, 2025

On my Adafruit Metro ESP32-S3 with 16 MB Flash 8 MB PSRAM, all versions after adafruit-circuitpython-adafruit_metro_esp32s3-en_US-20241206-main-PR9873-c41d452.uf2 (#9873) fail to restart CIRCUITPY upon flashing the UF2 and performing a hard reset of the board. I know there was a discussion on Issue #9888, and the thought was that PR9889 had fixed the issue but not for the Metro ESP32-S3 board. Upon flashing the UF2, CIRCUITPY does reappear, but if you reset the board, the volume fails to mount. If you go back to CircuitPython 9.2.1, it will work great. Try to upgrade to CP 9.2.2, but you get this error where the board falls back into JTAG mode. I suspect the underlying bug is not yet fixed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

#9841 has broken ESP32-S2 and ESP32-S3 USB
4 participants