stm32 merge (mdns disabled}

EthernetInterface.cpp

@@ -29,8 +29,15 @@
 #include "CommandDistributor.h"
 #include "WiThrottle.h"
 #include "DCCTimer.h"
+#if __has_include ( "MDNS_Generic.h")
+  #include "MDNS_Generic.h"
+ // #define DO_MDNS !!!!!!!!!!!!!  breaks mega
+  EthernetUDP udp;
+  MDNS mdns(udp);
+#endif
 
-extern void looptimer(unsigned long timeout, const FSH* message);
+
+//extern void looptimer(unsigned long timeout, const FSH* message);
 
 bool EthernetInterface::connected=false;
 EthernetServer * EthernetInterface::server= nullptr;
@@ -42,48 +49,73 @@ RingStream * EthernetInterface::outboundRing = nullptr;
  * @brief Setup Ethernet Connection
  * 
  */
-void EthernetInterface::setup()
-{
-    connected=false;
 
+void EthernetInterface::setup()  // STM32 VERSION
+{
+  DIAG(F("Ethernet begin"));
+  
+  #ifdef STM32_ETHERNET
+    // Set a HOSTNAME for the DHCP request - a nice to have, but hard it seems on LWIP for STM32
+    // The default is "lwip", which is **always** set in STM32Ethernet/src/utility/ethernetif.cpp
+    // for some reason. One can edit it to instead read:
+    //      #if LWIP_NETIF_HOSTNAME
+    //      /* Initialize interface hostname */
+    //      if (netif->hostname == NULL)
+    //         netif->hostname = "lwip";
+    //      #endif /* LWIP_NETIF_HOSTNAME */
+    // Which seems more useful! We should propose the patch... so the following line actually works!
+    netif_set_hostname(&gnetif, WIFI_HOSTNAME);   // Should probably be passed in the contructor...
+    #define _MAC_ MacAddressDefault()
+  #else 
     byte mac[6];
     DCCTimer::getSimulatedMacAddress(mac);
-    DIAG(F("Ethernet begin"));
+    #define _MAC_ mac
+  #endif   
   
-#ifdef IP_ADDRESS
+  #ifdef IP_ADDRESS
     static IPAddress myIP(IP_ADDRESS);
-    Ethernet.begin(mac, myIP);
+    Ethernet.begin(_MAC_,myIP);
-#else
+    setup(false);
-    if (Ethernet.begin(mac) == 0)
+  #else
-    {
+    if (Ethernet.begin(_MAC_)==0)
-      LCD(4,F("IP: No DHCP"));
+  {
-      return;
+    LCD(4,F("IP: No DHCP"));
-    }
+    return;
-#endif
+  }
-    auto ip = Ethernet.localIP();    // look what IP was obtained (dynamic or static)
+  #endif
-    if (!ip) {
-      LCD(4,F("IP: None"));
-      return;
-    }
-    server = new EthernetServer(IP_PORT); // Ethernet Server listening on default port IP_PORT
-    server->begin();
-    #ifdef LCD_DRIVER
-      const byte lcdData[]={LCD_DRIVER};
-      const bool wideDisplay=lcdData[1]>=24; // data[1] is cols. 
-    #else 
-      const bool wideDisplay=true;
-    #endif    
-    if (wideDisplay) {
-      // OLEDS or just usb diag is ok on one line. 
-      LCD(4,F("IP %d.%d.%d.%d:%d"), ip[0], ip[1], ip[2], ip[3], IP_PORT);    
-    } 
-    else { // LCDs generally too narrow, so take 2 lines
-      LCD(4,F("IP %d.%d.%d.%d"), ip[0], ip[1], ip[2], ip[3]);
-      LCD(5,F("Port %d"), IP_PORT);
-    }
 
-    outboundRing=new RingStream(OUTBOUND_RING_SIZE);
+  auto ip = Ethernet.localIP();    // look what IP was obtained (dynamic or static)
-    connected=true;
+  if (!ip) {
+    LCD(4,F("IP: None"));
+    return;
+  }
+  server = new EthernetServer(IP_PORT); // Ethernet Server listening on default port IP_PORT
+  server->begin();
+
+  // Arrange display of IP address and port
+  #ifdef LCD_DRIVER
+    const byte lcdData[]={LCD_DRIVER};
+    const bool wideDisplay=lcdData[1]>=24; // data[1] is cols. 
+  #else 
+    const bool wideDisplay=true;
+  #endif    
+  if (wideDisplay) {
+    // OLEDS or just usb diag is ok on one line. 
+    LCD(4,F("IP %d.%d.%d.%d:%d"), ip[0], ip[1], ip[2], ip[3], IP_PORT);    
+  } 
+  else { // LCDs generally too narrow, so take 2 lines
+    LCD(4,F("IP %d.%d.%d.%d"), ip[0], ip[1], ip[2], ip[3]);
+    LCD(5,F("Port %d"), IP_PORT);
+  }
+ 
+  outboundRing=new RingStream(OUTBOUND_RING_SIZE);
+  #ifdef DO_MDNS
+    mdns.begin(Ethernet.localIP(), WIFI_HOSTNAME); // hostname
+    mdns.addServiceRecord(WIFI_HOSTNAME "._withrottle", IP_PORT, MDNSServiceTCP);
+  // Not sure if we need to run it once, but just in case!
+    mdns.run();
+  #endif  
+  connected=true;    
 }
 
 
@@ -109,6 +141,11 @@ void EthernetInterface::loop()
       warnedAboutLink=false;
     } 
     
