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-ONLY] Add ApplicationSleepManager structure for the selective listenning implementation #131

Merged
merged 6 commits into from
Nov 27, 2024

Conversation

mkardous-silabs
Copy link
Contributor

@mkardous-silabs mkardous-silabs commented Nov 25, 2024

Description

PR introduces the ApplicationSleepManager for Wi-Fi low power apps. The ApplicationSleepManager is responsible of analysing the Matter SDK states and pipe that information to the WifiSleepManager. Based on the information given to the WifiSleepManager by the CanGoToLIBasedSleep callback, the WiFiSleepManager can go to the most optimal sleep state.

PR only introduces the structure; the buisness logic that will go in CanGoToLIBasedSleep will be in a follow Up PR.

PR also has two fixes for the wf200 that were missed in a previous PR that are already upstreamed in project-chip/connectedhomeip#36628

Note to reviewers

I wasn't able to add unit tests to cover this class due to a dependecy with underlying platform code. In a follow up, i will need to figure out how to brake the platform dependency or leverage the test driver which doesn't support the 917 at this point.

Tests

Manual tests to validate behavior

@mkardous-silabs mkardous-silabs added the sl-only This TAG indicates that this commit is a silabs-specific commit that cannot be upstream to CSA. label Nov 25, 2024
@mkardous-silabs mkardous-silabs requested a review from a team as a code owner November 25, 2024 19:53
Co-authored-by: Ricardo Casallas <[email protected]>
@mkardous-silabs mkardous-silabs merged commit c16843b into release_2.5-1.4 Nov 27, 2024
11 of 12 checks passed
@mkardous-silabs mkardous-silabs deleted the feature/selective_listening branch November 27, 2024 03:09
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 upstream to CSA.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants