TT_RENIX_66_2_2_2

firmware/controllers/core/state_sequence.h

@@ -17,7 +17,7 @@
  * Looks like 252 is explained by 60 tooth * 2 (number of fronts) * 2 (number of crank rotations within engine cycle)
  */
 #ifndef PWM_PHASE_MAX_COUNT
-#define PWM_PHASE_MAX_COUNT 252
+#define PWM_PHASE_MAX_COUNT 280
 #endif /* PWM_PHASE_MAX_COUNT */
 #define PWM_PHASE_MAX_WAVE_PER_PWM 3
 

firmware/controllers/trigger/decoders/trigger_renix.cpp

@@ -10,10 +10,7 @@
 
 #include "trigger_renix.h"
 
-// TT_RENIX_44_2_2
-void initializeRenix44_2_2(TriggerWaveform *s) {
-	s->initialize(FOUR_STROKE_SYMMETRICAL_CRANK_SENSOR);
-
+static void commonRenix(TriggerWaveform *s) {
 	// 44-2-2 is symmetrical so we only need to define one half
 	int count = 22;
 	float tooth = s->getCycleDuration() / count; // hint: tooth = 8.181818 degrees
@@ -29,5 +26,16 @@ void initializeRenix44_2_2(TriggerWaveform *s) {
 
 	// float math error accumulates at this point so we have to spell out 180
 	s->addEventAngle(s->getCycleDuration(), T_PRIMARY, TV_FALL);
-
+}
+
+// TT_RENIX_44_2_2
+void initializeRenix44_2_2(TriggerWaveform *s) {
+	s->initialize(FOUR_STROKE_SYMMETRICAL_CRANK_SENSOR);
+	commonRenix(s);
+}
+
+// TT_RENIX_66_2_2_2
+void initializeRenix66_2_2(TriggerWaveform *s) {
+	s->initialize(FOUR_STROKE_THREE_TIMES_CRANK_SENSOR);
+	commonRenix(s);
 }

firmware/controllers/trigger/decoders/trigger_structure.cpp

@@ -471,6 +471,9 @@ void TriggerWaveform::initializeTriggerWaveform(Logging *logger, operation_mode_
 		break;
 
 	case TT_RENIX_66_2_2_2:
+		initializeRenix66_2_2(this);
+		break;
+
 	case TT_RENIX_44_2_2:
 		initializeRenix44_2_2(this);
 		break;

unit_tests/.gitignore

@@ -2,4 +2,4 @@
 build/
 gcov_working_area
 triggers
-
+triggers.txt