-
Notifications
You must be signed in to change notification settings - Fork 11
Add support for the STM32F429ZI target #52
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 support for the STM32F429ZI target #52
Conversation
CI failed because DISCO is a little bit too slow.
|
f2031d1
to
28fdc8f
Compare
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.
Consider moving the "Reduce memory footprint of this app" and "Add linker script memory filler for ARMv7-M MPU" commits to before adding support for new targets.
|-------------------|-----------|-----------| | ||
| `K64F` | `GCC_ARM` | 9600 | | ||
| `DISCO_F429ZI` | `GCC_ARM` | 9600 | | ||
| `EFM32GG_STK3700` | `GCC_ARM` | 9600 | |
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.
Mention fixing #33 in the commit message to auto-close the issue.
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.
Done.
28fdc8f
to
951f118
Compare
This can be merged once our mbed OS updates are merged into mbed OS. |
retest uvisor |
951f118
to
57f1db7
Compare
This will help supporting more targets.
The code snippet added here has been auto-generated by uVisor when running the app in debug mode. The additional reserved space is needed as a result of an ARMv7-M MPU limitation.
Fixes ARMmbed#33 "mbed-os-example-uvisor-thread doesn't work on DISCO_F429ZI"
57f1db7
to
8bd29b6
Compare
retest uvisor |
Putting this up to test the CI with the ST target.
@Patater @niklas-arm