-
Notifications
You must be signed in to change notification settings - Fork 3k
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
TARGET_TFM_V1_0: compatibility with Mbed TLS 2.24.0 #14333
TARGET_TFM_V1_0: compatibility with Mbed TLS 2.24.0 #14333
Conversation
@LDong-Arm, thank you for your changes. |
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.
Looks good.
@ARMmbed/team-cypress FYI |
CI started |
6b17b7c
to
388335e
Compare
ci restarted |
Jenkins CI Test : ✔️ SUCCESSBuild Number: 2 | 🔒 Jenkins CI Job | 🌐 Logs & ArtifactsCLICK for Detailed Summary
|
Previously, some IAR-related configs in
Rather than fixing configs for an unsupported toolchain, I added a commit to remove IAR altogether from Now Jenkins passes. But Travis seems unhappy, because it still has test cases for IAR support: https://travis-ci.com/github/ARMmbed/mbed-os/jobs/485834959. |
We could update the tests Travis is running not to check for IAR support. This probably makes sense if we are no longer supporting IAR. Given we don't have any IAR licenses to test the support works, removing the check for IAR support makes sense to me. |
@0xc0170, @Patater: |
721d4d8
to
8ac80fe
Compare
Sometimes tasks have larger scope than initially thought. Solving things without accumulating tech debt should save us work in the medium to long term. |
I've created #14350 to fix the conflict in IAR in |
Mbed TLS 2.24.0 requires a few new macros and an inline function in the PSA Crypto header. This PR adds them to make sure the TF-M v1.0 target (specifically CYTFM_064B0S2_4343W) continues to compile with the new Mbed TLS. Note: Support for older versions of TF-M than v1.2 will be dropped, so existing TF-M targets should migrate to TF-M v1.2 as soon as possible.
56b66d1
to
6ab4b1c
Compare
8ac80fe
to
330e4d0
Compare
Rebased feature-tf-m-1.2-integration and this PR to include the CI fix. |
CI started |
Jenkins CI Test : ✔️ SUCCESSBuild Number: 3 | 🔒 Jenkins CI Job | 🌐 Logs & ArtifactsCLICK for Detailed Summary
|
Summary of changes
Mbed TLS 2.24.0 requires a few new macros and an inline function in the PSA Crypto header. This PR adds them to make sure the TF-M v1.0 target (specifically CYTFM_064B0S2_4343W) continues to compile with the new Mbed TLS.
Note: Support for older versions of TF-M than v1.2 will be dropped, so existing TF-M targets should migrate to TF-M v1.2 as soon as possible. Once Cypress has updated CYTFM_064B0S2_4343W to TF-M v1.2, we should remove the entire
TARGET_TFM_V1_0
directory where this patch exists.Impact of changes
All existing TF-M targets supported by Mbed OS, including the TF-M v1.0 CYTFM_064B0S2_4343W, now compile successfully.
Migration actions required
None.
Documentation
None.
Pull request type
Test results
Reviewers