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: st: nucleo_u5a5zj_q: enable USB HS #81774

Closed

Conversation

mniestroj
Copy link
Member

Use HSE as PLL input source, to achieve clock increased accuracy. This allows to drive USB HS
peripheral clock, which is not possible with MSIS.

Provide configuration for USB HS with embedded USB HS PHY.

Tested with west build -b nucleo_u5a5zj_q zephyr/samples/subsys/usb/cdc_acm:

[27617.468275] usb 1-2.3.2: new high-speed USB device number 51 using xhci_hcd
[27617.583341] usb 1-2.3.2: New USB device found, idVendor=2fe3, idProduct=0001, bcdDevice= 4.00
[27617.583346] usb 1-2.3.2: New USB device strings: Mfr=1, Product=2, SerialNumber=3
[27617.583348] usb 1-2.3.2: Product: Zephyr CDC ACM sample
[27617.583350] usb 1-2.3.2: Manufacturer: ZEPHYR
[27617.583351] usb 1-2.3.2: SerialNumber: 39365007001B001B

LL_APB2_GRP1_PERIPH_OTGPHYC is only provided by STM32F7 HAL layer, so add
'defined(CONFIG_SOC_SERIES_STM32F7X)' to support build on other platforms.

Signed-off-by: Marcin Niestroj <[email protected]>
HSE is populated on Nucleo board, so it is better to use it due to
increased accuracy. Additionally HSE will be reused to drive USB HS
peripheral, which is not possible with MSIS.

Signed-off-by: Marcin Niestroj <[email protected]>
Provide configuration for USB HS with embedded USB HS PHY.

Disable USB_DC_STM32_CLOCK_CHECK, since USB PLL is expected to be sourced
by 16MHz clock, instead of 48MHz.

Signed-off-by: Marcin Niestroj <[email protected]>
@erwango
Copy link
Member

erwango commented Nov 22, 2024

Thanks @mniestroj. Did you had a check to #80249 ?

@mniestroj
Copy link
Member Author

No, I didn't. I will take a look at it and give it a try.

Patches in this PR are from ~1.5 months ago, I just didn't have time to push them yet. I'll close this PR then.

@mniestroj mniestroj closed this Nov 22, 2024
@erwango
Copy link
Member

erwango commented Nov 22, 2024

No, I didn't. I will take a look at it and give it a try.

Thanks, don't hesitate to provide feedback in that PR!

@erikarn
Copy link

erikarn commented Nov 26, 2024

No, I didn't. I will take a look at it and give it a try.

Thanks, don't hesitate to provide feedback in that PR!

hi! please do, I'll go try rebasing my tree (although this week is thanksgiving week so things are a bit weird timing wise here) and see how things go.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: USB Universal Serial Bus platform: STM32 ST Micro STM32
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants