-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
[Infineon] Not to increment RebootCount if wakes from sleep state #26275
base: master
Are you sure you want to change the base?
Conversation
|
||
err = StoreRebootCount(rebootCount + 1); | ||
SuccessOrExit(err); | ||
if (wiced_sleep_get_boot_mode() != WICED_SLEEP_FAST_BOOT) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Another question: when is PlatformMgrImpl().GetStartTime()
being set, for purposes of GetUpTime? That one:
- Needs to match the reboot count bit
- Needs to match the time base for events.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For the first question, we will save data into retention RAM before sleep. And it count as "wakes up from a sleep state".
For the second question, I think PlatformMgrImpl().GetStartTime() is for Linux specific, and we didn't implement it on our platform.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@AlvinHsiao Which data is being saved into retention RAM?
I see that I just misread things and GetStartTime is relevant to Infineon/PSOC6
, not Infineon/CYW30739
, and that Infineon/CYW30739
just does not implement the UpTime attribute at all. Is that a correct description of the current situation?
And you're shutting down the Matter stack before entering the sleep state and restarting it when you exit?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@bzbarsky-apple Yes, this is our temporary workaround to support sleepy end device.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@HuangCC-IFX That doesn't quite answer my question around which data is persisted. For example, when you enter this sleep state, do you clear out the queued events in the Matter stack? Do you issue a new StartUp event when coming out and a ShutDown event when entering?
PR #26275: Size comparison from 328c553 to edbe4b4 Increases (1 build for cc32xx)
Full report (1 build for cc32xx)
|
This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. |
This stale pull request has been automatically closed. Thank you for your contributions. |
PR #26275: Size comparison from 7423b08 to edbe4b4 Increases (5 builds for bl702l, cyw30739)
Full report (49 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, qpg)
|
This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. |
Problem
RebootCount would increment if wakes from sleep state.
Change overview
Not to increment RebootCount if wakes from sleep state.
Testing
Tested manually with chip-tool.