Add printGNSSDebugMessages and gnssFactoryDefault

Author: PaulZC

Firmware/OpenLog_Artemis/autoDetect.ino

@@ -353,6 +353,7 @@ bool beginQwiicDevices()
           SFE_UBLOX_GNSS *tempDevice = (SFE_UBLOX_GNSS *)temp->classPtr;
           struct_uBlox *nodeSetting = (struct_uBlox *)temp->configPtr; //Create a local pointer that points to same spot as node does
           if (nodeSetting->powerOnDelayMillis > qwiicPowerOnDelayMillis) qwiicPowerOnDelayMillis = nodeSetting->powerOnDelayMillis; // Increase qwiicPowerOnDelayMillis if required
+          if(settings.printGNSSDebugMessages == true) tempDevice->enableDebugging(); // Enable debug messages if required
           temp->online = tempDevice->begin(qwiic, temp->address); //Wire port, Address
           setQwiicPullups(settings.qwiicBusPullUps); //Re-enable pullups.
         }
@@ -1206,7 +1207,7 @@ deviceType_e testDevice(uint8_t i2cAddress, uint8_t muxAddress, uint8_t portNumb
         //Confidence: High - Sends/receives CRC checked data response
         setQwiicPullups(0); //Disable pullups to minimize CRC issues
         SFE_UBLOX_GNSS sensor;
-        if(settings.printDebugMessages == true) sensor.enableDebugging(); // Enable debug messages if required
+        if(settings.printGNSSDebugMessages == true) sensor.enableDebugging(); // Enable debug messages if required
         if (sensor.begin(qwiic, i2cAddress) == true) //Wire port, address
         {
           setQwiicPullups(settings.qwiicBusPullUps); //Re-enable pullups to prevent ghosts at 0x43 onwards

Firmware/OpenLog_Artemis/menuAttachedDevices.ino

@@ -1006,6 +1006,8 @@ void menuConfigure_uBlox(void *configPtr)
       if (sensorSetting->useAutoPVT == true) SerialPrintln(F("Yes"));
       else SerialPrintln(F("No"));
 
+      SerialPrintln(F("16) Reset GNSS to factory defaults"));
+
       SerialFlush();
     }
     SerialPrintln(F("x) Exit"));
@@ -1051,6 +1053,18 @@ void menuConfigure_uBlox(void *configPtr)
       }
       else if (incoming == 15)
         sensorSetting->useAutoPVT ^= 1;
+      else if (incoming == 16)
+      {
+        SerialPrintln(F("Reset GNSS module to factory defaults. This will take 5 seconds to complete."));
+        SerialPrintln(F("Are you sure? Press 'y' to confirm: "));
+        byte bContinue = getByteChoice(menuTimeout);
+        if (bContinue == 'y')
+        {
+          gnssFactoryDefault();
+        }
+        else
+          SerialPrintln(F("Reset GNSS aborted"));
+      } 
       else if (incoming == STATUS_PRESSED_X)
         break;
       else if (incoming == STATUS_GETNUMBER_TIMEOUT)
@@ -1065,7 +1079,6 @@ void menuConfigure_uBlox(void *configPtr)
     else
       printUnknown(incoming);
   }
-
 }
 
 bool isUbloxAttached()
@@ -1102,6 +1115,9 @@ void getUbloxDateTime(int &year, int &month, int &day, int &hour, int &minute, i
           SFE_UBLOX_GNSS *nodeDevice = (SFE_UBLOX_GNSS *)temp->classPtr;
           struct_uBlox *nodeSetting = (struct_uBlox *)temp->configPtr;
 
+          //If autoPVT is enabled, flush the data to make sure we get fresh date and time
+          if (nodeSetting->useAutoPVT) nodeDevice->flushPVT();
+
           //Get latested date/time from GPS
           //These will be extracted from a single PVT packet
           year = nodeDevice->getYear();
@@ -1121,6 +1137,37 @@ void getUbloxDateTime(int &year, int &month, int &day, int &hour, int &minute, i
   }
 }
 
