Skip to content

UDP for RAK4631 Eth Gw and the t-eth-elite. Solves #7149#7385

Merged
thebentern merged 5 commits into
masterfrom
rak-udp
Jul 20, 2025
Merged

UDP for RAK4631 Eth Gw and the t-eth-elite. Solves #7149#7385
thebentern merged 5 commits into
masterfrom
rak-udp

Conversation

@caveman99

Copy link
Copy Markdown
Member

Also enable IP output on Portduino.

@caveman99 caveman99 requested review from Copilot and thebentern July 19, 2025 14:09

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR enables UDP multicast functionality for ethernet gateway devices (RAK4631 Eth Gw and t-eth-elite) and enhances IP output support on Portduino. The changes address issue #7149 by implementing UDP multicast capabilities for ethernet-connected devices.

Key changes:

  • Enables UDP multicast support for RAK4631 and t-eth-elite ethernet gateway devices
  • Implements platform-specific AsyncUDP class for NRF52 architecture with ethernet support
  • Updates UDP multicast handler to support ethernet connectivity alongside WiFi

Reviewed Changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
variants/t-eth-elite/platformio.ini Adds HAS_UDP_MULTICAST=1 build flag to enable UDP multicast
variants/rak4631_eth_gw/platformio.ini Adds HAS_UDP_MULTICAST=1 build flag to enable UDP multicast
src/platform/nrf52/AsyncUDP.h New header file defining AsyncUDP and AsyncUDPPacket classes for NRF52 ethernet
src/platform/nrf52/AsyncUDP.cpp Implementation of AsyncUDP functionality using EthernetUDP
src/mesh/udp/UdpMulticastHandler.h Updates to support NRF52 ethernet and improved IP address logging
src/mesh/eth/ethClient.h Removes unused functional include
src/mesh/eth/ethClient.cpp Adds UDP handler initialization when ethernet connects
arch/portduino/portduino.ini Standardizes HAS_UDP_MULTICAST flag format

Comment thread src/platform/nrf52/AsyncUDP.cpp
Comment thread src/platform/nrf52/AsyncUDP.cpp Outdated
Comment thread src/platform/nrf52/AsyncUDP.h
Comment thread src/platform/nrf52/AsyncUDP.cpp
caveman99 and others added 2 commits July 19, 2025 16:11
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@vidplace7 vidplace7 added the enhancement New feature or request label Jul 19, 2025
@caveman99 caveman99 self-assigned this Jul 19, 2025
@caveman99 caveman99 requested a review from jp-bennett July 19, 2025 20:48
@thebentern thebentern merged commit a9c9b96 into master Jul 20, 2025
53 checks passed
@caveman99 caveman99 deleted the rak-udp branch July 20, 2025 11:53
oscgonfer pushed a commit to fablabbcn/smartcitizen-meshtastic that referenced this pull request Jul 22, 2025
…eshtastic#7385)

* UDP for RAK4631 Eth Gw and the t-eth-elite. Also enable IP output on Portduino. Solves meshtastic#7149

* Copilot suggestion

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* fix portduino build

* initialize local port

---------

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Ben Meadors <benmmeadors@gmail.com>
oscgonfer pushed a commit to fablabbcn/smartcitizen-meshtastic that referenced this pull request Jul 23, 2025
…eshtastic#7385)

* UDP for RAK4631 Eth Gw and the t-eth-elite. Also enable IP output on Portduino. Solves meshtastic#7149

* Copilot suggestion

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* fix portduino build

* initialize local port

---------

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Ben Meadors <benmmeadors@gmail.com>
jeek pushed a commit to jeek/Meshtastic-Exploiteers-Hacker-Pager that referenced this pull request Jun 30, 2026
…eshtastic#7385)

* UDP for RAK4631 Eth Gw and the t-eth-elite. Also enable IP output on Portduino. Solves meshtastic#7149

* Copilot suggestion

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* fix portduino build

* initialize local port

---------

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Ben Meadors <benmmeadors@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants