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

'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 f3c46f2720
commit 9753b6e020
2 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
firmware/controllers/trigger/trigger_central.cpp
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?

2
unit_tests/tests/trigger/test_cam_vvt_input.cpp
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());
}