MbedTLS 4.1.0 update fixes for headers#108979
Open
krish2718 wants to merge 4 commits into
Open
Conversation
|
The following west manifest projects have changed revision in this Pull Request:
Additional metadata changed:
⛔ DNM label due to: 1 project with PR revision and 1 project with metadata changes Note: This message is automatically posted and updated by the Manifest GitHub Action. |
26308db to
694e751
Compare
tomi-font
reviewed
May 13, 2026
08ac791 to
28616d4
Compare
tomi-font
requested changes
May 14, 2026
1217e58 to
6e38c16
Compare
tomi-font
previously approved these changes
May 14, 2026
jukkar
previously approved these changes
May 14, 2026
d3zd3z
previously approved these changes
May 16, 2026
Contributor
d3zd3z
left a comment
There was a problem hiding this comment.
I think this is a good idea, obviously needing the west.yml changes to merge and using sha hashes before this can merge.
fb8d03c to
71c27ab
Compare
Use the MbedTLS private headers directly without copy. Signed-off-by: Chaitanya Tata <Chaitanya.Tata@nordicsemi.no>
Use MbedTLS private headers directly without copying. Signed-off-by: Chaitanya Tata <Chaitanya.Tata@nordicsemi.no>
Use MbedTLS private headers directly without copying them. Signed-off-by: Chaitanya Tata <Chaitanya.Tata@nordicsemi.no>
CONFIG_MBEDTLS_DECLARE_PRIVATE_IDENTIFIERS no longer mirrors TF-PSA-Crypto builtin private headers into CMAKE_BINARY_DIR without the private/ segment. Consumers must include mbedtls/private/... explicitly; the mbedtls library INTERFACE already exposes drivers/builtin/include. When building the MCUboot image, add drivers/builtin/src to the mbedTLS INTERFACE include path so bootutil can include rsa_alt_helpers.h by basename. Signed-off-by: Chaitanya Tata <Chaitanya.Tata@nordicsemi.no> Assisted-by: Cursor: Auto
71c27ab to
334bc9b
Compare
|
valeriosetti
approved these changes
May 20, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.



Pull the fix to use TF-PSA-Crypto version header over MbedTLS.