spark_logic.h File Reference

Functions
void	onTriggerEventSparkLogic (float rpm, efitick_t edgeTimestamp, float currentPhase, float nextPhase)
void	turnSparkPinHighStartCharging (IgnitionEvent *event)
void	fireSparkAndPrepareNextSchedule (IgnitionEvent *event)
int	getNumberOfSparks (ignition_mode_e mode)
percent_t	getCoilDutyCycle (float rpm)
void	initializeIgnitionActions ()

Function Documentation

fireSparkAndPrepareNextSchedule()

void fireSparkAndPrepareNextSchedule

(

IgnitionEvent *

event

)

TL,DR: each IgnitionEvent is in charge of it's own scheduling forever, we plant next event while finishing handling of the current one

ratio of desired dwell duration to actual dwell duration gives us some idea of how good is input trigger jitter

Definition at line 209 of file spark_logic.cpp.

                                                           {
#if EFI_UNIT_TEST
    if (engine->onIgnitionEvent) {
        engine->onIgnitionEvent(event, false);
    }
#endif
 
    for (int i = 0; i< MAX_OUTPUTS_FOR_IGNITION;i++) {
        IgnitionOutputPin *output = event->outputs[i];
 
        if (output) {
            fireSparkBySettingPinLow(event, output);
        }
    }
 
    efitick_t nowNt = getTimeNowNt();
 
#if EFI_TOOTH_LOGGER
    LogTriggerCoilState(nowNt, false, event->coilIndex);
#endif // EFI_TOOTH_LOGGER
    if (!event->wasSparkLimited) {
        /**
         * ratio of desired dwell duration to actual dwell duration gives us some idea of how good is input trigger jitter
         */
        float actualDwellMs = event->actualDwellTimer.getElapsedSeconds(nowNt) * 1e3;
        float ratio = actualDwellMs / event->sparkDwell;
 
        if (ratio > 1.2) {
            engine->engineState.dwellOverChargeCounter++;
        } else if (ratio < 0.8) {
            engine->engineState.dwellUnderChargeCounter++;
        }
        engine->engineState.dwellActualRatio = ratio;
    }
 
    // now that we've just fired a coil let's prepare the new schedule for the next engine revolution
 
    angle_t dwellAngleDuration = engine->ignitionState.dwellDurationAngle;
    floatms_t sparkDwell = engine->ignitionState.getDwell();
    if (std::isnan(dwellAngleDuration) || std::isnan(sparkDwell)) {
        // we are here if engine has just stopped
        return;
    }
 
    // If there are more sparks to fire, schedule them
    if (event->sparksRemaining > 0) {
        event->sparksRemaining--;
 
        efitick_t nextDwellStart = nowNt + engine->engineState.multispark.delay;
        efitick_t nextFiring = nextDwellStart + engine->engineState.multispark.dwell;
#if SPARK_EXTREME_LOGGING
        efiPrintf("schedule multispark");
#endif /* SPARK_EXTREME_LOGGING */
 
        // We can schedule both of these right away, since we're going for "asap" not "particular angle"
        engine->scheduler.schedule("dwell", &event->dwellStartTimer, nextDwellStart, action_s::make<turnSparkPinHighStartCharging>( event ));
        engine->scheduler.schedule("firing", &event->sparkEvent.eventScheduling, nextFiring, action_s::make<fireSparkAndPrepareNextSchedule>( event ));
    } else {
        if (engineConfiguration->enableTrailingSparks) {
#if SPARK_EXTREME_LOGGING
            efiPrintf("scheduleByAngle TrailingSparks");
#endif /* SPARK_EXTREME_LOGGING */
 
            // Trailing sparks are enabled - schedule an event for the corresponding trailing coil
            scheduleByAngle(
                &event->trailingSparkFire, nowNt, engine->ignitionState.trailingSparkAngle,
                action_s::make<fireTrailingSpark>( &enginePins.trailingCoils[event->coilIndex] )
            );
        }
 
        // If all events have been scheduled, prepare for next time.
        prepareCylinderIgnitionSchedule(dwellAngleDuration, sparkDwell, event);
    }
 
    engine->onSparkFireKnockSense(event->cylinderIndex, nowNt);
}

Referenced by overFireSparkAndPrepareNextSchedule().

Here is the call graph for this function:

Here is the caller graph for this function:

getCoilDutyCycle()

percent_t getCoilDutyCycle

(

float

rpm

)

See also

getInjectorDutyCycle

