From 8f2478d84f37f7651e61d93bb0d21420528d3e75 Mon Sep 17 00:00:00 2001
From: FOG_Yamato <contact@cybersnets.com>
Date: Wed, 5 Apr 2023 23:38:50 +0300
Subject: [PATCH 1/5] Added clock update capability via NTP servers

---
 html/scan.js         |  4 ++++
 html/vrx_index.html  | 24 ++++++++++++++++++++++++
 lib/MSP/msptypes.h   |  1 +
 lib/WIFI/devWIFI.cpp | 18 ++++++++++++++++++
 src/Vrx_main.cpp     |  6 ++++++
 src/hdzero.cpp       | 24 ++++++++++++++++++++++++
 src/hdzero.h         |  1 +
 src/module_base.cpp  |  5 +++++
 src/module_base.h    |  1 +
 9 files changed, 84 insertions(+)

diff --git a/html/scan.js b/html/scan.js
index 7868c0a..51e589e 100644
--- a/html/scan.js
+++ b/html/scan.js
@@ -13,6 +13,7 @@ function get_mode() {
             var data = JSON.parse(this.responseText);
             if (data.mode==="STA") {
                 _('stamode').style.display = 'block';
+                _('rtctab').style.display = 'block';
                 _('ssid').textContent = data.ssid;
             } else {
                 _('apmode').style.display = 'block';
@@ -347,6 +348,9 @@ _('forget').addEventListener('click', callback("Forget Home Network", "An error
 if (_('modelmatch') != undefined) {
     _('modelmatch').addEventListener('submit', callback("Set Model Match", "An error occurred updating the model match number", "/model", null));
 }
+_('setrtc').addEventListener('submit', callback("Set RTC Time", "An error occured setting the RTC time", "/setrtc", function() {
+    return new FormData(_('setrtc'));
+}));
 
 //=========================================================
 
diff --git a/html/vrx_index.html b/html/vrx_index.html
index 74b1b52..12c42d4 100644
--- a/html/vrx_index.html
+++ b/html/vrx_index.html
@@ -27,6 +27,7 @@ <h1>Welcome to your <b>ExpressLRS</b><br /> update page<br />
 			<ul class="mui-tabs__bar mui-tabs__bar--justified">
 				<li class="mui--is-active"><a data-mui-toggle="tab" data-mui-controls="pane-justified-1">Update</a></li>
 				<li><a data-mui-toggle="tab" data-mui-controls="pane-justified-2">Network</a></li>
+				<li id="rtctab" style="display:none;"><a data-mui-toggle="tab" data-mui-controls="pane-justified-3">RTC Time</a></li>
 			</ul>
 
 			<div class="mui-tabs__pane mui--is-active" id="pane-justified-1">
@@ -81,6 +82,29 @@ <h2>Home Network: <span id="ssid"></span></h2>
 					<a id="access" href="#" class="mui-btn mui-btn--primary">Disconnect</a>
 				</div>
 			</div>
+
+			<div class="mui-tabs__pane" id="pane-justified-3">
+				<div class="mui-panel">
+					<h2>RTC Update via NTP</h2>
+					Here you can update your goggle RTC clock time using any available NTP server.
+					Make sure that the home network you're connected to has access to the Internet in order to connect to the NTP server.
+					<form action="/setrtc" id="setrtc" method="POST" class="mui-form">
+						<div class="mui-textfield" style="width: 50%;">
+							<input id="server" type="text" name="server" placeholder="NTP Server"/>
+						</div>
+						<div class="mui-textfield" style="width: 50%;">
+							<input id="offset" type="number" name="offset" min="-12" max="12" placeholder="UTC Offset"/>
+						</div>
+						<div class="mui-checkbox">
+							<label>
+								<input id="dst" type="checkbox" name="dst">
+								Daylight Saving
+							</label>
+						</div>
+						<input type="submit" value="Set Time" class="mui-btn mui-btn--primary">
+					</form>
+				</div>
+			</div>
 		</div>
 	</div>
 	<footer>
diff --git a/lib/MSP/msptypes.h b/lib/MSP/msptypes.h
index a7cae37..c80986c 100644
--- a/lib/MSP/msptypes.h
+++ b/lib/MSP/msptypes.h
@@ -43,6 +43,7 @@
 #define MSP_ELRS_BACKPACK_SET_BUZZER            0x030B
 #define MSP_ELRS_BACKPACK_SET_OSD_ELEMENT       0x030C
 #define MSP_ELRS_BACKPACK_SET_HEAD_TRACKING     0x030D  // enable/disable head-tracking forwarding packets to the TX
+#define MSP_ELRS_BACKPACK_SET_RTC               0x030E
 
 // incoming, packets originating from the VRx
 #define MSP_ELRS_BACKPACK_SET_MODE              0x0380  // enable wifi/binding mode
diff --git a/lib/WIFI/devWIFI.cpp b/lib/WIFI/devWIFI.cpp
index 8d0ee63..332474a 100644
--- a/lib/WIFI/devWIFI.cpp
+++ b/lib/WIFI/devWIFI.cpp
@@ -25,12 +25,14 @@
 #include "helpers.h"
 
 #include "UpdateWrapper.h"
+#include "time.h"
 
 #include "WebContent.h"
 
 #include "config.h"
 #if defined(TARGET_VRX_BACKPACK)
 extern VrxBackpackConfig config;
+extern bool sendRTCChangesToVrx;
 #else
 extern TxBackpackConfig config;
 #endif
@@ -422,6 +424,21 @@ static void WebUploadForceUpdateHandler(AsyncWebServerRequest *request) {
   }
 }
 
+static void WebUploadRTCUpdateHandler(AsyncWebServerRequest *request) {
+  String ntpServer = request->arg("server");
+  long offset = request->arg("offset").toInt();
+  long dst = request->arg("dst") == "on" ? 3600 : 0;
+  long utcOffset = offset < 0 ? (12 + abs(offset)) * 3600 : offset * 3600;
+
+  DBGLN("Getting NTP data from %s", ntpServer.c_str());
+  configTime(dst, utcOffset, ntpServer.c_str());
+  sendRTCChangesToVrx = true;
+  AsyncWebServerResponse *response = request->beginResponse(200, "text/plain", "RTC clock synced with NTP server.");
+  response->addHeader("Connection", "close");
+  request->send(response);
+  request->client()->close();
+}
+
 static void wifiOff()
 {
   wifiStarted = false;
@@ -554,6 +571,7 @@ static void startServices()
 
   server.on("/update", HTTP_POST, WebUploadResponseHandler, WebUploadDataHandler);
   server.on("/forceupdate", WebUploadForceUpdateHandler);
+  server.on("/setrtc", WebUploadRTCUpdateHandler);
 
   server.on("/log.js", WebUpdateSendContent);
   server.on("/log.html", WebUpdateSendContent);
diff --git a/src/Vrx_main.cpp b/src/Vrx_main.cpp
index 2f28368..d9bc1e9 100644
--- a/src/Vrx_main.cpp
+++ b/src/Vrx_main.cpp
@@ -69,6 +69,7 @@ unsigned long rebootTime = 0;
 uint8_t cachedIndex = 0;
 bool sendChannelChangesToVrx = false;
 bool sendHeadTrackingChangesToVrx = false;
+bool sendRTCChangesToVrx = false;
 bool gotInitialPacket = false;
 bool headTrackingEnabled = false;
 uint32_t lastSentRequest = 0;
@@ -445,6 +446,11 @@ void loop()
 
   if (connectionState == wifiUpdate)
   {
+    if (sendRTCChangesToVrx)
+    {
+      sendRTCChangesToVrx = false;
+      vrxModule.SetRTC();
+    }
     return;
   }
 
diff --git a/src/hdzero.cpp b/src/hdzero.cpp
index 686f224..d290fd9 100644
--- a/src/hdzero.cpp
+++ b/src/hdzero.cpp
@@ -2,6 +2,7 @@
 #include "hdzero.h"
 #include "msptypes.h"
 #include "logging.h"
+#include "time.h"
 
 void
 HDZero::Init()
@@ -108,3 +109,26 @@ HDZero::SendHeadTrackingEnableCmd(bool enable)
 
     msp.sendPacket(&packet, m_port);
 }
+
+void
+HDZero::SetRTC()
+{
+    MSP msp;
+    mspPacket_t packet;
+    tm timeData;
+    if(!getLocalTime(&timeData)) {
+        DBGLN("Could not obtain time data.");
+        return;
+    }
+    packet.reset();
+    packet.makeCommand();
+    packet.function = MSP_ELRS_BACKPACK_SET_RTC;
+    packet.addByte(timeData.tm_year);
+    packet.addByte(timeData.tm_mon);
+    packet.addByte(timeData.tm_mday);
+    packet.addByte(timeData.tm_hour);
+    packet.addByte(timeData.tm_min);
+    packet.addByte(timeData.tm_sec);
+    
+    msp.sendPacket(&packet, m_port);
+}
\ No newline at end of file
diff --git a/src/hdzero.h b/src/hdzero.h
index 04573c6..d898260 100644
--- a/src/hdzero.h
+++ b/src/hdzero.h
@@ -24,4 +24,5 @@ class HDZero : public MSPModuleBase
     uint8_t GetRecordingState();
     void SetRecordingState(uint8_t recordingState, uint16_t delay);
     void SendHeadTrackingEnableCmd(bool enable);
+    void SetRTC();
 };
diff --git a/src/module_base.cpp b/src/module_base.cpp
index 20f80b0..912d272 100644
--- a/src/module_base.cpp
+++ b/src/module_base.cpp
@@ -37,6 +37,11 @@ ModuleBase::SendHeadTrackingEnableCmd(bool enable)
 {
 }
 
+void
+ModuleBase::SetRTC()
+{
+}
+
 void
 ModuleBase::Loop(uint32_t now)
 {
diff --git a/src/module_base.h b/src/module_base.h
index 1694602..c1d6033 100644
--- a/src/module_base.h
+++ b/src/module_base.h
@@ -11,6 +11,7 @@ class ModuleBase
     void SetRecordingState(uint8_t recordingState, uint16_t delay);
     void SetOSD(mspPacket_t *packet);
     void SendHeadTrackingEnableCmd(bool enable);
+    void SetRTC();
     void Loop(uint32_t now);
 };
 

From f781fd39eb06b614f632cc9b4650439d377bcf35 Mon Sep 17 00:00:00 2001
From: FOG_Yamato <contact@cybersnets.com>
Date: Thu, 6 Apr 2023 00:21:48 +0300
Subject: [PATCH 2/5] Added guard for TX backpacks

---
 lib/WIFI/devWIFI.cpp | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/lib/WIFI/devWIFI.cpp b/lib/WIFI/devWIFI.cpp
index 332474a..93ee11b 100644
--- a/lib/WIFI/devWIFI.cpp
+++ b/lib/WIFI/devWIFI.cpp
@@ -432,7 +432,9 @@ static void WebUploadRTCUpdateHandler(AsyncWebServerRequest *request) {
 
   DBGLN("Getting NTP data from %s", ntpServer.c_str());
   configTime(dst, utcOffset, ntpServer.c_str());
+  #if defined(TARGET_VRX_BACKPACK)
   sendRTCChangesToVrx = true;
+  #endif
   AsyncWebServerResponse *response = request->beginResponse(200, "text/plain", "RTC clock synced with NTP server.");
   response->addHeader("Connection", "close");
   request->send(response);

From f3d48ba4b1615b561de4a2de509c1a3cd546f667 Mon Sep 17 00:00:00 2001
From: FOG_Yamato <contact@cybersnets.com>
Date: Thu, 6 Apr 2023 01:01:25 +0300
Subject: [PATCH 3/5] Upgrade PIO platform version

---
 targets/common.ini | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/targets/common.ini b/targets/common.ini
index ae38fe1..cc8e974 100644
--- a/targets/common.ini
+++ b/targets/common.ini
@@ -1,6 +1,6 @@
 # ------------------------- COMMON ENV DEFINITIONS -----------------
 [env]
-platform = espressif8266@3.2.0
+platform = espressif8266@4.2.0
 framework = arduino
 extra_scripts =
 	pre:python/build_flags.py

From 8fe6c7567d8c4c25056fc27199fa28f2f264ae09 Mon Sep 17 00:00:00 2001
From: Paul Kendall <pkendall64@gmail.com>
Date: Sat, 15 Apr 2023 14:29:52 +1200
Subject: [PATCH 4/5] Added defaults for pool and DST

- Fix NTP lookup as the server string is kept in the NTP code so it has be static
- Try to get time in web server so we can send an error response
---
 html/vrx_index.html  |  4 ++--
 lib/WIFI/devWIFI.cpp | 20 +++++++++++++++-----
 2 files changed, 17 insertions(+), 7 deletions(-)

diff --git a/html/vrx_index.html b/html/vrx_index.html
index 12c42d4..2e35d51 100644
--- a/html/vrx_index.html
+++ b/html/vrx_index.html
@@ -90,14 +90,14 @@ <h2>RTC Update via NTP</h2>
 					Make sure that the home network you're connected to has access to the Internet in order to connect to the NTP server.
 					<form action="/setrtc" id="setrtc" method="POST" class="mui-form">
 						<div class="mui-textfield" style="width: 50%;">
-							<input id="server" type="text" name="server" placeholder="NTP Server"/>
+							<input id="server" type="text" name="server" value="pool.ntp.org"/>
 						</div>
 						<div class="mui-textfield" style="width: 50%;">
 							<input id="offset" type="number" name="offset" min="-12" max="12" placeholder="UTC Offset"/>
 						</div>
 						<div class="mui-checkbox">
 							<label>
-								<input id="dst" type="checkbox" name="dst">
+								<input id="dst" type="checkbox" name="dst" checked>
 								Daylight Saving
 							</label>
 						</div>
diff --git a/lib/WIFI/devWIFI.cpp b/lib/WIFI/devWIFI.cpp
index 93ee11b..7f9d4a2 100644
--- a/lib/WIFI/devWIFI.cpp
+++ b/lib/WIFI/devWIFI.cpp
@@ -425,20 +425,30 @@ static void WebUploadForceUpdateHandler(AsyncWebServerRequest *request) {
 }
 
 static void WebUploadRTCUpdateHandler(AsyncWebServerRequest *request) {
-  String ntpServer = request->arg("server");
+  static String ntpServer = request->arg("server");
   long offset = request->arg("offset").toInt();
   long dst = request->arg("dst") == "on" ? 3600 : 0;
   long utcOffset = offset < 0 ? (12 + abs(offset)) * 3600 : offset * 3600;
 
   DBGLN("Getting NTP data from %s", ntpServer.c_str());
   configTime(dst, utcOffset, ntpServer.c_str());
-  #if defined(TARGET_VRX_BACKPACK)
-  sendRTCChangesToVrx = true;
-  #endif
-  AsyncWebServerResponse *response = request->beginResponse(200, "text/plain", "RTC clock synced with NTP server.");
+
+  tm timeData;
+  AsyncWebServerResponse *response;
+  if(!getLocalTime(&timeData)) {
+    response = request->beginResponse(500);
+  }
+  else {
+    response = request->beginResponse(200, "text/plain", "RTC clock synced with NTP server.");
+  }
+
   response->addHeader("Connection", "close");
   request->send(response);
   request->client()->close();
+
+  #if defined(TARGET_VRX_BACKPACK)
+  sendRTCChangesToVrx = true;
+  #endif
 }
 
 static void wifiOff()

From 4cdad550c7d8e8c7d15bd5635096a788220d850c Mon Sep 17 00:00:00 2001
From: Paul Kendall <pkendall64@gmail.com>
Date: Wed, 10 May 2023 16:59:37 +1200
Subject: [PATCH 5/5] Remove `.` from target name

because, you know, PIO!
---
 targets/hdzero.ini | 18 +++++++++---------
 1 file changed, 9 insertions(+), 9 deletions(-)

diff --git a/targets/hdzero.ini b/targets/hdzero.ini
index bb03c3c..fbd9c49 100644
--- a/targets/hdzero.ini
+++ b/targets/hdzero.ini
@@ -2,7 +2,7 @@
 # VRX backpack targets
 # ********************************
 
-[env:HDZero_RX5.1_ESP8285_Backpack_via_UART]
+[env:HDZero_RX51_ESP8285_Backpack_via_UART]
 extends = env_common_esp8285, hdzero_vrx_backpack_common
 monitor_speed = 115200
 build_flags =
@@ -10,10 +10,10 @@ build_flags =
 	${hdzero_vrx_backpack_common.build_flags}
 	-D PIN_LED=16
 
-[env:HDZero_RX5.1_ESP8285_Backpack_via_WIFI]
-extends = env:HDZero_RX5.1_ESP8285_Backpack_via_UART
+[env:HDZero_RX51_ESP8285_Backpack_via_WIFI]
+extends = env:HDZero_RX51_ESP8285_Backpack_via_UART
 
-[env:HDZero_RX5.1_ESP32_Backpack_via_UART]
+[env:HDZero_RX51_ESP32_Backpack_via_UART]
 extends = env_common_esp32, hdzero_vrx_backpack_common
 monitor_speed = 115200
 build_flags =
@@ -22,10 +22,10 @@ build_flags =
 	-D PIN_LED=4
 lib_ignore = STM32UPDATE
 
-[env:HDZero_RX5.1_ESP32_Backpack_via_WIFI]
-extends = env:HDZero_RX5.1_ESP32_Backpack_via_UART
+[env:HDZero_RX51_ESP32_Backpack_via_WIFI]
+extends = env:HDZero_RX51_ESP32_Backpack_via_UART
 
-[env:HDZero_Goggle_RX5.1_ESP32_Backpack_via_UART]
+[env:HDZero_Goggle_ESP32_Backpack_via_UART]
 extends = env_common_esp32, hdzero_vrx_backpack_common
 monitor_speed = 115200
 build_flags =
@@ -35,5 +35,5 @@ build_flags =
 	-D NO_AUTOBIND=1
 lib_ignore = STM32UPDATE
 
-[env:HDZero_Goggle_RX5.1_ESP32_Backpack_via_WIFI]
-extends = env:HDZero_Goggle_RX5.1_ESP32_Backpack_via_UART
+[env:HDZero_Goggle_ESP32_Backpack_via_WIFI]
+extends = env:HDZero_Goggle_ESP32_Backpack_via_UART