Move RTCM movement out of I2C ZED transport layer

Fix #222

Author: nseidle

Firmware/RTK_Everywhere/Base.ino

@@ -1,10 +1,20 @@
+// This function gets called when an RTCM packet passes parser check in processUart1Message() task
+// Pass data along to NTRIP Server, or ESP-NOW radio
+void processRTCM(uint8_t *rtcmData, uint16_t dataLength)
+{
+    // Give this byte to the various possible transmission methods
+    for (int x = 0; x < dataLength; x++)
+    {
+        for (int serverIndex = 0; serverIndex < NTRIP_SERVER_MAX; serverIndex++)
+            ntripServerProcessRTCM(serverIndex, rtcmData[x]);
+    }
 
+    for (int x = 0; x < dataLength; x++)
+        espnowProcessRTCM(rtcmData[x]);
+
+    rtcmLastPacketSent = millis();
+    rtcmPacketsSent++;
 
-// This function gets called from the SparkFun u-blox Arduino Library.
-// As each RTCM byte comes in you can specify what to do with it
-// Useful for passing the RTCM correction data to a radio, Ntrip broadcaster, etc.
-void DevUBLOXGNSS::processRTCM(uint8_t incoming)
-{
     // Check for too many digits
     if (settings.enableResetDisplay == true)
     {
@@ -16,22 +26,4 @@ void DevUBLOXGNSS::processRTCM(uint8_t incoming)
         if (rtcmPacketsSent > 999)
             rtcmPacketsSent = 1; // Trim to three digits to avoid log icon and increasing bar
     }
-
-    // Determine if we should check this byte with the RTCM checker or simply pass it along
-    bool passAlongIncomingByte = true;
-
-    if (settings.enableRtcmMessageChecking == true)
-        passAlongIncomingByte &= checkRtcmMessage(incoming);
-
-    // Give this byte to the various possible transmission methods
-    if (passAlongIncomingByte)
-    {
-        rtcmLastReceived = millis();
-        rtcmBytesSent++;
-
-        for (int serverIndex = 0; serverIndex < NTRIP_SERVER_MAX; serverIndex++)
-            ntripServerProcessRTCM(serverIndex, incoming);
-
-        espnowProcessRTCM(incoming);
-    }
-}
+}

Firmware/RTK_Everywhere/RTK_Everywhere.ino

@@ -383,7 +383,7 @@ uint16_t ARPECEFH;
 
 const byte haeNumberOfDecimals = 8; // Used for printing and transmitting lat/lon
 bool lBandForceGetKeys; // Used to allow key update from display
-unsigned long rtcmLastPacketReceived;
+unsigned long rtcmLastPacketReceived; //Time stamp of RTCM coming in (from BT, NTRIP, etc)
 // Monitors the last time we received RTCM. Proctors PMP vs RTCM prioritization.
 //=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=
 
@@ -667,8 +667,7 @@ bool logIncreasing; // Goes true when log file is greater than lastLogSize or lo
 bool reuseLastLog;  // Goes true if we have a reset due to software (rather than POR)
 
 uint16_t rtcmPacketsSent; // Used to count RTCM packets sent via processRTCM()
-uint32_t rtcmBytesSent;
-uint32_t rtcmLastReceived;
+uint32_t rtcmLastPacketSent;  //Time stamp of RTCM going out (to NTRIP Server, ESP-NOW, etc)
 
 uint32_t maxSurveyInWait_s = 60L * 15L; // Re-start survey-in after X seconds
 
@@ -1421,13 +1420,6 @@ void rtcUpdate()
 // Internal ESP NOW radio - Use the ESP32 to directly transmit/receive RTCM over 2.4GHz (no WiFi needed)
 void updateRadio()
 {
-    // If we have not gotten new RTCM bytes for a period of time, assume the end of frame
-    if (millis() - rtcmLastReceived > 50 && rtcmBytesSent > 0)
-    {
-        rtcmBytesSent = 0;
-        rtcmPacketsSent++; // If not checking RTCM CRC, count based on timeout
-    }
-
 #ifdef COMPILE_ESPNOW
     if (settings.enableEspNow == true)
     {