'triggerEventAngle' not really needed
This commit is contained in:
parent
3bf29b6963
commit
f0c4817e1b
|
@ -373,7 +373,6 @@ void TriggerShape::findTriggerPosition(event_trigger_position_s *position,
|
|||
}
|
||||
|
||||
position->triggerEventIndex = triggerEventIndex;
|
||||
position->triggerEventAngle = triggerEventAngle;
|
||||
position->angleOffsetFromTriggerEvent = angle - triggerEventAngle;
|
||||
}
|
||||
|
||||
|
|
|
@ -47,11 +47,6 @@
|
|||
class event_trigger_position_s {
|
||||
public:
|
||||
uint32_t triggerEventIndex = 0;
|
||||
/**
|
||||
* angle of that 'triggerEventIndex' event
|
||||
* todo: Technically we can simply take angle of trigger event from trigger shape by 'triggerEventIndex'?
|
||||
*/
|
||||
angle_t triggerEventAngle = 0;
|
||||
|
||||
angle_t angleOffsetFromTriggerEvent = 0;
|
||||
};
|
||||
|
|
|
@ -283,7 +283,7 @@ static ALWAYS_INLINE void handleSparkEvent(bool limitedSpark, uint32_t trgEventI
|
|||
#if EFI_UNIT_TEST
|
||||
if (verboseMode) {
|
||||
printf("spark dwell@ %d/%d spark@ %d/%d id=%d\r\n", iEvent->dwellPosition.triggerEventIndex, (int)iEvent->dwellPosition.angleOffsetFromTriggerEvent,
|
||||
iEvent->sparkPosition.triggerEventAngle, (int)iEvent->sparkPosition.angleOffsetFromTriggerEvent,
|
||||
iEvent->sparkPosition.triggerEventIndex, (int)iEvent->sparkPosition.angleOffsetFromTriggerEvent,
|
||||
iEvent->sparkId);
|
||||
}
|
||||
#endif
|
||||
|
|
|
@ -351,7 +351,7 @@ TEST(misc, testRpmCalculator) {
|
|||
assertEqualsM("fuel #2", 4.5450, engine->injectionDuration);
|
||||
assertEqualsM("one degree", 111.1111, engine->rpmCalculator.oneDegreeUs);
|
||||
ASSERT_EQ( 1, ilist->isReady) << "size #2";
|
||||
ASSERT_EQ( 0, ilist->elements[0].dwellPosition.triggerEventAngle) << "dwell angle";
|
||||
ASSERT_EQ( 0, ilist->elements[0].dwellPosition.triggerEventIndex) << "dwell @ index";
|
||||
assertEqualsM("dwell offset", 8.5, ilist->elements[0].dwellPosition.angleOffsetFromTriggerEvent);
|
||||
|
||||
ASSERT_EQ( 0, eth.engine.triggerCentral.triggerState.getCurrentIndex()) << "index #2";
|
||||
|
|
Loading…
Reference in New Issue