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

nucleo-f439zi support #1258

Merged
merged 1 commit into from Dec 22, 2021
Merged

nucleo-f439zi support #1258

merged 1 commit into from Dec 22, 2021

Conversation

leptun
Copy link
Contributor

@leptun leptun commented Dec 21, 2021

Describe the PR
This PR implements the board definition for the ST nucleo f439zi development board.
I tested the uac2_headset and cdc_msc examples and both worked as they should. The uart config also works since I got an assert error with cdc_dual_ports (not enough endpoints, 5 needed, 4 available).

Limitations:

  • The USB HS peripheral is not implemented. So only OTG_FS can be used. Nucleo boards have only one downstream USB port connected to OTG_FS.
  • The host mode power switch pin doesn't seem to be defined/initialized anywhere. This is also the case for the f412zg nucleo board definition, so I suspect this isn't working there as well.

Copy link
Owner

@hathach hathach left a comment

Choose a reason for hiding this comment

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

look great, thank you for your PR. Will merge when ci passed.

@hathach
Copy link
Owner

hathach commented Dec 22, 2021

  • The USB HS peripheral is not implemented. So only OTG_FS can be used. Nucleo boards have only one downstream USB port connected to OTG_FS.

OTG HS required external PHY, which is not available on nucleo platform. So it is the board limitation not your PR. HS work on other boards like f723 or h743.

  • The host mode power switch pin doesn't seem to be defined/initialized anywhere. This is also the case for the f412zg nucleo board definition, so I suspect this isn't working there as well.

host driver isn't implemented for synopsys dwc2 (used by ST MCU) just yet. We will worry about this later on.

@hathach hathach merged commit 557eb51 into hathach:master Dec 22, 2021
@leptun leptun deleted the nucleo-f439zi branch December 11, 2022 21:43
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.

None yet

2 participants