andreika-git
/
rusefi
mirror of https://github.com/andreika-git/rusefi.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

3-0 dies for no reason at idle speeds #7002 (#7006) 

different gaps for different wheels
This commit is contained in:
rusefillc 2024-11-04 10:14:24 -05:00 committed by GitHub
parent a3c7629cbe
commit b538494370
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
1 changed files with 6 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
firmware/controllers/trigger/decoders/trigger_universal.cpp
View File

@ -148,17 +148,17 @@ void configureQuickStartSenderWheel(TriggerWaveform *s) {
s->addToothRiseFall(360, /* width*/ 70); s->addToothRiseFall(360, /* width*/ 70);
} }
static void commonSymmetrical(TriggerWaveform* s, int count) { static void commonSymmetrical(TriggerWaveform* s, int count, float gapFrom, float gapTo) {
s->shapeWithoutTdc = true; s->shapeWithoutTdc = true;
// Sync after 2 good teeth // Sync after 2 good teeth
for (size_t i = 0; i < 2; i++) { for (size_t i = 0; i < 2; i++) {
/** /**
* https://github.com/rusefi/rusefi/issues/4943#issuecomment-1376289608 * https://github.com/rusefi/rusefi/issues/4943#issuecomment-1376289608
* gaps would be nice during running but horrible during cranking * gaps would be nice during running but horrible during running
* Hopefully we do not want variable gap logic yet? * Hopefully we do not want variable gap logic yet?
*/ */
s->setTriggerSynchronizationGap3(i, 0.2f, 3.4f); s->setTriggerSynchronizationGap3(i, gapFrom, gapTo);
} }
float width = 360 / count; float width = 360 / count;
@ -175,15 +175,15 @@ void configure12ToothCrank(TriggerWaveform* s) {
s->initialize(FOUR_STROKE_TWELVE_TIMES_CRANK_SENSOR, SyncEdge::RiseOnly); s->initialize(FOUR_STROKE_TWELVE_TIMES_CRANK_SENSOR, SyncEdge::RiseOnly);
// 2JZ would be global trigger offset 65 but same wheel could be Honda, not hard coding for now // 2JZ would be global trigger offset 65 but same wheel could be Honda, not hard coding for now
commonSymmetrical(s, 12); commonSymmetrical(s, 12, 0.2f, 3.4f);
} }
void configure3ToothCrank(TriggerWaveform* s) { void configure3ToothCrank(TriggerWaveform* s) {
s->initialize(FOUR_STROKE_THREE_TIMES_CRANK_SENSOR, SyncEdge::RiseOnly); s->initialize(FOUR_STROKE_THREE_TIMES_CRANK_SENSOR, SyncEdge::RiseOnly);
commonSymmetrical(s, 3); commonSymmetrical(s, 3, 0.7, 1.4);
} }
void configure6ToothCrank(TriggerWaveform* s) { void configure6ToothCrank(TriggerWaveform* s) {
s->initialize(FOUR_STROKE_SIX_TIMES_CRANK_SENSOR, SyncEdge::RiseOnly); s->initialize(FOUR_STROKE_SIX_TIMES_CRANK_SENSOR, SyncEdge::RiseOnly);
commonSymmetrical(s, 6); commonSymmetrical(s, 6, 0.7, 1.4);
} }