Clean up prints

Author: nseidle

Firmware/RTK_Everywhere/GNSS.ino

@@ -115,11 +115,6 @@ void gnssDetectReceiverType()
     if (productVariant != RTK_FLEX)
         return;
 
-    // settings.detectedGnssReceiver = GNSS_RECEIVER_LG290P;
-    // gnss = (GNSS *)new GNSS_LG290P();
-    // systemPrintln("Starting GNSS receiver: LG290P");
-    // return;
-
     gnssBoot(); // Tell GNSS to run
 
     // TODO remove after testing, force retest on each boot
@@ -131,6 +126,7 @@ void gnssDetectReceiverType()
 #ifdef COMPILE_LG290P
         if (lg290pIsPresent() == true)
         {
+            systemPrintln("Auto-detected GNSS receiver: LG290P");
             settings.detectedGnssReceiver = GNSS_RECEIVER_LG290P;
             recordSystemSettings(); // Record the detected GNSS receiver and avoid this test in the future
         }
@@ -142,6 +138,7 @@ void gnssDetectReceiverType()
 // TODO - this uses UART2, but Flex is UART1. We need to make the mosaic send routines flexible to use
 // whichever UART we specify.
 // if (mosaicIsPresent() == true)
+            // systemPrintln("Auto-detected GNSS receiver: mosaic-X5");
 //     settings.detectedGnssReceiver = GNSS_RECEIVER_MOSAIC_X5;
 // recordSystemSettings(); // Record the detected GNSS receiver and avoid this test in the future
 #else  // COMPILE_MOSAICX5
@@ -155,15 +152,13 @@ void gnssDetectReceiverType()
 #ifdef COMPILE_LG290P
         gnss = (GNSS *)new GNSS_LG290P();
         present.gnss_lg290p = true;
-        systemPrintln("Starting GNSS receiver: LG290P");
 #endif // COMPILE_LGP290P
     }
     else if (settings.detectedGnssReceiver == GNSS_RECEIVER_MOSAIC_X5)
     {
 #ifdef COMPILE_MOSAICX5
         gnss = (GNSS *)new GNSS_MOSAIC();
         present.gnss_mosaicX5 = true;
-        systemPrintln("Starting GNSS receiver: mosaic-X5");
 #endif // COMPILE_MOSAICX5
     }
 

Firmware/RTK_Everywhere/GNSS_LG290P.ino

@@ -2393,17 +2393,14 @@ bool lg290pIsPresent()
     // LG290P communicates at 460800bps.
     uint32_t platformGnssCommunicationRate = 115200 * 4;
 
-    // systemPrintf("Starting GNSS primary UART at %d on pins RX: %d TX: %d\r\n", platformGnssCommunicationRate,
-    //              pin_GnssUart_RX, pin_GnssUart_TX);
-
     serialTestGNSS.begin(platformGnssCommunicationRate, SERIAL_8N1, pin_GnssUart_RX, pin_GnssUart_TX);
 
     LG290P lg290p;
 
     if (lg290p.begin(serialTestGNSS) == true) // Give the serial port over to the library
     {
-        // if (settings.debugGnss)
-        systemPrintln("LG290P detected");
+         if (settings.debugGnss)
+            systemPrintln("LG290P detected");
         serialTestGNSS.end();
         return true;
     }

Firmware/RTK_Everywhere/System.ino

@@ -923,7 +923,6 @@ void gpioExpanderGnssReset()
     {
         if (settings.detectedGnssReceiver != GNSS_RECEIVER_LG290P)
         {
-            Serial.println("Driving GNSS reset LOW");
             gpioExpanderSwitches->digitalWrite(gpioExpanderSwitch_GNSS_Reset, LOW);
         }
         else