Skip to content
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

Update for mbedtls 2.25.0 #92

Merged
merged 2 commits into from
Jun 11, 2021

Conversation

Patater
Copy link
Contributor

@Patater Patater commented May 14, 2021

  • Upgrade to the new Mbed TLS 2.25.0 version of PSA Crypto, which is closer to the PSA Crypto 1.0 API.
  • Add better error reporting to help with debugging

Depends on ARMmbed/mbed-os#14652

@Patater Patater requested a review from LDong-Arm May 27, 2021 10:32
@LDong-Arm LDong-Arm changed the base branch from master to development June 1, 2021 09:59
@LDong-Arm
Copy link
Contributor

Fix for Travis: #94
This PR needs to be rebased onto "development" after that gets in.

Copy link
Contributor

@LDong-Arm LDong-Arm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM otherwise

getting-started/main.cpp Outdated Show resolved Hide resolved
Patater added 2 commits June 11, 2021 10:17
Add status numbers to errors, so that the Mbed TLS's PSA decoder tool
"psa_constant_names" can be used to display a more readable error
message. This helps with debugging.

    $ psa_constant_names error -134
    PSA_ERROR_NOT_SUPPORTED
@Patater Patater force-pushed the update-for-mbedtls-2.25.0 branch from 88000c1 to a82c710 Compare June 11, 2021 09:28
@Patater Patater requested a review from LDong-Arm June 11, 2021 09:32
@Patater
Copy link
Contributor Author

Patater commented Jun 11, 2021

Rebased on latest development branch

Copy link
Contributor

@LDong-Arm LDong-Arm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, LGTM. Tested on Musca S1.

@Patater Patater merged commit 4e149e4 into ARMmbed:development Jun 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants