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

ESP32S3 USB external PHY pinout (IDFGH-7761) #9302

Merged
merged 1 commit into from
Jul 11, 2022
Merged

ESP32S3 USB external PHY pinout (IDFGH-7761) #9302

merged 1 commit into from
Jul 11, 2022

Conversation

Lsitar
Copy link
Contributor

@Lsitar Lsitar commented Jul 6, 2022

Change USB pinout for ESP32-S3 as in Reference Manual Figure 29-3.
Now the pinout causes crash and uses pins non existing in ESP32-S3. The connection for external PHY is not well described, pinout seems not to be user configurable and the only place where it is documented is Reference Manual chapter 29.
I use internal PHY for JTAG and external for USB CDC simultaneously, after this pinout change it works ok.

@espressif-bot espressif-bot added the Status: Opened Issue is new label Jul 6, 2022
@github-actions github-actions bot changed the title ESP32S3 USB external PHY pinout ESP32S3 USB external PHY pinout (IDFGH-7761) Jul 6, 2022
@Alvin1Zhang
Copy link
Collaborator

Thanks for your contribution.

@suda-morris suda-morris added the PR-Sync-Merge Pull request sync as merge commit label Jul 7, 2022
@suda-morris
Copy link
Collaborator

sha=9ceff23c6d50b37a9fba4c370cffa1654fedeecd

@espressif-bot espressif-bot added Status: In Progress Work is in progress Status: Reviewing Issue is being reviewed Resolution: NA Issue resolution is unavailable Status: Done Issue is done internally Resolution: Done Issue is done internally and removed Status: Opened Issue is new Status: In Progress Work is in progress Status: Reviewing Issue is being reviewed Resolution: NA Issue resolution is unavailable labels Jul 7, 2022
@espressif-bot espressif-bot merged commit 9ceff23 into espressif:master Jul 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR-Sync-Merge Pull request sync as merge commit Resolution: Done Issue is done internally Status: Done Issue is done internally
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants