-
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
Add MBEDTLS_ENTROPY_NV_SEED to NUCLEO_F411RE to fix the build #9677
Conversation
Any other target that was fixed in #9605 needs this or only nucleo affected? |
No, all the other have TRNG |
Ci started |
This fixes nightly client breakage from the last run |
Test run: SUCCESSSummary: 12 of 12 test jobs passed |
@ARMmbed/team-st-mcd Please review. We would like to have this fix on master today to resolve nightly failure |
This may fix the build failure, but does the device work properly at run-time? Can a fresh F411 successfully initialize and use crypto functionality? I'm OK with this PR as-is since it's important to fix the nightly build, but more follow up is needed as to how we want to handle entropy with F411 targets. |
I got build failure for Nucleo_F411RE in PR #9571 is this related? It was not for nightly build though
|
Yes related, this needs to get in asap. |
@ARMmbed/mbed-os-psa Thoughts about #9677 (comment) ? |
Fix went to 5.11.5 so this one as well |
Moved to 5.12 due to failures with 5.11 branch |
Description
Add MBEDTLS_ENTROPY_NV_SEED to support psa_crypto on NUCLEO_F411RE
The platform does not have TRNG and must have a source of entropy to use psa-crypto
Pull request type
Reviewers