simplify quad vvt test #3018

better parameter name and explicit constraint clarification

firmware/controllers/trigger/trigger_decoder.cpp

@@ -376,9 +376,10 @@ void TriggerCentral::validateCamVvtCounters() {
 	}
 }
 
-bool TriggerState::syncSymmetricalCrank(int mod, int remainder) {
+bool TriggerState::syncSymmetricalCrank(int divider, int remainder) {
+	efiAssert(OBD_PCM_Processor_Fault, remainder < divider, "syncSymmetricalCrank", false);
 	bool isSync = false;
-	while (getTotalRevolutionCounter() % mod != remainder) {
+	while (getTotalRevolutionCounter() % divider != remainder) {
 		/**
 		 * we are here if we've detected the cam sensor within the wrong crank phase
 		 * let's increase the trigger event counter, that would adjust the state of

firmware/controllers/trigger/trigger_decoder.h

@@ -90,7 +90,7 @@ public:
 	 * this is important for crank-based virtual trigger and VVT magic
 	 */
 	void incrementTotalEventCounter();
-	bool syncSymmetricalCrank(int mod, int remainder);
+	bool syncSymmetricalCrank(int divider, int remainder);
 
 	efitime_t getTotalEventCounter() const;