From 7369126d91cca1eb59da935fed83eac46555c3fc Mon Sep 17 00:00:00 2001 From: Andrey Date: Wed, 7 Sep 2022 15:38:15 -0400 Subject: [PATCH] tear down Engine god object #4511 just trying different things --- .../controllers/engine_cycle/main_trigger_callback.cpp | 5 ++--- firmware/controllers/engine_cycle/spark_logic.cpp | 8 ++++++-- 2 files changed, 8 insertions(+), 5 deletions(-) diff --git a/firmware/controllers/engine_cycle/main_trigger_callback.cpp b/firmware/controllers/engine_cycle/main_trigger_callback.cpp index 7f40f88564..993716344c 100644 --- a/firmware/controllers/engine_cycle/main_trigger_callback.cpp +++ b/firmware/controllers/engine_cycle/main_trigger_callback.cpp @@ -28,8 +28,6 @@ #include "rpm_calculator_api.h" // end of injection -#include "os_access.h" - #if EFI_PRINTF_FUEL_DETAILS bool printFuelDebug = false; #endif // EFI_PRINTF_FUEL_DETAILS @@ -47,9 +45,10 @@ #include "local_version_holder.h" #include "event_queue.h" #include "injector_model.h" + #if EFI_LAUNCH_CONTROL #include "launch_control.h" -#endif +#endif // EFI_LAUNCH_CONTROL #include "backup_ram.h" diff --git a/firmware/controllers/engine_cycle/spark_logic.cpp b/firmware/controllers/engine_cycle/spark_logic.cpp index ae9930aa84..71d56d2de9 100644 --- a/firmware/controllers/engine_cycle/spark_logic.cpp +++ b/firmware/controllers/engine_cycle/spark_logic.cpp @@ -8,7 +8,11 @@ #include "pch.h" #include "spark_logic.h" -#include "os_access.h" + +// dependency injection +#include "engine_state.h" +#include "rpm_calculator_api.h" +// end of injection #include "utlist.h" #include "event_queue.h" @@ -75,7 +79,7 @@ static void prepareCylinderIgnitionSchedule(angle_t dwellAngleDuration, floatms_ const angle_t sparkAngle = // Negate because timing *before* TDC, and we schedule *after* TDC - - engine->engineState.timingAdvance[event->cylinderNumber] + - getEngineState()->timingAdvance[event->cylinderNumber] // Offset by this cylinder's position in the cycle + getCylinderAngle(event->cylinderIndex, event->cylinderNumber) // Pull any extra timing for knock retard