Fix new file create bug.

The newFileName array was too small. The unit would attempt to find a new file name and then record a much larger file name into the array corrupting the RAM. This was regularly called when the unit was awaking between power cycles and seems to be the main culprit behind the lack of logging in issue #2. It doesn't seem to be related to exFat or variability between cards.

Author: nseidle

Firmware/OpenLog_Artemis/logging.ino

@@ -2,7 +2,7 @@
 void msg(const char * message)
 {
   Serial.println(message);
-  if(online.microSD)
+  if (online.microSD)
     sensorDataFile.println(message);
 }
 
@@ -17,15 +17,16 @@ char* findNextAvailableLog(int &newFileNumber, const char *fileLeader)
     newFileNumber--; //Check if last log file was empty
 
   //Search for next available log spot
-  static char newFileName[13];
+  static char newFileName[40];
   while (1)
   {
     sprintf(newFileName, "%s%05u.TXT", fileLeader, newFileNumber); //Splice the new file number into this file name
 
     if (sd.exists(newFileName) == false) break; //File name not found so we will use it.
 
     //File exists so open and see if it is empty. If so, use it.
-    newFile.open(newFileName, O_READ);
+    //newFile = sd.open(newFileName, O_READ);
+    newFile.open(newFileName, O_READ); //exFat
     if (newFile.size() == 0) break; // File is empty so we will use it.
 
     newFile.close(); // Close this existing file we just opened.

Firmware/OpenLog_Artemis/lowerPower.ino

@@ -77,12 +77,12 @@ void goToSleep()
   if (online.dataLogging == true)
   {
     sensorDataFile.sync();
-    //sensorDataFile.close(); //No need to close files. https://forum.arduino.cc/index.php?topic=149504.msg1125098#msg1125098
+    sensorDataFile.close(); //No need to close files. https://forum.arduino.cc/index.php?topic=149504.msg1125098#msg1125098
   }
   if (online.serialLogging == true)
   {
     serialDataFile.sync();
-    //serialDataFile.close();
+    serialDataFile.close();
   }
 
   Serial.flush(); //Finish any prints
@@ -108,14 +108,16 @@ void goToSleep()
   am_hal_pwrctrl_periph_disable(AM_HAL_PWRCTRL_PERIPH_UART0);
   am_hal_pwrctrl_periph_disable(AM_HAL_PWRCTRL_PERIPH_UART1);
 
-
   //Disable all pads
   for (int x = 0 ; x < 50 ; x++)
     am_hal_gpio_pinconfig(x, g_AM_HAL_GPIO_DISABLE);
 
   //We can't leave these power control pins floating
   imuPowerOff();
-  microSDPowerOff();
+  //microSDPowerOff();
+
+  //Testing file record issues
+  microSDPowerOn();
 
   //Keep Qwiic bus powered on if user desires it
   if (settings.powerDownQwiicBusBetweenReads == true)
@@ -191,12 +193,8 @@ void wakeFromSleep()
 
   beginSD(); //285 - 293ms
 
-  //loadSettings(); //50 - 250ms
-
   beginQwiic();
 
-  //analogReadResolution(14); //Increase from default of 10
-
   beginDataLogging(); //180ms
 
   beginSerialLogging(); //20 - 99ms

Firmware/OpenLog_Artemis/nvm.ino

@@ -46,7 +46,8 @@ void recordSettingsToFile()
     if (sd.exists("OLA_settings.cfg"))
       sd.remove("OLA_settings.cfg");
 
-    FsFile settingsFile;
+    //File settingsFile; //FAT16/32
+    FsFile settingsFile; //exFat
     if (settingsFile.open("OLA_settings.cfg", O_CREAT | O_APPEND | O_WRITE) == false)
     {
       Serial.println("Failed to create settings file");
@@ -116,7 +117,8 @@ bool loadSettingsFromFile()
   {
     if (sd.exists("OLA_settings.cfg"))
     {
-      FsFile settingsFile;
+      //File settingsFile; //FAT16/32
+      FsFile settingsFile; //exFat
       if (settingsFile.open("OLA_settings.cfg", O_READ) == false)
       {
         Serial.println("Failed to open settings file");