mirror of https://github.com/ARMmbed/mbed-os.git
Fix LowPowerTickerWrapper operation when suspended
Update the LowPowerTickerWrapper class logic to stop using its internal Timeout object for scheduling LP ticker interrupts after the wrapper has been suspended. Fixes #8278pull/8279/head
parent
83dada3bdc
commit
023603faa7
|
|
@ -32,15 +32,7 @@ void LowPowerTickerWrapper::irq_handler(ticker_irq_handler_type handler)
|
||||||
{
|
{
|
||||||
core_util_critical_section_enter();
|
core_util_critical_section_enter();
|
||||||
|
|
||||||
if (_suspended) {
|
if (_pending_fire_now || _match_check(_intf->read()) || _suspended) {
|
||||||
if (handler) {
|
|
||||||
handler(&data);
|
|
||||||
}
|
|
||||||
core_util_critical_section_exit();
|
|
||||||
return;
|
|
||||||
}
|
|
||||||
|
|
||||||
if (_pending_fire_now || _match_check(_intf->read())) {
|
|
||||||
_timeout.detach();
|
_timeout.detach();
|
||||||
_pending_timeout = false;
|
_pending_timeout = false;
|
||||||
_pending_match = false;
|
_pending_match = false;
|
||||||
|
|
@ -78,6 +70,14 @@ void LowPowerTickerWrapper::resume()
|
||||||
{
|
{
|
||||||
core_util_critical_section_enter();
|
core_util_critical_section_enter();
|
||||||
|
|
||||||
|
// Wait until rescheduling is allowed
|
||||||
|
while (!_set_interrupt_allowed) {
|
||||||
|
timestamp_t current = _intf->read();
|
||||||
|
if (((current - _last_actual_set_interrupt) & _mask) >= _min_count_between_writes) {
|
||||||
|
_set_interrupt_allowed = true;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
_suspended = false;
|
_suspended = false;
|
||||||
|
|
||||||
core_util_critical_section_exit();
|
core_util_critical_section_exit();
|
||||||
|
|
@ -118,7 +118,7 @@ uint32_t LowPowerTickerWrapper::read()
|
||||||
core_util_critical_section_enter();
|
core_util_critical_section_enter();
|
||||||
|
|
||||||
timestamp_t current = _intf->read();
|
timestamp_t current = _intf->read();
|
||||||
if (_match_check(current)) {
|
if (!_suspended && _match_check(current)) {
|
||||||
_intf->fire_interrupt();
|
_intf->fire_interrupt();
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
@ -133,7 +133,13 @@ void LowPowerTickerWrapper::set_interrupt(timestamp_t timestamp)
|
||||||
_last_set_interrupt = _intf->read();
|
_last_set_interrupt = _intf->read();
|
||||||
_cur_match_time = timestamp;
|
_cur_match_time = timestamp;
|
||||||
_pending_match = true;
|
_pending_match = true;
|
||||||
|
if (!_suspended) {
|
||||||
_schedule_match(_last_set_interrupt);
|
_schedule_match(_last_set_interrupt);
|
||||||
|
} else {
|
||||||
|
_intf->set_interrupt(timestamp);
|
||||||
|
_last_actual_set_interrupt = _last_set_interrupt;
|
||||||
|
_set_interrupt_allowed = false;
|
||||||
|
}
|
||||||
|
|
||||||
core_util_critical_section_exit();
|
core_util_critical_section_exit();
|
||||||
}
|
}
|
||||||
|
|
|
||||||
Loading…
Reference in New Issue