Code size reduction in SLC components#520
Merged
Jean-Francois Penven (jepenven-silabs) merged 3 commits intorelease_2.9-1.6from May 6, 2026
Merged
Code size reduction in SLC components#520Jean-Francois Penven (jepenven-silabs) merged 3 commits intorelease_2.9-1.6from
Jean-Francois Penven (jepenven-silabs) merged 3 commits intorelease_2.9-1.6from
Conversation
Jean-Francois Penven (jepenven-silabs)
requested a review
from a team
as a code owner
sarthak shaha (Sarthak-Shaha)
pushed a commit
to Sarthak-Shaha/matter_extension
that referenced
this pull request
Mar 20, 2026
…ations Improvement
Merge in WMN_TOOLS/matter_extension from improvement/icd_server_config_component to release_2.6-1.4
Squashed commit of the following:
commit e9bb74d51d04690f309c88076acc7fd43afc38a5
Author: ardutta <arghya.dutta@silabs.com>
Date: Fri Jul 4 13:58:29 2025 +0530
Added Capitalization in the ICD Server Configuration UI
commit be2ed0ad61e165114112e5817a936289660aafcf
Author: ardutta <arghya.dutta@silabs.com>
Date: Fri Jul 4 10:13:44 2025 +0530
Updated the ICD Server Configuration
commit 246b2269f670c07f3a54b0c051d5bf7c7c8c77c7
Author: Arghya Dutta <arghya.dutta@silabs.com>
Date: Thu Jul 3 11:39:43 2025 +0530
Added units in the ICD Server Configuration UI
commit 810abaecbcb96701e0d8427b83bd378fe4591c65
Author: Arghya Dutta <arghya.dutta@silabs.com>
Date: Wed Jul 2 13:26:50 2025 +0530
Updated the ICD Server Configuration
Jean-Francois Penven (jepenven-silabs)
changed the base branch from
main
to
release_2.9-1.6
Jean-Francois Penven (jepenven-silabs)
force-pushed
the
matter_5920_code_size_reduction
branch
from
2f6217e to
714fdeb
Compare
Contributor
There was a problem hiding this comment.
Pull request overview
This PR aims to reduce default SLC build size by disabling optional Matter features (notably the Interaction Model Read Client) unless explicitly enabled via components.
Changes:
- Disable
CHIP_CONFIG_ENABLE_READ_CLIENTby default in the SLC app build config. - Introduce a new
matter_read_clientSLC component that re-enables the read client and adds its sources/headers. - Adjust related SLC components (shell, interaction model/datamodel packaging, OTA requestor includes, no-debug size-reduction defines) to align with the new optional split.
Reviewed changes
Copilot reviewed 7 out of 7 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| slc/inc/app/AppBuildConfig.h | Changes default read-client enablement to reduce baseline footprint. |
| slc/component/matter-platform/size-reduction/matter_no_debug.slcc | Disables CLI arg parser when shell isn’t present to save size. |
| slc/component/matter-platform/shell/matter_shell.slcc | Gates IM shell command sources behind the new read-client option. |
| slc/component/matter-core-sdk/read_client.slcc | Adds a new optional component encapsulating ReadClient sources/define. |
| slc/component/matter-core-sdk/libchipdatamodel.slcc | Removes read-client-related headers/sources from the default datamodel packaging. |
| slc/component/matter-core-sdk/interaction_model.slcc | Removes ReadClient.cpp from default build, keeps header available for compilation. |
| slc/component/matter-clusters/matter_ota_requestor.slcc | Updates include dependencies to account for read-client headers no longer being default. |
Junior Martinez (jmartinez-silabs)
approved these changes
Apr 24, 2026
Contributor
Junior Martinez (jmartinez-silabs)
left a comment
Junior Martinez (jmartinez-silabs)
left a comment
There was a problem hiding this comment.
Approving.
Current copilot comments should be applied
lpbeliveau-silabs
approved these changes
May 5, 2026
sarthak shaha (Sarthak-Shaha)
approved these changes
May 5, 2026
Jean-Francois Penven (jepenven-silabs)
force-pushed
the
matter_5920_code_size_reduction
branch
from
fd508f9 to
9fa13f4
Compare
Jean-Francois Penven (jepenven-silabs)
force-pushed
the
matter_5920_code_size_reduction
branch
from
9fa13f4 to
e15f398
Compare
Jean-Francois Penven (jepenven-silabs)
merged commit 053a7a0
into
release_2.9-1.6
212 checks passed
Jean-Francois Penven (jepenven-silabs)
deleted the
matter_5920_code_size_reduction
branch
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
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.
Issue Link:
MATTER-5921
Description of Problem/Feature:
SLC builds are usually bigger than the GN ones. This PR is an attempt at closing that gap by making optionnal and the ReadClient feature (only needed when the sample apps reads something on another node) and other optionnal feature.
Description of Fix/Solution:
Removes optionnal features from the default offering
Testing Done:
Samples builded and were commissionnable.