Skip to content

bootutil: Use mbedtls/private paths for TF-PSA-Crypto#2732

Open
krish2718 wants to merge 1 commit into
mcu-tools:mainfrom
krish2718:topic/mbedtls-private-include-paths
Open

bootutil: Use mbedtls/private paths for TF-PSA-Crypto#2732
krish2718 wants to merge 1 commit into
mcu-tools:mainfrom
krish2718:topic/mbedtls-private-include-paths

Conversation

@krish2718
Copy link
Copy Markdown

MCUboot Mbed TLS crypto wrappers and simulator custom crypto helpers included legacy headers as mbedtls/.h. TF-PSA-Crypto now exposes these declarations only under mbedtls/private/.

Assisted-by: Cursor: Auto

Copy link
Copy Markdown
Member

@d3zd3z d3zd3z left a comment

Choose a reason for hiding this comment

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

This will probably take a bit to get to be "right". One thing I'm curious about (I might investigate this next week) is if this change is sufficient to get everything to pass against mbed TLS 4.1. Change #2713 ends up bringing in both 4.1 and 3.6 because of this issue.

My other concern is how can we encourage the use of the private headers to "go away" at some point. We definitely want to make sure we aren't introducing new instances.

@krish2718
Copy link
Copy Markdown
Author

This will probably take a bit to get to be "right". One thing I'm curious about (I might investigate this next week) is if this change is sufficient to get everything to pass against mbed TLS 4.1. Change #2713 ends up bringing in both 4.1 and 3.6 because of this issue

Ah I see, so, if we are maintaining both 3.6 and 4.1 version, then can we absorb this change in 4.1 specific code?

My other concern is how can we encourage the use of the private headers to "go away" at some point. We definitely want to make sure we aren't introducing new instances.

That's actually one of the reasons for this PR (and associated ones), the current Zephyr copies these headers and drops "private", so, the users would have no idea that they are using headers that could be removed. And as for the actual deletion, it depends on migrating to PSA only versions and that might take a while esp. if the ecosystem continues to use these deprecated algos.

MCUboot Mbed TLS crypto wrappers and simulator custom crypto helpers
included legacy headers as mbedtls/<alg>.h. TF-PSA-Crypto now exposes
these declarations only under mbedtls/private/.

Signed-off-by: Chaitanya Tata <Chaitanya.Tata@nordicsemi.no>
Assisted-by: Cursor: Auto
@krish2718 krish2718 force-pushed the topic/mbedtls-private-include-paths branch from 732f553 to 63a798d Compare May 18, 2026 20:21
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