+void gnssFactoryDefault(void)
+{
+  //Step through node list
+  node *temp = head;
+
+  while (temp != NULL)
+  {
+    switch (temp->deviceType)
+    {
+      case DEVICE_GPS_UBLOX:
+        {
+          setQwiicPullups(0); //Disable pullups to minimize CRC issues
+
+          SFE_UBLOX_GNSS *nodeDevice = (SFE_UBLOX_GNSS *)temp->classPtr;
+          struct_uBlox *nodeSetting = (struct_uBlox *)temp->configPtr;
+
+          //Reset the module to the factory defaults
+          nodeDevice->factoryDefault();
+          
+          delay(5000); //Blocking delay to allow module to reset
+
+          nodeDevice->setI2COutput(COM_TYPE_UBX); //Set the I2C port to output UBX only (turn off NMEA noise)
+          nodeDevice->saveConfigSelective(VAL_CFG_SUBSEC_IOPORT); //Save (only) the current ioPortsettings to flash and BBR
+
+          setQwiicPullups(settings.qwiicBusPullUps); //Re-enable pullups
+        }
+    }
+    temp = temp->next;
+  }
+}
+
 void menuConfigure_MCP9600(void *configPtr)
 {
   struct_MCP9600 *sensorSetting = (struct_MCP9600*)configPtr;

Firmware/OpenLog_Artemis/menuDebug.ino

@@ -13,6 +13,10 @@ void menuDebug()
     if (settings.resetOnZeroDeviceCount == true) SerialPrintln(F("Enabled"));
     else SerialPrintln(F("Disabled"));
 
+    SerialPrint(F("3) GNSS Debug Messages: "));
+    if (settings.printGNSSDebugMessages == true) SerialPrintln(F("Enabled"));
+    else SerialPrintln(F("Disabled"));
+
     SerialPrintln(F("x) Exit"));
 
     byte incoming = getByteChoice(menuTimeout); //Timeout after x seconds
@@ -42,6 +46,10 @@ void menuDebug()
         settings.resetOnZeroDeviceCount ^= 1;
       }
     }
+   else if (incoming == '3')
+    {
+      settings.printGNSSDebugMessages ^= 1;
+    }
     else if (incoming == 'x')
       break;
     else if (incoming == STATUS_GETBYTE_TIMEOUT)

Firmware/OpenLog_Artemis/menuTimeStamp.ino

@@ -57,7 +57,7 @@ void menuTimeStamp()
       sprintf(rtcHundredths, "0%d", myRTC.hundredths);
     else
       sprintf(rtcHundredths, "%d", myRTC.hundredths);
-    sprintf(rtcTime, "%s:%s:%s.%s,", rtcHour, rtcMin, rtcSec, rtcHundredths);
+    sprintf(rtcTime, "%s:%s:%s.%s", rtcHour, rtcMin, rtcSec, rtcHundredths);
 
     SerialPrintln(rtcTime);
 
@@ -174,6 +174,8 @@ void menuTimeStamp()
         SerialPrint(F("Enter current day (1 to 31): "));
         dd = getNumber(menuTimeout); //Timeout after x seconds
 
+        myRTC.getTime();
+        h = myRTC.hour; m = myRTC.minute; s = myRTC.seconds;
         myRTC.setTime(0, s, m, h, dd, mm, yy); //Manually set RTC
         lastSDFileNameChangeTime = rtcMillis(); // Record the time of the file name change
       }

Firmware/OpenLog_Artemis/nvm.ino

@@ -181,6 +181,7 @@ void recordSystemSettingsToFile()
     settingsFile.println("minimumAwakeTimeMillis=" + (String)settings.minimumAwakeTimeMillis);
     settingsFile.println("identifyBioSensorHubs=" + (String)settings.identifyBioSensorHubs);
     settingsFile.println("serialTxRxDuringSleep=" + (String)settings.serialTxRxDuringSleep);
+    settingsFile.println("printGNSSDebugMessages=" + (String)settings.printGNSSDebugMessages);
     updateDataFileAccess(&settingsFile); // Update the file access time & date
     settingsFile.close();
   }
@@ -460,6 +461,8 @@ bool parseLine(char* str) {
     settings.identifyBioSensorHubs = d;
   else if (strcmp(settingName, "serialTxRxDuringSleep") == 0)
     settings.serialTxRxDuringSleep = d;
+  else if (strcmp(settingName, "printGNSSDebugMessages") == 0)
+    settings.printGNSSDebugMessages = d;
   else
     {
       SerialPrintf2("Unknown setting %s. Ignoring...\r\n", settingName);

Firmware/OpenLog_Artemis/settings.h

@@ -424,6 +424,7 @@ struct struct_settings {
   unsigned long minimumAwakeTimeMillis = 0; // Set to greater than zero to keep the Artemis awake for this long between sleeps
   bool identifyBioSensorHubs = false; // If true, Bio Sensor Hubs (Pulse Oximeters) will be included in autoDetect (requires exclusive use of pins 32 and 11)
   bool serialTxRxDuringSleep = false; // If true, the Serial Tx and Rx pins are left enabled during sleep - to prevent the COM port reinitializing
+  bool printGNSSDebugMessages = false;
 } settings;
 
 //These are the devices on board OpenLog that may be on or offline.