Protect W5500/ENC28J60 isLinked from interrupt

Fixes #1786
earlephilhower · Nov 10, 2023 · 003d91e · 003d91e
1 parent 8ac616e
commit 003d91e
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 3 deletions.
diff --git a/libraries/lwIP_enc28j60/src/utility/enc28j60.cpp b/libraries/lwIP_enc28j60/src/utility/enc28j60.cpp
@@ -39,6 +39,7 @@
 #include <stdio.h>
 #include <stdarg.h>
 #include <string.h>
+#include <LwipEthernet.h>
 
 #include "enc28j60.h"
 
@@ -696,6 +697,7 @@ uint16_t ENC28J60::phyread(uint8_t reg) {
 
 bool ENC28J60::isLinked() {
     // ( https://github.com/JAndrassy/EthernetENC/tree/master/src/utility/enc28j60.h )
-
-    return !!(phyread(MACSTAT2) & 0x400);
+    ethernet_arch_lwip_begin();
+    auto ret = !!(phyread(MACSTAT2) & 0x400);
+    ethernet_arch_lwip_end();
 }
diff --git a/libraries/lwIP_w5500/src/utility/w5500.h b/libraries/lwIP_w5500/src/utility/w5500.h
@@ -38,6 +38,7 @@
 #include <stdint.h>
 #include <Arduino.h>
 #include <SPI.h>
+#include <LwipEthernet.h>
 
 class Wiznet5500 {
 public:
@@ -83,7 +84,10 @@ class Wiznet5500 {
         @return true when physical link is up
     */
     bool isLinked() {
-        return wizphy_getphylink() == PHY_LINK_ON;
+        ethernet_arch_lwip_begin();
+        auto ret = wizphy_getphylink() == PHY_LINK_ON;
+        ethernet_arch_lwip_end();
+        return ret;
     }
 
     /**