-
Notifications
You must be signed in to change notification settings - Fork 979
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
Add variant: NUCLEO-H503RB. #2092
Conversation
Thanks for squashing. Anyway simply force update on the same branch will avoid creating new PR. :wink : Let ci run to check this PR first. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
First thanks for this PR.
In addition to my comments, could you add the generic and the Nucleo in the README.md and also declare them to the boards.txt.
Finally, split in 2 commit, 1 for the generic addition and one for the dedicated Nucleo board.
You can refers to this PR as example:
#1859
Thank you for detailed explanation, it really helped me. For the time being, I've fixed the part you told me, so I'll commit it. |
The Arduino pinout only goes up to |
Hi @dojyorin |
Add variant: NUCLEO-H503RB
Summary
This PR fixes/implements the following bugs/features
NUCLEO-H503RB
@fpistm feat adds STM32H5 core implementation and NUCLEO-H563ZI support (That was really great help! Thank you!), but variant support is still lacking.
This time, I'll add new H5 NUCLEO board NUCLEO-H503RB with this pull request.
Related to #722 (comment) and #2091
Progress
There's still some work to do.
Basically implementation of NUCLEO-H563ZI is used reference.
I'm new to STM32, so excuse me if I'm doing something wrong.
I would like to discuss with everyone and produce good results.