v1.2.3

Author: PaulZC

Binaries/OpenLog_Artemis-V10-v23_BETA.bin renamed to Binaries/OpenLog_Artemis-V10-v23.bin

@@ -130,6 +130,13 @@
     Change GNSS maximum rate to 25Hz as per:
       https://github.com/sparkfun/OpenLog_Artemis/issues/121
       https://forum.sparkfun.com/viewtopic.php?f=172&t=57512
+    Decided not to include this in the next full release.
+    I suspect it will cause major badness on (e.g.) M8 modules that cannot support 25Hz.
+    
+  v2.3:
+    Resolve https://forum.sparkfun.com/viewtopic.php?f=171&t=58109
+    
+    
 */
 
 const int FIRMWARE_VERSION_MAJOR = 2;
@@ -143,7 +150,7 @@ const int FIRMWARE_VERSION_MINOR = 3;
 //    the variant * 0x100 (OLA = 1; GNSS_LOGGER = 2; GEOPHONE_LOGGER = 3)
 //    the major firmware version * 0x10
 //    the minor firmware version
-#define OLA_IDENTIFIER 0x122 // Stored as 290 decimal in OLA_settings.txt
+#define OLA_IDENTIFIER 0x123 // Stored as 291 decimal in OLA_settings.txt
 
 #include "settings.h"
 
@@ -1031,7 +1038,7 @@ void beginIMU()
       {
         printDebug("beginIMU: second attempt at myICM.begin failed. myICM.status = " + (String)myICM.status + "\r\n");
         digitalWrite(PIN_IMU_CHIP_SELECT, HIGH); //Be sure IMU is deselected
-        SerialPrintln(F("ICM-20948 failed to init."));
+        //SerialPrintln(F("ICM-20948 failed to init."));
         imuPowerOff();
         online.IMU = false;
         return;

Binaries/OpenLog_Artemis-X04-v23.bin

@@ -674,10 +674,10 @@ void configureDevice(node * temp)
           // So the slowest rate we can set with setNavigationFrequency is 1Hz
           // (Whereas UBX_CFG_RATE can actually support intervals as slow as 65535ms)
           sensor->setNavigationFrequency(1); //Set output rate to 1Hz
-        else if (1000000ULL / settings.usBetweenReadings <= 25) //If we are slower than 25Hz logging rate
+        else if (1000000ULL / settings.usBetweenReadings <= 10) //If we are slower than 10Hz logging rate
           sensor->setNavigationFrequency((uint8_t)(1000000ULL / settings.usBetweenReadings)); //Set output rate equal to our query rate
         else
-          sensor->setNavigationFrequency(25); //Set nav freq to 25Hz. Max output depends on the module used.
+          sensor->setNavigationFrequency(10); //Set nav freq to 10Hz. Max output depends on the module used.
 
         setQwiicPullups(settings.qwiicBusPullUps); //Re-enable pullups.
       }

Firmware/OpenLog_Artemis/OpenLog_Artemis.ino

@@ -95,7 +95,8 @@ void powerDownOLA(void)
   powerControlADC(false); // power_adc_disable(); //Power down ADC. It it started by default before setup().
 
   Serial.end(); //Power down UART
-  Serial1.end();
+  if ((settings.useTxRxPinsForTerminal == true) || (online.serialLogging == true))
+    Serial1.end();
 
   //Force the peripherals off
   //This will cause badness with v2.1 of the core but we don't care as we are waiting for a reset
@@ -182,7 +183,8 @@ void resetArtemis(void)
   powerControlADC(false); // power_adc_disable(); //Power down ADC. It it started by default before setup().
 
   Serial.end(); //Power down UART
-  Serial1.end();
+  if ((settings.useTxRxPinsForTerminal == true) || (online.serialLogging == true))
+    Serial1.end();
 
   //Force the peripherals off
   //This will cause badness with v2.1 of the core but we don't care as we are going to force a WDT reset
@@ -282,7 +284,8 @@ void goToSleep(uint32_t sysTicksToSleep)
 
   SerialFlush(); //Finish any prints
   Serial.end();
-  Serial1.end();
+  if ((settings.useTxRxPinsForTerminal == true) || (online.serialLogging == true))
+    Serial1.end();
 
   //Force the peripherals off