Skip to content

Fix: W5500 Ethernet initialization and network fallback stability on …#2297

Draft
philvana wants to merge 1 commit into1technophile:developmentfrom
philvana:fix/xiao-w5500-ethernet-bringup
Draft

Fix: W5500 Ethernet initialization and network fallback stability on …#2297
philvana wants to merge 1 commit into1technophile:developmentfrom
philvana:fix/xiao-w5500-ethernet-bringup

Conversation

@philvana
Copy link
Copy Markdown

…XIAO ESP32S3

  • Add ETH macro translation layer (ETH_* -> ETH_PHY_*) for Arduino-ESP32 compatibility
  • Refactor wifi_reconnect_bypass() to prevent watchdog resets from repeated WiFi.begin() calls
  • Split Ethernet state tracking into separate link detection and IP acquisition phases
  • Implement explicit SPI initialization matching Seeed W5500 reference implementation
  • Add conditional WiFiManager skip when Ethernet link is detected
  • Ensure stable fallback to WiFi/AP mode when Ethernet is disconnected

Fixes W5500 adapter not being detected on XIAO ESP32S3. Maintains compatibility with all existing BLE Coded PHY features. Tested on Seeed XIAO ESP32S3 with W5500 Ethernet Adapter.

Description:

Checklist:

  • The pull request is done against the latest development branch
  • Only one feature/fix was added per PR and the code change compiles without warnings
  • I accept the DCO.

…XIAO ESP32S3

- Add ETH macro translation layer (ETH_* -> ETH_PHY_*) for Arduino-ESP32 compatibility
- Refactor wifi_reconnect_bypass() to prevent watchdog resets from repeated WiFi.begin() calls
- Split Ethernet state tracking into separate link detection and IP acquisition phases
- Implement explicit SPI initialization matching Seeed W5500 reference implementation
- Add conditional WiFiManager skip when Ethernet link is detected
- Ensure stable fallback to WiFi/AP mode when Ethernet is disconnected

Fixes W5500 adapter not being detected on XIAO ESP32S3.
Maintains compatibility with all existing BLE Coded PHY features.
Tested on Seeed XIAO ESP32S3 with W5500 Ethernet Adapter.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants