[Do Not Merge] Introduce DWC2_I_WANT_FS_ON_HS_BOARD hack#3450
Closed
ceedriic wants to merge 3 commits intohathach:masterfrom
Closed
[Do Not Merge] Introduce DWC2_I_WANT_FS_ON_HS_BOARD hack#3450ceedriic wants to merge 3 commits intohathach:masterfrom
ceedriic wants to merge 3 commits intohathach:masterfrom
Conversation
Size Difference ReportBecause TinyUSB code size varies by port and configuration, the metrics below represent the averaged totals across all example builds. Note: If there is no change, only one value is shown. Changes >1% in sizeNo entries. Changes <1% in sizeNo entries. No changes
|
hathach
changed the title
Owner
|
why not use the built-in FS PHY instead ? does the mcu only has 1 phy, and that is used by the other usb controller ? |
Contributor
Author
You're right, I could use the built-in FS PHY of the MCU, but the board has been designed with the USB3315C external PHY, to be high-speed capable. So the built-in FS PHY pins are not connected. At the moment, for some reasons I'm trying to understand, high-speed devices do not enumerate on that board with the latest TinyUSB, but full-speed devices work well. So with this patch I'm able to run high-speed devices in full speed mode. |
Contributor
Author
|
Deleted in favour of #3494 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Describe the PR
This is a proof-of-concept patch to support Full-Speed USB on DWC2 on a board with High-Speed ULPI
Additional context
I've a stm32h747 board with a high-speed external PHY.
But for testing/reliability purposes I want to limit the speed to Full-Speed.
I tried with:
#define BOARD_TUH_MAX_SPEED OPT_MODE_HIGH_SPEED#define CFG_TUH_MAX_SPEED OPT_MODE_FULL_SPEEDIn my
tusb_config.h, but with bad results (hang in trying to setup the low-speed phy)This patch is just a proof-of-concept that it is not too difficult to make that work, In the hope that someone can clean that up :)