feat(net): Add ETH handle to events and data to lost_ip#11986
Merged
feat(net): Add ETH handle to events and data to lost_ip#11986
Conversation
Contributor
👋 Hello me-no-dev, we appreciate your contribution to this project! 📘 Please review the project's Contributions Guide for key guidelines on code, documentation, testing, and more. 🖊️ Please also make sure you have read and signed the Contributor License Agreement for this project. Click to see more instructions ...
Review and merge process you can expect ...
|
Contributor
There was a problem hiding this comment.
Pull Request Overview
This pull request adds event data fields to the arduino_event_info_t union to support populating event information for lost IP and Ethernet lifecycle events (start, stop, connect, disconnect). The changes ensure event consumers can access relevant context data when these events occur.
- Adds
lost_ipfield to store IP event data when an interface loses its IP address - Adds
eth_started,eth_stopped, andeth_disconnectedfields to store Ethernet handle information for lifecycle events - Populates these new fields in the event handlers where the corresponding events are generated
Reviewed Changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| libraries/Network/src/NetworkEvents.h | Adds new union members for lost_ip and Ethernet event handles |
| libraries/Network/src/NetworkInterface.cpp | Populates lost_ip field when IP_EVENT_LOST_IP occurs |
| libraries/Ethernet/src/ETH.cpp | Populates Ethernet handle fields for START, STOP, and DISCONNECTED events |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Contributor
Test Results 76 files 76 suites 16m 52s ⏱️ Results for commit 5133773. ♻️ This comment has been updated with latest results. |
Contributor
|
Seems to work just fine. |
Contributor
Memory usage test (comparing PR against master branch)The table below shows the summary of memory usage change (decrease - increase) in bytes and percentage for each target.
Click to expand the detailed deltas report [usage change in BYTES]
|
|||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
P-R-O-C-H-Y
approved these changes
Nov 6, 2025
lucasssvaz
approved these changes
Nov 6, 2025
me-no-dev
added
Status: Pending Merge
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This pull request enhances the Ethernet and network event handling by ensuring that event data is consistently populated for various Ethernet and IP-related events. The main improvements include passing the Ethernet handle in more event types and adding support for lost IP event data.
Ethernet event handling improvements:
esp_eth_handle_t) is now included in the event info forETHERNET_EVENT_DISCONNECTED,ETHERNET_EVENT_START, andETHERNET_EVENT_STOPevents, ensuring consumers of these events have access to the relevant interface handle. [1] [2]Network event data structure updates:
arduino_event_info_tunion now includes fields foreth_started,eth_stopped, andeth_disconnected, in addition to the existingeth_connected, to support the new event info assignments.lost_ipfield toarduino_event_info_tto store information about lost IP events.Lost IP event handling:
ip_event_got_ip_t) into the newlost_ipfield, ensuring that details about the lost IP are available to event consumers. [1] [2]cc: @TD-er