+  #ifdef DO_MDNS
+    // Always do this because we don't want traffic to intefere with being found!
+    mdns.run();
+  #endif
+
     //
     switch (Ethernet.maintain()) {
     case 1:
@@ -127,28 +164,45 @@ void EthernetInterface::loop()
         break;
     }
     
-    // get client from the server
+      // get client from the server
-    auto client = server->accept();
+  #if defined (STM32_ETHERNET)
-
+    // STM32Ethernet doesn't use accept(), just available()
-    // check for new client
+    auto client = server->available();
-    if (client)
+    if (client) {
-    {
+      // check for new client
-        if (Diag::ETHERNET) DIAG(F("Ethernet: New client "));
+      byte socket;
-        byte socket;
+      bool sockfound = false;
+      for (socket = 0; socket < MAX_SOCK_NUM; socket++)
+      {
+        if (client == clients[socket])
+        {
+          sockfound = true;
+          break;
+        }
+      }
+      if (!sockfound)
+      { // new client
         for (socket = 0; socket < MAX_SOCK_NUM; socket++)
         {
-            if (!clients[socket])
+          if (!clients[socket])
-            {
+          {
-                // On accept() the EthernetServer doesn't track the client anymore
+            clients[socket] = client;
-                // so we store it in our client array
+            sockFound=true;
-                if (Diag::ETHERNET) DIAG(F("Socket %d"),socket);
+            if (Diag::ETHERNET)
-                clients[socket] = client;
+              DIAG(F("Ethernet: New client socket %d"), socket);
-                break;
+            break;
-            }
+          }
         }
-        if (socket==MAX_SOCK_NUM) DIAG(F("new Ethernet OVERFLOW")); 
+      }
+      if (!sockFound) DIAG(F("new Ethernet OVERFLOW"));
     }
   
+  #else
+    auto client = server->accept();
+    if (client) clients[client.getSocketNumber()]=client;
+  #endif
+    
+
     // check for incoming data from all possible clients
     for (byte socket = 0; socket < MAX_SOCK_NUM; socket++)
     {

EthernetInterface.h

@@ -45,7 +45,7 @@
  * 
  */
 
-#define MAX_ETH_BUFFER 512
+#define MAX_ETH_BUFFER 128
 #define OUTBOUND_RING_SIZE 2048
 
 class EthernetInterface {

platformio.ini

@@ -104,6 +104,14 @@ lib_deps =
 	${env.lib_deps}
 	arduino-libraries/Ethernet
 	SPI
+	MDNS_Generic
+
+lib_ignore = WiFi101
+			WiFi101_Generic
+			WiFiEspAT
+			WiFiMulti_Generic
+			WiFiNINA_Generic
+
 monitor_speed = 115200
 monitor_echo = yes
 build_flags = 
@@ -246,6 +254,24 @@ monitor_echo = yes
 
 ; Experimental - Ethernet work still in progress
 ;
+[env:Nucleo-F429ZI]
+platform = ststm32
+board = nucleo_f429zi
+framework = arduino
+lib_deps = ${env.lib_deps}
+			stm32duino/STM32Ethernet @ ^1.3.0
+			stm32duino/STM32duino LwIP @ ^2.1.2
+			MDNS_Generic
+lib_ignore = WiFi101
+			WiFi101_Generic
+			WiFiEspAT
+			WiFiMulti_Generic
+			WiFiNINA_Generic
+build_flags = -std=c++17 -Os -g2 -Wunused-variable
+monitor_speed = 115200
+monitor_echo = yes
+upload_protocol = stlink
+
 ; [env:Nucleo-F429ZI]
 ; platform = ststm32
 ; board = nucleo_f429zi