Fix missing CHIP_CRYPTO_KEYSTORE_PSA flag for xg24devkit, xg24explorerkit, xiao_mg24 - #186
Merged
Tamas Jozsi (silabs-bozont) merged 1 commit intoJul 31, 2026
Conversation
…rkit, xiao_mg24 These boards are already configured for PSA crypto (MBEDTLS_PSA_CRYPTO_CONFIG_FILE, sli_psa_builtin_config.h) but never define CHIP_CRYPTO_KEYSTORE_PSA, so chip::Crypto::DefaultSessionKeystore is never defined and every Matter sketch fails to compile for these boards. nano_matter and thingplusmatter already set this flag; align the other three Matter-capable boards with them. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Ludovic BOUÉ (lboue)
requested a review
from Tamas Jozsi (silabs-bozont)
as a code owner
July 24, 2026 21:30
silabs-akosj
added
the
Community contribution
2 tasks
Tamas Jozsi (silabs-bozont)
added
Under review
Tamas Jozsi (silabs-bozont)
approved these changes
Jul 31, 2026
Tamas Jozsi (silabs-bozont)
merged commit Jul 31, 2026
387f71e
into
SiliconLabsSoftware:main
9 checks passed
Tamas Jozsi (silabs-bozont)
removed
the
Under review
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 subscribe to this conversation on GitHub.
Already have an account?
Sign in.
3 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.
Summary
boards.txtfound while investigating the CI failure on Add Matter Binding cluster support with switch/light examples #185 - every Matter sketch (not just Add Matter Binding cluster support with switch/light examples #185's new examples) currently fails to compile for thexg24devkit,xg24explorerkit, andxiao_mg24boards.MBEDTLS_PSA_CRYPTO_CONFIG_FILE=<psa_crypto_config.h>,MBEDTLS_USER_CONFIG_FILE=<sli_psa_builtin_config.h>) but never defineCHIP_CRYPTO_KEYSTORE_PSA. Without it,chip::Crypto::DefaultSessionKeystore(in the vendored Matter SDK'sDefaultSessionKeystore.h) is never defined, soServer.hfails with'DefaultSessionKeystore' in namespace 'chip::Crypto' does not name a type/'sSessionKeystore' was not declared in this scope.nano_matterandthingplusmatteralready set-DCHIP_CRYPTO_KEYSTORE_PSA=1in theirmatterprotocol stackextra_flags; this adds the same flag to the other three Matter-capable boards, right after-DRADIO_CONFIG_DMP_SUPPORT=1to match the existing ordering.main(unrelated to Add Matter Binding cluster support with switch/light examples #185's code changes) -git show main:boards.txtalready lacks the flag for these boards.Test plan
nano_matter,thingplusmatter,xg24devkit,xg24explorerkit,xiao_mg24)PR build testsCI to confirmmatter_air_purifier(and other existing Matter examples) now build forxg24devkit,xg24explorerkit,xiao_mg24🤖 Generated with Claude Code