Update Ethernet.linkStatus to handle Unknown status

Signed-off-by: Frederic Pillon <[email protected]>

Author: fpistm

src/STM32Ethernet.cpp

@@ -103,7 +103,7 @@ void EthernetClass::begin(uint8_t *mac, IPAddress local_ip, IPAddress dns_server
 
 EthernetLinkStatus EthernetClass::linkStatus()
 {
-  return (stm32_eth_link_up() ? LinkON : LinkOFF);
+  return (!stm32_eth_is_init()) ? Unknown : (stm32_eth_link_up() ? LinkON : LinkOFF);
 }
 
 int EthernetClass::maintain(){

src/utility/ethernetif.c

@@ -431,6 +431,17 @@ void ethernetif_input(struct netif *netif)
   }
 }
 
+/**
+  * @brief Returns the current state
+  *
+  * @param None
+  * @return 0 not initialized else 1
+  */
+uint8_t ethernetif_is_init(void)
+{
+  return (EthHandle.State != HAL_ETH_STATE_RESET);
+}
+
 /**
   * @brief Should be called at the beginning of the program to set up the
   * network interface. It calls the function low_level_init() to do the

src/utility/ethernetif.h

@@ -56,6 +56,7 @@
 #endif
 
 /* Exported types ------------------------------------------------------------*/
+uint8_t ethernetif_is_init(void);
 err_t ethernetif_init(struct netif *netif);
 void ethernetif_input(struct netif *netif);
 void ethernetif_set_link(struct netif *netif);

src/utility/stm32_eth.cpp

@@ -246,6 +246,15 @@ void stm32_eth_init(const uint8_t *mac, const uint8_t *ip, const uint8_t *gw, co
   stm32_eth_scheduler();
 }
 
+/**
+  * @brief Return Ethernet init status
+  * @param  None
+  * @retval 1 for initialized, 0 for not initialized
+  */
+uint8_t stm32_eth_is_init(void) {
+  return ethernetif_is_init();
+}
+
 /**
   * @brief Return Ethernet link status
   * @param  None

src/utility/stm32_eth.h

@@ -121,6 +121,7 @@ extern struct netif gnetif;
 
 /* Exported functions ------------------------------------------------------- */
 void stm32_eth_init(const uint8_t *mac, const uint8_t *ip, const uint8_t *gw, const uint8_t *netmask);
+uint8_t stm32_eth_is_init(void);
 uint8_t stm32_eth_link_up(void);
 void stm32_eth_scheduler(void);