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

config_psa: do not update legacy symbols in client-only PSA build #58

Closed

Conversation

valeriosetti
Copy link
Collaborator

Description

This commits belongs to PR 9138 of the official MbedTLS repo (Mbed-TLS/mbedtls#9138). It will be included both in the next LTS release (3.6.1) as well as the next standard one (4.0) so it can be dropped at the next Zephyr's downstream version bump.

This commits belongs to PR 9138 of the official MbedTLS repo
(Mbed-TLS/mbedtls#9138). It will be
included both in the next LTS release (3.6.1) as well as
the next standard one (4.0) so it can be dropped at the
next Zephyr's downstream version bump.

Signed-off-by: Valerio Setti <[email protected]>
@valeriosetti valeriosetti marked this pull request as draft July 23, 2024 08:51
@valeriosetti valeriosetti marked this pull request as ready for review July 23, 2024 09:07
@valeriosetti valeriosetti marked this pull request as draft July 23, 2024 13:56
@valeriosetti valeriosetti reopened this Jul 24, 2024
@valeriosetti valeriosetti marked this pull request as ready for review July 24, 2024 09:36
MCUBOOT has its own version of the PSA core, named
"thin_psa_crypto_core.c" which is included in the MCUBOOT's
build process when MCUBOOT_USE_PSA_CRYPTO build symbol is
set. MCUBOOT_USE_PSA_CRYPTO mimics what MBEDTLS_PSA_CRYPTO_C
does for the official Mbed TLS, so we need to replicate
this symbol in "config_psa.h" in order to get the proper
PSA builtin support.

Signed-off-by: Valerio Setti <[email protected]>
@tomi-font tomi-font mentioned this pull request Sep 2, 2024
@valeriosetti
Copy link
Collaborator Author

I'm closing this one because:

So basically #60 includes this PR, so this one can be closed.

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.

1 participant