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

[SL-TEMP] Fix commissioning with test credentials #366

Open
wants to merge 1 commit into
base: release_2.5-1.4
Choose a base branch
from

Conversation

jmartinez-silabs
Copy link
Contributor

Problem

Devices fail to commission when not provisioned when build with GN
CHIP_DEVICE_CONFIG_ENABLE_EXAMPLE_CREDENTIALS was replaced by SL_MATTER_ENABLE_EXAMPLE_CREDENTIALS with new matter_support structure within the gn build config. However within matter 1.4 ProvisionStorageDefault.cpp and ProvisionStorageFlash.cpp the #ifdef to enable de example fallback have not being changed. Additionally Matter_Extension 2.5-1.4 and 2.6-1.4 still define CHIP_DEVICE_CONFIG_ENABLE_EXAMPLE_CREDENTIALS

Solution

To minimize the change and upgrade rules with in matter 1.4 release, Re-add CHIP_DEVICE_CONFIG_ENABLE_EXAMPLE_CREDENTIALS in GN build for the provisionStorageXXX sources.

SL_MATTER_ENABLE_EXAMPLE_CREDENTIALS is already present in matter 1.5 and 1 single upgrade rule can be done for matter extension 2.7.0-1.5

Testing

Successfully commission BRD4187C without provisioning with fix in place

@jmartinez-silabs jmartinez-silabs added the sl-temp This TAG indicates that this commit is silabs-specific that will not be upstream or carried over. label Mar 21, 2025
@jmartinez-silabs jmartinez-silabs requested a review from a team as a code owner March 21, 2025 23:38
@jmartinez-silabs jmartinez-silabs changed the title [SL-TEMP] Fix/commissioning with test credentials [SL-TEMP] Fix commissioning with test credentials Mar 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
sl-temp This TAG indicates that this commit is silabs-specific that will not be upstream or carried over.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants