-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Implement RebootCount storage on Darwin. #29134
Implement RebootCount storage on Darwin. #29134
Conversation
PR #29134: Size comparison from 5ae3c65 to 1dc414b Decreases (1 build for efr32)
Full report (65 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, qpg, telink)
|
PR #29134: Size comparison from 5ae3c65 to c6523f4 Decreases (1 build for efr32)
Full report (65 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, qpg, telink)
|
@@ -89,6 +89,21 @@ CHIP_ERROR DiagnosticDataProviderImpl::ResetWatermarks() | |||
return CHIP_NO_ERROR; | |||
} | |||
|
|||
CHIP_ERROR DiagnosticDataProviderImpl::GetRebootCount(uint16_t & rebootCount) | |||
{ | |||
uint32_t count = 0; |
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.
why is this uint32_t? and then cast to uint16_t on line 101? could it be uint16_t count?
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.
why is this uint32_t? and then cast to uint16_t on line 101?
Because the configuration manager API is:
virtual CHIP_ERROR GetRebootCount(uint32_t & rebootCount) = 0;
Now why is that one 32-bit while the actual value in the spec is 16-bit? I don't really know, and it's very out of scope for this PR to change.
* Implement RebootCount storage on Darwin. Fixes project-chip#29108 * Address review comment.
* Implement RebootCount storage on Darwin. Fixes project-chip#29108 * Address review comment.
Fixes #29108