From 271dddf61ee378e6e916492ca80552866aae9b90 Mon Sep 17 00:00:00 2001 From: rusefi Date: Sat, 18 Jun 2022 15:18:45 -0400 Subject: [PATCH] verbose trigger sync should reuse engineSnifferRpmThreshold fix #4259 --- firmware/CHANGELOG.md | 1 + firmware/controllers/trigger/trigger_decoder.cpp | 4 +++- 2 files changed, 4 insertions(+), 1 deletion(-) diff --git a/firmware/CHANGELOG.md b/firmware/CHANGELOG.md index 5262a25ee0..e1fc761343 100644 --- a/firmware/CHANGELOG.md +++ b/firmware/CHANGELOG.md @@ -24,6 +24,7 @@ Release template (copy/paste this for new release): # Changelog ### Added + - verbose trigger sync should reuse engineSnifferRpmThreshold #4259 ### Fixed - Improved logic used to disambiguate trigger sync using cam/VVT information. Engine now runs in wasted spark until cam sync is achieved, at which point it switches to fully sequential. #4099 diff --git a/firmware/controllers/trigger/trigger_decoder.cpp b/firmware/controllers/trigger/trigger_decoder.cpp index 8c7bc97a09..455de2019d 100644 --- a/firmware/controllers/trigger/trigger_decoder.cpp +++ b/firmware/controllers/trigger/trigger_decoder.cpp @@ -559,7 +559,9 @@ expected TriggerDecoderBase::decodeTriggerEvent( #endif /* EFI_UNIT_TEST */ #if EFI_PROD_CODE || EFI_SIMULATOR - if (triggerConfiguration.VerboseTriggerSynchDetails || (someSortOfTriggerError() && !silentTriggerError)) { + bool verbose = engine->isEngineSnifferEnabled && triggerConfiguration.VerboseTriggerSynchDetails; + + if (verbose || (someSortOfTriggerError() && !silentTriggerError)) { int rpm = Sensor::getOrZero(SensorType::Rpm); floatms_t engineCycleDuration = getEngineCycleDuration(rpm);