Skip to content
This repository has been archived by the owner on Jun 5, 2019. It is now read-only.

Update CMSIS to 4.5 #386

Open
wants to merge 6 commits into
base: dev
Choose a base branch
from
Open

Update CMSIS to 4.5 #386

wants to merge 6 commits into from

Conversation

josesimoes
Copy link
Contributor

No description provided.

josesimoes and others added 6 commits October 29, 2015 10:34
- checked implementation of osXxxxDef objects
- need to add cpp define for __FPU_PRESENT because CMSIS code has
changed and the FP checks on previous versions that produced warning are
now errors
@smaillet-ms
Copy link
Member

What platforms did you test this against?

@josesimoes
Copy link
Contributor Author

Discovery4

@smaillet-ms
Copy link
Member

This requires more testing before it can be merged in. At the very least it must be validated against the MCBSTM32F400 using networking, as any change in the CMSIS-RTX implementation could impact the networking sys-arch layer. (we have some dependencies on internal implementation details that require testing - and have a history of breaking things)

@josesimoes
Copy link
Contributor Author

Can't help you with that... I don't have a MCBSTM32F400 board.
I just build it for the Discovery Solution and loaded the image. The boot was OK and the board showed on MFDeploy.

This "update" on the CMSIS was made as part of my quest to add networking with GCC.

If this is not the time to update CMSIS then leave it as it is. It hasn't changed anything on networking with GCC anyway...

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants