rusefi
/
rusefi-full
mirror of https://github.com/rusefi/rusefi-full.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Trigger decoding issues (60-2) #1488 

interesting stuff potentially with a not so great setting
This commit is contained in:
rusefi 2020-06-14 23:41:39 -04:00
parent f9087fdf72
commit f3344753e8
2 changed files with 6 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
firmware/controllers/system/timer/event_queue.cpp
View File

@ -94,10 +94,13 @@ efitime_t EventQueue::getNextEventTime(efitime_t nowX) const {
if (head != NULL) { if (head != NULL) {
if (head->momentX <= nowX) { if (head->momentX <= nowX) {
/** /**
* We are here if action timestamp is in the past * We are here if action timestamp is in the past. We should rarely be here since this 'getNextEventTime()' is
* always invoked by 'scheduleTimerCallback' which is always invoked right after 'executeAllPendingActions' - but still,
* for events which are really close to each other we would end up here.
* *
* looks like we end up here after 'writeconfig' (which freezes the firmware) - we are late * looks like we end up here after 'writeconfig' (which freezes the firmware) - we are late
* for the next scheduled event * for the next scheduled event
*
*/ */
efitime_t aBitInTheFuture = nowX + lateDelay; efitime_t aBitInTheFuture = nowX + lateDelay;
return aBitInTheFuture; return aBitInTheFuture;

3
firmware/controllers/system/timer/single_timer_executor.cpp
View File

@ -51,7 +51,8 @@ void globalTimerCallback() {
SingleTimerExecutor::SingleTimerExecutor() { SingleTimerExecutor::SingleTimerExecutor() {
/** /**
* todo: a good comment * See comments in "getNextEventTime"
* todo: is this current value of 'lateDelay' a bit too high?
*/ */
queue.setLateDelay(US2NT(100)); queue.setLateDelay(US2NT(100));
} }