Skip to content

Remove Cortex A boards as Cortex-A it is not supported in CMSIS5/RTX2 #4316

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
May 15, 2017
Merged

Remove Cortex A boards as Cortex-A it is not supported in CMSIS5/RTX2 #4316

merged 2 commits into from
May 15, 2017

Conversation

c1728p9
Copy link
Contributor

@c1728p9 c1728p9 commented May 12, 2017

Temporarily remove the Cortex-A targets since these targets will not be supported by CMSIS5 initially.

@c1728p9
Copy link
Contributor Author

c1728p9 commented May 12, 2017

/morph test-nightly

@c1728p9
Copy link
Contributor Author

c1728p9 commented May 12, 2017

Taken from the PR fa51930 on the CMSIS_5 branch.

@mbed-bot
Copy link

Result: SUCCESS

Your command has finished executing! Here's what you wrote!

/morph test-nightly

Output

mbed Build Number: 209

All builds and test passed!

@bulislaw
Copy link
Member

Actually I'll remove Cortex-A support from RTX in this PR as well, should make the final diff cleaner.

@bulislaw
Remove Cortex A support from CMSIS/RTOS
5f29445 
This change is temporary, Cortex A support will be reenabled when it's
fully supported in CMSIS/RTX 5.
@bulislaw
Copy link
Member

/morph test-nightly

@mbed-bot
Copy link

Result: SUCCESS

Your command has finished executing! Here's what you wrote!

/morph test-nightly

Output

mbed Build Number: 219

All builds and test passed!

@c1728p9 c1728p9 merged commit 0a3b256 into ARMmbed:master May 15, 2017
@sg- sg- removed the needs: CI label May 15, 2017
@jeromecoutant
Copy link
Collaborator

Hi
It seems that some IAR tests are now failed since this PR...
Could you check?

@bulislaw
Copy link
Member

@0xc0170 is working on it, we've removed main.S which is used in non-RTOS cases, it has to be reintroduced.

@0xc0170
Copy link
Contributor

0xc0170 commented May 19, 2017

@jeromecoutant This should fix it #4334 , waiting for CI to confirm green status, and we should be good to do!

If you report a problem, please provide more details. I assume you are running mbed sdk tests, no rtos included?

@jeromecoutant
Copy link
Collaborator

Thx Martin
I was analysing the issue before providing more details and patch... but @bulislaw has been very fast to answer :-)

@YarivCol YarivCol mentioned this pull request May 17, 2017
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bulislaw bulislaw bulislaw approved these changes

@pan- pan- pan- approved these changes

Assignees
No one assigned
Labels
release-version: 5.5.0-rc1
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@c1728p9 @mbed-bot @bulislaw @jeromecoutant @0xc0170 @pan- @sg-