-
Notifications
You must be signed in to change notification settings - Fork 11
Update for feature_cmsis5 #47
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
Conversation
bump |
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.
Thanks for the PR!
mbed_app.json
Outdated
@@ -3,6 +3,10 @@ | |||
"*": { | |||
"target.features_add": ["UVISOR"], | |||
"target.extra_labels_add": ["UVISOR_SUPPORTED"] | |||
}, | |||
"EFM32GG_STK3700": { |
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 not needed anymore, because of the *
in line 3.
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.
Fixed
mbed_app.json
Outdated
}, | ||
"EFM32GG_STK3700": { | ||
"target.features_add": ["UVISOR"], | ||
"target.extra_labels_add": ["UVISOR_SUPPORTED"] |
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.
Please modify test/filters.json
as well (remove EFM32GG_STK3700
from the blacklist).
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.
Updated
mbed OS has been updated and we no longer need to provide the backwards compatibility `ifdef` guard. Remove the `ifdef` guard as unnecessary.
Add support for building with RTX2, breaking compatibility with the previous version of RTX.
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
retest uvisor |
@AlessandroA @niklas-arm