rusefi
/
custom-board-bundle-sample-fork-of-rusefi
mirror of https://github.com/rusefi/custom-board-bundle-sample-fork-of-rusefi.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

comments & improve logic

This commit is contained in:
Matthew Kennedy 2020-07-16 23:55:41 -07:00
parent b3261f25f3
commit b7336e953e
2 changed files with 13 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
firmware/controllers/engine_cycle/rpm_calculator.cpp
View File

@ -165,8 +165,14 @@ void RpmCalculator::setRpmValue(float value DECLARE_ENGINE_PARAMETER_SUFFIX) {
#if EFI_ENGINE_CONTROL #if EFI_ENGINE_CONTROL
// This presumably fixes injection mode change for cranking-to-running transition. // This presumably fixes injection mode change for cranking-to-running transition.
// 'isSimultanious' flag should be updated for events if injection modes differ for cranking and running. // 'isSimultanious' flag should be updated for events if injection modes differ for cranking and running.
if (state != oldState) { if (state != oldState && CONFIG(crankingInjectionMode) != CONFIG(injectionMode)) {
// Reset the state of all injectors: when we change fueling modes, we could
// immediately reschedule an injection that's currently underway. That will cause
// the injector's overlappingCounter to get out of sync with reality. As the fix,
// every injector's state is forcibly reset just before we could cause that to happen.
engine->injectionEvents.resetOverlapping(); engine->injectionEvents.resetOverlapping();
// reschedule all injection events now that we've reset them
engine->injectionEvents.addFuelEvents(PASS_ENGINE_PARAMETER_SIGNATURE); engine->injectionEvents.addFuelEvents(PASS_ENGINE_PARAMETER_SIGNATURE);
} }
#endif #endif

7
firmware/controllers/system/efi_gpio.cpp
View File

@ -277,7 +277,12 @@ bool NamedOutputPin::stop() {
} }
void InjectorOutputPin::reset() { void InjectorOutputPin::reset() {
overlappingCounter = 0; // If this injector was open, close it and reset state
if (overlappingCounter != 0) {
overlappingCounter = 0;
setValue(0);
}
// todo: this could be refactored by calling some super-reset method // todo: this could be refactored by calling some super-reset method
currentLogicValue = INITIAL_PIN_STATE; currentLogicValue = INITIAL_PIN_STATE;
} }