Skip to content

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

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