'displayLogicLevelsInEngineSniffer' on VVT input reports too much into engine sniffer fix #2528

This commit is contained in:
rusefillc 2021-04-04 23:12:07 -04:00
parent f8aa71bf63
commit a9e822f790
2 changed files with 2 additions and 2 deletions

View File

@ -137,7 +137,7 @@ void hwHandleVvtCamSignal(trigger_value_e front, efitick_t nowNt, int index DECL
return;
}
if (CONFIG(displayLogicLevelsInEngineSniffer)) {
if (CONFIG(displayLogicLevelsInEngineSniffer) && isImportantFront) {
if (CONFIG(vvtCamSensorUseRise)) {
// todo: unify TS composite logger code with console Engine Sniffer
// todo: better API to reduce copy/paste?

View File

@ -167,5 +167,5 @@ TEST(sensors, testNB2CamInput) {
// actually position based on VVT!
ASSERT_EQ(totalRevolutionCountBeforeVvtSync + 2, engine->triggerCentral.triggerState.getTotalRevolutionCounter());
ASSERT_EQ(28, waveChart.getSize());
ASSERT_EQ(26, waveChart.getSize());
}