-
-
Notifications
You must be signed in to change notification settings - Fork 773
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
Stm32u5 support #1574
Stm32u5 support #1574
Conversation
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.
This is looking great, there's just one tweak that needs making and when done we can then merge this. Thank you for the contribution!
I will agree if you ask me to integrate the change request into the second commit. I was not so confident that the operation proposed by the github interface will do that or not. |
Yes, please do - sorry, meant to get back to this PR earlier and ask you to do just that. With that change made part of the second commit, we're happy to mark as accepted and merge this |
Don't worry, it's part of the normal learning curve for working together. |
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.
LGTM now, merging.
Thank you! |
Detailed description
Addition of the documentation reference for the STM32U5.
Addition of the DP_TARGETIDR id for the STM32U5.
Addition of the stm32l4_device_info for the STM32U5.
Addition of the RAM map for the STM32U5.
Your checklist for this pull request
make PROBE_HOST=native
)make PROBE_HOST=hosted
)Closing issues