Merge 7294535321 into 3b162996ad

CommandStation-EX.ino

@@ -52,9 +52,9 @@
 #include "DCCEX.h"
 #include "Display_Implementation.h"
 
-#ifdef ARDUINO_ARCH_ESP32
+#ifdef OTA_ENABLED
   #include <ArduinoOTA.h>
-#endif // ARDUINO_ARCH_ESP32
+#endif // OTA_ENABLED
 
 #ifdef CPU_TYPE_ERROR
 #error CANNOT COMPILE - DCC++ EX ONLY WORKS WITH THE ARCHITECTURES LISTED IN defines.h
@@ -105,9 +105,16 @@ void setup()
 #else
   // ESP32 needs wifi on always
   WifiESP::setup(WIFI_SSID, WIFI_PASSWORD, WIFI_HOSTNAME, IP_PORT, WIFI_CHANNEL, WIFI_FORCE_AP);
-  #if OTA_AUTO_INIT
+
-    Diag::OTA = true;
+  // Start OTA if enabled
-  #endif // OTA_AUTO_INIT
+  #ifdef OTA_ENABLED
+    ArduinoOTA.setHostname(WIFI_HOSTNAME);
+    #ifdef OTA_AUTH
+      ArduinoOTA.setPassword(OTA_AUTH);
+    #endif // OTA_AUTH
+    ArduinoOTA.begin();
+  #endif // OTA_ENABLED
+
 #endif // ARDUINO_ARCH_ESP32
 
 #if ETHERNET_ON
@@ -157,24 +164,9 @@ void loop()
 #ifndef WIFI_TASK_ON_CORE0
   WifiESP::loop();
 #endif
-  // Responsibility 4: Optionally handle Arduino OTA updates
+  #ifdef OTA_ENABLED
-  if (Diag::OTA) {
-    static bool otaInitialised = false;
-    // Initialise OTA if not already done
-    if (!otaInitialised) {
-      ArduinoOTA.setHostname(WIFI_HOSTNAME);
-      // Set OTA password if defined
-      #ifdef OTA_AUTH
-        ArduinoOTA.setPassword(OTA_AUTH);
-      #endif // OTA_AUTH
-      ArduinoOTA.begin();
-      otaInitialised = true;
-    }
-    // Handle OTA if initialised
-    else {
     ArduinoOTA.handle();
-    }
+  #endif // OTA_ENABLED
-  }
 #endif //ARDUINO_ARCH_ESP32
 #if ETHERNET_ON
   EthernetInterface::loop();

DCCEXParser.cpp

@@ -168,7 +168,6 @@ const int16_t HASH_KEYWORD_ANOUT = -26399;
 const int16_t HASH_KEYWORD_WIFI = -5583;
 const int16_t HASH_KEYWORD_ETHERNET = -30767;
 const int16_t HASH_KEYWORD_WIT = 31594;
-const int16_t HASH_KEYWORD_OTA = 22938;
 
 int16_t DCCEXParser::stashP[MAX_COMMAND_PARAMS];
 bool DCCEXParser::stashBusy;
@@ -1085,11 +1084,6 @@ bool DCCEXParser::parseD(Print *stream, int16_t params, int16_t p[])
         IODevice::writeAnalogue(p[1], p[2], params>3 ? p[3] : 0);
         break;
 
-    case HASH_KEYWORD_OTA:    // <D OTA ON/OFF>
-        Diag::OTA = onOff;
-        DIAG(F("OTA=%S"), onOff ? F("ON") : F("OFF"));
-        return true;
-
     default: // invalid/unknown
         break;
     }

StringFormatter.cpp

@@ -26,7 +26,6 @@ bool Diag::WIFI=false;
 bool Diag::WITHROTTLE=false;
 bool Diag::ETHERNET=false;
 bool Diag::LCN=false;
-bool Diag::OTA=false;
 
  
 void StringFormatter::diag( const FSH* input...) {

StringFormatter.h

@@ -30,7 +30,7 @@ class Diag {
   static bool WITHROTTLE;
   static bool ETHERNET;
   static bool LCN;
-  static bool OTA;
+  
 };
 
 class StringFormatter

config.example.h

@@ -133,19 +133,15 @@ The configuration file for DCC-EX Command Station
 // with that SSID.
 #define WIFI_FORCE_AP false
 //
-// OTA_AUTO_INIT: Set this to true if you want OTA updates to be initialized
+// OTA_ENABLED: If you'd like to enable OTA updates, set this to true. Otherwise,
-// automatically upon startup. If set to false, OTA updates will remain
+// OTA updates will not be available.
-// unavailable until the "<C OTA 1>" command is executed.
+#define OTA_ENABLED true
-// Please note that this feature requires the use of ARDUINO_ARCH_ESP32 as your board.
-#define OTA_AUTO_INIT false
 //
-// OTA_AUTH: Set this to your desired password if you wish to secure OTA updates.
+// OTA_AUTH: If you'd like to change the OTA password, set this to the password
-// If not set, OTA updates will be password-free.
+// you'd like to use. Otherwise, the default password will be "dccex-ota".
-// Note: Upon modifying the OTA password, ensure to update the "upload_flags → --auth"
+// Note: After changing the OTA password, you must update the "upload_flags → --auth"
-// in the relevant environment within the platformio.ini file.
+// in the corresponding environment within the platformio.ini file.
-// To deactivate OTA authorization, comment out the line below and comment out
+#define OTA_AUTH "dccex-ota"
-// the "upload_flags" line in the platformio.ini file.
-// #define OTA_AUTH "dccex-ota"
 
 
 /////////////////////////////////////////////////////////////////////////////////////