Update the watchdog timing requirements #1155
Conversation
Update the requirements according to PR #11203 that differentiates lower timeout limit for calibrated and uncalibrated watchdog clock.
docs/porting/target/Watchdog.md
Outdated
| @@ -11,7 +11,8 @@ Implement the ResetReason API when implementing the Watchdog API. The ResetReaso | |||
| - Sleep and debug modes don't stop the watchdog timer from counting down. | |||
| - The function `hal_watchdog_init` is safe to call repeatedly. The function's implementation must not do anything if `hal_watchdog_init` has already initialized the hardware watchdog timer. | |||
| - `UINT32_MAX` milliseconds is the maximum allowed max_timeout `hal_watchdog_get_platform_features()` returns; minimum timeout is 1 ms. | |||
| - The watchdog should trigger at or after the timeout value. | |||
| - The uncalibrated watchdog should trigger at or after the timeout value multiplied by the frequency accuracy ratio of its oscillator (typical_frequency / max_frequency). | |||
There was a problem hiding this comment.
Does the slash represent division, or does it mean "or" here?
There was a problem hiding this comment.
Division; Do you think we should use the word instead of the symbol here?
There was a problem hiding this comment.
Yes, I think that would clarify here. I'll make that change.
Clarify slash, as discussed in comments.
|
The code dependency has merged to master but has a future tag on it, so I'm waiting to merge it. |
Allow the watchdog reset to be delayed up to twice the timeout value in deepsleep mode according to #11774.
|
There is another update for the watchdog timing. This time only for the deepsleep mode (based on ARMmbed/mbed-os#11774). Since this PR is waiting, I pushed changes here. @AnotherButler, I hope this is acceptable; please let me know if this should land on a new docs PR. I'll remove the last commit in that case. |
Add space for consistent spelling.
|
ARMmbed/mbed-os#11773 has been merged into master. |
|
Merging PR, as code was merged to master. |
Update the requirements according to ARMmbed/mbed-os#11203, that differentiates lower timeout limit for calibrated and uncalibrated watchdog clock.
This should wait until ARMmbed/mbed-os#11203 is merged to Mbed OS repo.