Definition at line 681 of file spark_logic.cpp.

                                      {
    floatms_t totalPerCycle = engine->ignitionState.getDwell() * getNumberOfSparks(getCurrentIgnitionMode());
    floatms_t engineCycleDuration = getCrankshaftRevolutionTimeMs(rpm) * (getEngineRotationState()->getOperationMode() == TWO_STROKE ? 1 : 2);
    return 100 * totalPerCycle / engineCycleDuration;
}

Referenced by populateFrame(), and updateIgnition().

Here is the call graph for this function:

Here is the caller graph for this function:

getNumberOfSparks()

int getNumberOfSparks

(

ignition_mode_e

mode

)

Number of sparks per physical coil

See also

getNumberOfInjections

Definition at line 662 of file spark_logic.cpp.

                                            {
    switch (mode) {
    case IM_ONE_COIL:
        return engineConfiguration->cylindersCount;
    case IM_TWO_COILS:
        return engineConfiguration->cylindersCount / 2;
    case IM_INDIVIDUAL_COILS:
        return 1;
    case IM_WASTED_SPARK:
        return 2;
    default:
        firmwareError(ObdCode::CUSTOM_ERR_IGNITION_MODE, "Unexpected ignition_mode_e %d", mode);
        return 1;
    }
}

Referenced by getCoilDutyCycle().

Here is the call graph for this function:

Here is the caller graph for this function:

initializeIgnitionActions()

void initializeIgnitionActions

(

)

Definition at line 502 of file spark_logic.cpp.

                                 {
    IgnitionEventList *list = &engine->ignitionEvents;
    angle_t dwellAngle = engine->ignitionState.dwellDurationAngle;
    floatms_t sparkDwell = engine->ignitionState.getDwell();
    if (std::isnan(engine->engineState.timingAdvance[0]) || std::isnan(dwellAngle)) {
        // error should already be reported
        // need to invalidate previous ignition schedule
        list->isReady = false;
        return;
    }
    efiAssertVoid(ObdCode::CUSTOM_ERR_6592, engineConfiguration->cylindersCount > 0, "cylindersCount");
 
    for (size_t cylinderIndex = 0; cylinderIndex < engineConfiguration->cylindersCount; cylinderIndex++) {
        list->elements[cylinderIndex].cylinderIndex = cylinderIndex;
        prepareCylinderIgnitionSchedule(dwellAngle, sparkDwell, &list->elements[cylinderIndex]);
    }
    list->isReady = true;
}

Referenced by prepareIgnitionSchedule().

Here is the call graph for this function:

Here is the caller graph for this function:

onTriggerEventSparkLogic()

void onTriggerEventSparkLogic	(	float	rpm,
		efitick_t	edgeTimestamp,
		float	currentPhase,
		float	nextPhase
	)

Ignition schedule is defined once per revolution See initializeIgnitionActions()

