Skip to content

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

Merged
merged 6 commits into from
Jun 12, 2017

Conversation

AlessandroA
Copy link

Putting this up to test the CI with the ST target.

@Patater @niklas-arm

@Patater
Copy link
Contributor

Patater commented May 23, 2017

CI failed because DISCO is a little bit too slow.

12:12:39 [1495537942.33][CONN][RXD] Main loop count: 31

12:12:39 [1495537942.86][CONN][RXD] Main loop count: 32

12:14:31 [1495538043.26][HTST][INF] test suite run finished after 120.25 sec...
12:14:31 [1495538043.26][HTST][ERR] Expected output [Main loop count: 33] not received in log.

@AlessandroA AlessandroA force-pushed the stm32f4_support_rebased branch from f2031d1 to 28fdc8f Compare May 25, 2017 15:39
@AlessandroA AlessandroA changed the title WIP: Add support for the STM32F429ZI target Add support for the STM32F429ZI target May 25, 2017
Copy link
Contributor

@Patater Patater left a 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 |
Copy link
Contributor

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.

Copy link
Author

Choose a reason for hiding this comment

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

Done.

@AlessandroA AlessandroA force-pushed the stm32f4_support_rebased branch from 28fdc8f to 951f118 Compare May 25, 2017 17:01
@AlessandroA
Copy link
Author

This can be merged once our mbed OS updates are merged into mbed OS.

@Patater
Copy link
Contributor

Patater commented Jun 5, 2017

retest uvisor

@AlessandroA AlessandroA force-pushed the stm32f4_support_rebased branch from 951f118 to 57f1db7 Compare June 9, 2017 10:37
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"
@AlessandroA AlessandroA force-pushed the stm32f4_support_rebased branch from 57f1db7 to 8bd29b6 Compare June 12, 2017 09:51
@AlessandroA
Copy link
Author

retest uvisor

@AlessandroA AlessandroA merged commit 979b9f9 into ARMmbed:master Jun 12, 2017
@AlessandroA AlessandroA deleted the stm32f4_support_rebased branch June 12, 2017 11:35
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.

2 participants