Fix the siwx91x_ota example build error to align with latest mbedtls#156
Open
Swamidas Nampalli (silabs-SwamidasN) wants to merge 1 commit intoSiliconLabsSoftware:mainfrom
Conversation
Update the project configuration (mbedTLS options in prj.conf) so the sample builds with current Zephyr and satisfies mbedtls/check_config.h when RSA key exchange is enabled. Disable MBEDTLS_USE_PSA_CRYPTO for TLS so legacy AES/GCM paths provide MBEDTLS_SSL_HAVE_*; with PSA-for-TLS and no PSA AES wants, the client could advertise no usable cipher suites. Enable cipher and hash sets plus RSA key exchange with the required MD_C, RSA_C, PKCS1, and ASN1 parse options. Use sl_wifi_firmware_header_t for firmware header size instead of the deprecated sl_si91x_firmware_header_t (WiseConnect 4.0). Signed-off-by: Swamidas Nampalli <Swami.Das@silabs.com>
Swamidas Nampalli (silabs-SwamidasN)
requested a review
from a team
as a code owner
Swamidas Nampalli (silabs-SwamidasN)
force-pushed
the
fix-siwx91x_ota-build-error
branch
5 times, most recently
from
9f472eb to
c6048c8
Compare
Jérôme Pouiller (jerome-pouiller)
approved these changes
Mar 30, 2026
Contributor
Jérôme Pouiller (jerome-pouiller)
left a comment
Jérôme Pouiller (jerome-pouiller)
left a comment
There was a problem hiding this comment.
I have hesitated to merge this PR. The upcoming migration to mbedtls4.0 (#155 and zephyrproject-rtos/zephyr#104031) will break this configuration yet another time.
In another hand, it may be more comfortable to have a working OTA sample in the tree before the migration to mbedtls 4.0. So we can bisect if required.
Aksel Mellbye (@asmellby), do you share this opinion?
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
2 participants
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.
Fix the build error