Definition at line 545 of file spark_logic.cpp.

                                                                                                       {
    ScopePerf perf(PE::OnTriggerEventSparkLogic);
 
    if (!engineConfiguration->isIgnitionEnabled) {
        return;
    }
 
    LimpState limitedSparkState = getLimpManager()->allowIgnition();
 
    // todo: eliminate state copy logic by giving limpManager it's owm limp_manager.txt and leveraging LiveData
    engine->outputChannels.sparkCutReason = (int8_t)limitedSparkState.reason;
    bool limitedSpark = !limitedSparkState.value;
 
    const floatms_t dwellMs = engine->ignitionState.getDwell();
    if (std::isnan(dwellMs) || dwellMs <= 0) {
        warning(ObdCode::CUSTOM_DWELL, "invalid dwell to handle: %.2f", dwellMs);
        return;
    }
 
    if (!engine->ignitionEvents.isReady) {
        prepareIgnitionSchedule();
    }
 
 
    /**
     * Ignition schedule is defined once per revolution
     * See initializeIgnitionActions()
     */
 
 
    // Only apply odd cylinder count wasted logic if:
    // - odd cyl count
    // - current mode is wasted spark
    // - four stroke
    bool enableOddCylinderWastedSpark =
        engine->engineState.useOddFireWastedSpark
        && getCurrentIgnitionMode() == IM_WASTED_SPARK;
 
    if (engine->ignitionEvents.isReady) {
        for (size_t i = 0; i < engineConfiguration->cylindersCount; i++) {
            IgnitionEvent *event = &engine->ignitionEvents.elements[i];
 
            angle_t dwellAngle = event->dwellAngle;
 
            angle_t sparkAngle = event->sparkAngle;
            if (std::isnan(sparkAngle)) {
                warning(ObdCode::CUSTOM_ADVANCE_SPARK, "NaN advance");
                continue;
            }
 
            bool isOddCylWastedEvent = false;
            if (enableOddCylinderWastedSpark) {
                auto dwellAngleWastedEvent = dwellAngle + 360;
                if (dwellAngleWastedEvent > 720) {
                    dwellAngleWastedEvent -= 720;
                }
 
                // Check whether this event hits 360 degrees out from now (ie, wasted spark),
                // and if so, twiddle the dwell and spark angles so it happens now instead
                isOddCylWastedEvent = isPhaseInRange(dwellAngleWastedEvent, currentPhase, nextPhase);
 
                if (isOddCylWastedEvent) {
                    dwellAngle = dwellAngleWastedEvent;
 
                    sparkAngle += 360;
                    if (sparkAngle > 720) {
                        sparkAngle -= 720;
                    }
                }
            }
 
            if (!isOddCylWastedEvent && !isPhaseInRange(dwellAngle, currentPhase, nextPhase)) {
                continue;
            }
 
            if (i == 0 && engineConfiguration->artificialTestMisfire && (getRevolutionCounter() % ((int)engineConfiguration->scriptSetting[5]) == 0)) {
                // artificial misfire on cylinder #1 for testing purposes
                // enable artificialMisfire
                warning(ObdCode::CUSTOM_ARTIFICIAL_MISFIRE, "artificial misfire on cylinder #1 for testing purposes %d", engine->engineState.globalSparkCounter);
                continue;
            }
#if EFI_LAUNCH_CONTROL
            bool sparkLimited = engine->softSparkLimiter.shouldSkip() || engine->hardSparkLimiter.shouldSkip();
            engine->ignitionState.luaIgnitionSkip = sparkLimited;
            if (sparkLimited) {
                continue;
            }
#endif // EFI_LAUNCH_CONTROL
 
#if EFI_ANTILAG_SYSTEM && EFI_LAUNCH_CONTROL
/*
       if (engine->antilagController.isAntilagCondition) {
            if (engine->ALSsoftSparkLimiter.shouldSkip()) {
                continue;
            }
        }
        float throttleIntent = Sensor::getOrZero(SensorType::DriverThrottleIntent);
        engine->antilagController.timingALSSkip = interpolate3d(
            config->ALSIgnSkipTable,
            config->alsIgnSkipLoadBins, throttleIntent,
            config->alsIgnSkiprpmBins, rpm
        );
 
            auto ALSSkipRatio = engine->antilagController.timingALSSkip;
            engine->ALSsoftSparkLimiter.setTargetSkipRatio(ALSSkipRatio/100);
*/
#endif // EFI_ANTILAG_SYSTEM
 
            scheduleSparkEvent(limitedSpark, event, rpm, dwellMs, dwellAngle, sparkAngle, edgeTimestamp, currentPhase, nextPhase);
        }
    }
}

Referenced by mainTriggerCallback().

Here is the call graph for this function:

Here is the caller graph for this function:

turnSparkPinHighStartCharging()

void turnSparkPinHighStartCharging

(

IgnitionEvent *

event

)

Definition at line 332 of file spark_logic.cpp.

                                                         {
    efitick_t nowNt = getTimeNowNt();
 
    event->actualDwellTimer.reset(nowNt);
 
  bool skippedDwellDueToTriggerNoised = false;
    for (int i = 0; i< MAX_OUTPUTS_FOR_IGNITION;i++) {
        IgnitionOutputPin *output = event->outputs[i];
        if (output != NULL) {
          // at the moment we have a funny xor as if outputs could have different destiny. That's probably an over exaggeration,
          // realistically it should be enough to check the sequencing of only the first output but that would be less elegant
          //
          // maybe it would have need nicer if instead of an array of outputs we had a linked list of outputs? but that's just daydreaming.
            skippedDwellDueToTriggerNoised |= startDwellByTurningSparkPinHigh(event, output);
        }
    }
 
#if EFI_UNIT_TEST
    engine->incrementBailedOnDwellCount();
#endif
 
 
  if (!skippedDwellDueToTriggerNoised) {
 
#if EFI_UNIT_TEST
    if (engine->onIgnitionEvent) {
        engine->onIgnitionEvent(event, true);
    }
#endif
 
#if EFI_TOOTH_LOGGER
    LogTriggerCoilState(nowNt, true, event->coilIndex);
#endif // EFI_TOOTH_LOGGER
  }
 
 
    if (engineConfiguration->enableTrailingSparks) {
        IgnitionOutputPin *output = &enginePins.trailingCoils[event->coilIndex];
        // Trailing sparks are enabled - schedule an event for the corresponding trailing coil
        scheduleByAngle(
            &event->trailingSparkCharge, nowNt, engine->ignitionState.trailingSparkAngle,
            action_s::make<chargeTrailingSpark>( output )
        );
    }
}

Here is the call graph for this function:

Go to the source code of this file.