Seems to work reliable

RoSchmi · Apr 5, 2021 · 3f21bf9 · 3f21bf9
1 parent 4185dcd
commit 3f21bf9
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 9 deletions.
diff --git a/include/config.h b/include/config.h
@@ -22,7 +22,7 @@
 // The credentials of your WiFi router and the name and key of your
 // Azure Storage Account have to be set in the file config_secret.h 
 
-#define SENDINTERVAL_MINUTES  2.0            // Sendinterval in minutes, in this interval data are sent to the Cloud                                       
+#define SENDINTERVAL_MINUTES   5.0           // Sendinterval in minutes, in this interval data are sent to the Cloud                                       
                                              // is limited to be not below 1 second
 
 // Names for Tables in Azure Account, please obey rules for Azure Tablenames (e.g. no underscore allowed)
@@ -53,7 +53,7 @@
                                         // (Sensor readings are considered to be invalid if not successsfully
                                         // read within this timespan)
 
-#define NTP_UPDATE_INTERVAL_MINUTES 20   //  With this interval sytem time is updated via NTP
+#define NTP_UPDATE_INTERVAL_MINUTES 14400   //  With this interval sytem time is updated via NTP
                                         //  with internet time (is limited to be not below 1 min)
 
 #define UPDATE_TIME_FROM_AZURE_RESPONSE 1  // 1 = yes, 0 = no. SystemTime is updated from the Post response from Azure.

diff --git a/platformio.ini b/platformio.ini
@@ -12,9 +12,9 @@
 platform = atmelsam
 board = seeed_wio_terminal
 framework = arduino
-upload_protocol = jlink
-upload_port = jlink
-debug_tool = jlink
+;upload_protocol = jlink
+;upload_port = jlink
+;debug_tool = jlink
 
 ;platform_packages = framework-arduino-samd-seeed@https://github.com/Seeed-Studio/ArduinoCore-samd.git
 platform_packages = framework-arduino-samd-seeed@https://github.com/Seeed-Studio/ArduinoCore-samd.git#v1.8.1

diff --git a/src/main.cpp b/src/main.cpp
@@ -594,20 +594,20 @@ void loop()
       SAMCrashMonitor::iAmAlive();
     #endif
 
-    // Update RTC from Ntp when ntpUpdateInterval has expired
-    if (((currentMillis - previousNtpUtdateMillis) >= ntpUpdateInterval) && ((currentMillis - previousNtpRequestMillis) >= 5000)) 
+    // Update RTC from Ntp when ntpUpdateInterval has expired, retry after 20 sec if update was not successful
+    if (((currentMillis - previousNtpUtdateMillis) >= ntpUpdateInterval) && ((currentMillis - previousNtpRequestMillis) >= 20000)) 
     {
-        previousNtpRequestMillis = currentMillis;
         dateTimeUTCNow = sysTime.getTime();
         uint32_t actRtcTime = dateTimeUTCNow.secondstime();
-
+    
         int loopCtr = 0;
         unsigned long  utcNtpTime = getNTPtime();   // Get NTP time, try up to 4 times        
         while ((loopCtr < 4) && utcTime == 0)
         {
           loopCtr++;
           utcTime = getNTPtime();
         }
+        previousNtpRequestMillis = currentMillis;
 
         if (utcNtpTime != 0 )       // if NTP request was successful --> synchronize RTC 
         {