Skip to content

[SL-ONLY] Compilation errors fixed: Unused variables. - #1207

Merged
Ricardo Casallas (rcasallas-silabs) merged 1 commit into
mainfrom
fix_unused_variables
Sep 25, 2026
Merged

Ricardo Casallas (rcasallas-silabs) merged 1 commit into
mainfrom
fix_unused_variables

Conversation

@rcasallas-silabs

Copy link
Copy Markdown
Contributor

Summary

Experimental use of sl_bt_system_stop_bluetooth to disable bluetooth after commissioning.

Related issues

MATTER-6292.

Testing

BRD4187C (MG24) compiled, and commissioned. On/off toggle after commissioning.

@cursor

cursor Bot commented Sep 24, 2026 •

Copy link
Copy Markdown

PR Summary

Low Risk
Conditional compilation and attribute-only changes with no runtime behavior change in the default enabled paths.

Overview
Build fixes for Silicon Labs apps that stop Bluetooth after commissioning (experimental sl_bt_system_stop_bluetooth work under MATTER-6292).

In BaseApplication, the Zigbee leave-announce delay constant is now declared only when SL_MATTER_ZIGBEE_SEQUENTIAL is enabled, avoiding an unused symbol in other build flavors. sCodeDrivenIdentifyVariant is marked [[maybe_unused]] so configurations that store the variant but do not read it still compile cleanly.

In BLEManagerImpl, the mBleSideChannel member is compiled in only when SL_BLE_SIDE_CHANNEL_ENABLED is set, matching the rest of the side-channel API guards instead of tying it to “not SI91x BLE.” That keeps the BLE manager layout consistent when Matter BLE is torn down or side channel is disabled.

Reviewed by Cursor Bugbot for commit b1ef89d. Configure here.

@Sarthak-Shaha sarthak shaha (Sarthak-Shaha) changed the title [SL_TEMP] Compilation errors fixed: Unused variables. [SL-TEMP] Compilation errors fixed: Unused variables. Sep 24, 2026
@silabs-matter-ci-bot silabs-matter-ci-bot Bot added the sl-temp This TAG indicates that this commit is silabs-specific that will not be upstreamed or carried over. label Sep 24, 2026
@jmartinez-silabs

Copy link
Copy Markdown
Contributor

Ricardo Casallas (@rcasallas-silabs) I think this should be a [SL-ONLY]

@rcasallas-silabs Ricardo Casallas (rcasallas-silabs) changed the title [SL-TEMP] Compilation errors fixed: Unused variables. [SL-ONLY] Compilation errors fixed: Unused variables. Sep 24, 2026
@silabs-matter-ci-bot silabs-matter-ci-bot Bot added sl-only This TAG indicates that this commit is a silabs-specific commit that cannot be upstreamed to CSA. and removed sl-temp This TAG indicates that this commit is silabs-specific that will not be upstreamed or carried over. labels Sep 24, 2026
@rcasallas-silabs
Ricardo Casallas (rcasallas-silabs) merged commit c41e20c into main Sep 25, 2026
116 of 117 checks passed
@rcasallas-silabs
Ricardo Casallas (rcasallas-silabs) deleted the fix_unused_variables branch September 25, 2026 12:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

sl-only This TAG indicates that this commit is a silabs-specific commit that cannot be upstreamed to CSA.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants