Logging: unique semaphore wait, reduce loop delay, add taskYIELD

Author: nseidle

Firmware/RTK_Surveyor/RTK_Surveyor.ino

@@ -125,7 +125,8 @@ int startCurrentLogTime_minutes = 0; //Mark when we start this specific log file
 //System crashes if two tasks access a file at the same time
 //So we use a semaphore to see if file system is available
 SemaphoreHandle_t sdCardSemaphore;
-TickType_t fatSemaphore_shortWait_ms = 50 / portTICK_PERIOD_MS;
+TickType_t loggingSemaphore_shortWait_ms = 10 / portTICK_PERIOD_MS;
+const TickType_t fatSemaphore_shortWait_ms = 10 / portTICK_PERIOD_MS;
 const TickType_t fatSemaphore_longWait_ms = 200 / portTICK_PERIOD_MS;
 
 //Display used/free space in menu and config page

Firmware/RTK_Surveyor/Tasks.ino

@@ -203,7 +203,7 @@ void F9PSerialReadTask(void *e)
         {
           //Attempt to gain access to the SD card, avoids collisions with file
           //writing from other functions like recordSystemSettingsToFile()
-          if (xSemaphoreTake(sdCardSemaphore, fatSemaphore_shortWait_ms) == pdPASS)
+          if (xSemaphoreTake(sdCardSemaphore, loggingSemaphore_shortWait_ms) == pdPASS)
           {
             //Reduce bytes to send if we have more to send then the end of the buffer
             //We'll wrap next loop
@@ -240,7 +240,8 @@ void F9PSerialReadTask(void *e)
     //Let other tasks run, prevent watch dog timer (WDT) resets
     //----------------------------------------------------------------------
 
-    delay(10);
+    delay(1);
+    taskYIELD();
   }
 }
 

Firmware/RTK_Surveyor/menuMessages.ino

@@ -819,7 +819,7 @@ void updateLogTest()
   {
     setLogTestFrequencyMessages(rate, messages); //Set messages and rate for both UART1 and USB ports
 
-    fatSemaphore_shortWait_ms = semaphoreWait / portTICK_PERIOD_MS; //Update variable
+    loggingSemaphore_shortWait_ms = semaphoreWait / portTICK_PERIOD_MS; //Update variable
 
     startCurrentLogTime_minutes = millis() / 1000L / 60; //Mark now as start of logging