-
Notifications
You must be signed in to change notification settings - Fork 3k
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
SysTimer should be buildable without lp ticker #10597
Conversation
The equivalent functionality is present in #10104 |
If you adjust it to match #10104, retaining the default constructor, then I guess it's fine to proceed. |
rtos/TARGET_CORTEX/SysTimer.h
Outdated
@@ -119,8 +118,8 @@ class SysTimer: private mbed::TimerEvent, private mbed::NonCopyable<SysTimer> { | |||
static void _set_irq_pending(); | |||
us_timestamp_t _time_us; | |||
uint64_t _tick; | |||
bool _suspend_time_passed; | |||
bool _suspended; | |||
bool _suspend_time_passed = true; |
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.
This is C++11, isn't it? We've not activated that yet.
#10104 fixed this in the constructor initializer lists.
@LDong-Arm, thank you for your changes. |
SysTimer can run on us ticker now. Only deep sleep test strictly requires lp ticker and it already has build conditions in place.
If a ticker is unavailable, its macro is undefined and cannot be checked with MBED_STATIC_ASSERT.
a4c923d
to
427c7db
Compare
Now the SysTimer constructor is aligned with #10104 |
CI started |
Test run: SUCCESSSummary: 11 of 11 test jobs passed |
Description
Tickless from us ticker is possible since #9785, but SysTimer cannot be built if lp ticker is not enabled even if the latter is not used. This PR drops such restriction and is a dependency for #10572 .
Pull request type