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

zephyr: update main Mbed TLS configuration header file #124

Conversation

valeriosetti
Copy link

@valeriosetti valeriosetti commented Feb 17, 2025

This PR should be merged in sync with zephyrproject-rtos/zephyr#78641.

Note for reviewers: since zephyrproject-rtos/zephyr#78641 is changing the name of the Mbed TLS configuration file, the CI on this PR is always going to fail. Please evaluate CI of the PR in the main repo instead.

valeriosetti added a commit to valeriosetti/zephyr that referenced this pull request Feb 17, 2025
This is needed to include changes from zephyrproject-rtos/mcuboot#124.

Signed-off-by: Valerio Setti <[email protected]>
@de-nordic
Copy link
Collaborator

First we need to get this in zephyrproject-rtos/zephyr#85620, otherwise we are not able to update MCUboot in downstream repo.

Mbed TLS already provides MBEDTLS_CFG_FILE Kconfig when it's builtin. So
we only need to override that config when not using the builtin Mbed TLS
version.

This commit also fixes some hardcoding of "mcuboot-mbedtls-cfg.h" in
Kconfig and prj.conf.

Signed-off-by: Valerio Setti <[email protected]>
@valeriosetti valeriosetti force-pushed the remove-unused-headers-mcuboot branch from 81388b3 to fdf644c Compare February 24, 2025 12:40
@nordicjm
Copy link
Collaborator

Needs to be submitted to https://github.com/mcu-tools/mcuboot we do not accept PRs here

@valeriosetti
Copy link
Author

Closing this one because it was superseded by mcu-tools#2222

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.

3 participants