Changed support W5500 SPI ethernet using four SPI GPIOs only without …

…IRQ and RESET
arendst · May 23, 2024 · a9e7426 · a9e7426
1 parent 522f6c5
commit a9e7426
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 3 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -18,6 +18,7 @@ All notable changes to this project will be documented in this file.
 - Berry binary compiled with gcc (#21426)
 - GPIOViewer from v1.5.2 to v1.5.3 (No functional change)
 - ESP32 I2S audio improvements (#21433)
+- Support W5500 SPI ethernet using four SPI GPIOs only without IRQ and RESET
 
 ### Fixed
 - Domoticz re-subscribe on MQTT reconnect. Regression from v13.4.0.3 (#21281)

diff --git a/RELEASENOTES.md b/RELEASENOTES.md
@@ -131,6 +131,7 @@ The latter links can be used for OTA upgrades too like ``OtaUrl https://ota.tasm
 ### Changed
 - GPIOViewer from v1.5.2 to v1.5.3
 - On universal display remove default backlight power if a PWM channel is used for backlight
+- Support W5500 SPI ethernet using four SPI GPIOs only without IRQ and RESET
 - ESP32 compiler option from `target-align` to `no-target-align` [#21407](https://github.com/arendst/Tasmota/issues/21407)
 - ESP32 I2S audio improvements [#21433](https://github.com/arendst/Tasmota/issues/21433)
 - Berry binary compiled with gcc [#21426](https://github.com/arendst/Tasmota/issues/21426)

diff --git a/tasmota/tasmota_xdrv_driver/xdrv_82_esp32_ethernet.ino b/tasmota/tasmota_xdrv_driver/xdrv_82_esp32_ethernet.ino
@@ -233,9 +233,9 @@ void EthernetInit(void) {
 #endif  // CONFIG_ETH_USE_ESP32_EMAC
 
   if (eth_uses_spi) {
-    // Uses SPI Ethernat
-    if (!PinUsed(GPIO_ETH_PHY_MDC) || !PinUsed(GPIO_ETH_PHY_MDIO) || !PinUsed(GPIO_ETH_PHY_POWER)) {
-      AddLog(LOG_LEVEL_DEBUG_MORE, PSTR(D_LOG_ETH "No ETH MDC (SPI CS), ETH MDIO (SPI IRQ) and ETH POWER (SPI RST) GPIO defined"));
+    // Uses SPI Ethernet and needs at least SPI CS being ETH MDC
+    if (!PinUsed(GPIO_ETH_PHY_MDC)) {
+      AddLog(LOG_LEVEL_DEBUG_MORE, PSTR(D_LOG_ETH "No ETH MDC as SPI CS GPIO defined"));
       return;
     }
   } else {