Skip to content

[SL-UP] Matter AWS fixes for Thermosat app refactor, memory issues#1016

Merged
sabollim-silabs merged 3 commits into
release_2.9-1.6from
bugfix/aws_mem_bugfix
Jun 15, 2026
Merged

[SL-UP] Matter AWS fixes for Thermosat app refactor, memory issues#1016
sabollim-silabs merged 3 commits into
release_2.9-1.6from
bugfix/aws_mem_bugfix

Conversation

@sabollim-silabs

@sabollim-silabs sabollim-silabs commented Jun 12, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Summary

This PR aims to fix below issues:

  1. Thermostat AWS publish broken after CRTP app reafactor.
  2. Matter AWS sees boot up failure due to memory issues in 917 SoC only

Fixes :

  1. Refactored thermostat aws code to adopt new CRTP code.
  2. Decrease Memory in Total heap allocated for aws in 917 Soc , decresed cert size for non dual stack case
  3. removed additional protection added to avoid multiple triggers of awsinit task. As awstaskinit check sfor nullpointer , so init happens only one time.

Matter AWS is PoC, will be completely refactored for GA

Related issues

Testing

Booted lock app on 917Soc and validated AWS
Booted thermostat on 917Soc and validated AWS
Booted lock app on 917 NCP and validated AWS

@silabs-matter-ci-bot silabs-matter-ci-bot Bot added the sl-up This TAG indicates that this commit needs to be upstreamed to CSA before its next release. label Jun 12, 2026
@sabollim-silabs sabollim-silabs marked this pull request as ready for review June 12, 2026 15:21
@sabollim-silabs sabollim-silabs requested a review from a team as a code owner June 12, 2026 15:21
Sarthak-Shaha
sarthak shaha (Sarthak-Shaha) approved these changes Jun 12, 2026
View reviewed changes

@Sarthak-Shaha sarthak shaha (Sarthak-Shaha) left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Validated thermostat related changes

rosahay-silabs
Rohan Sahay (rosahay-silabs) approved these changes Jun 15, 2026
View reviewed changes
Comment thread examples/platform/silabs/BaseApplication.cpp
@sabollim-silabs sabollim-silabs mentioned this pull request Jun 15, 2026
Merged
@sabollim-silabs sabollim-silabs merged commit d62f380 into release_2.9-1.6 Jun 15, 2026
89 of 90 checks passed
@sabollim-silabs sabollim-silabs deleted the bugfix/aws_mem_bugfix branch June 15, 2026 14:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@selissia Sergei Lissianoi (selissia) selissia approved these changes

@chirag-silabs chirag-silabs chirag-silabs approved these changes

@rosahay-silabs Rohan Sahay (rosahay-silabs) rosahay-silabs approved these changes

@Sarthak-Shaha sarthak shaha (Sarthak-Shaha) Sarthak-Shaha approved these changes

Assignees

No one assigned

Labels

sl-up This TAG indicates that this commit needs to be upstreamed to CSA before its next release.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@sabollim-silabs @selissia @chirag-silabs @rosahay-silabs @Sarthak-Shaha