Skip to content

WiFi udp Multicast #119

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 4 commits into from
Jun 10, 2025
Merged

WiFi udp Multicast #119

merged 4 commits into from
Jun 10, 2025

Conversation

IFX-Anusha
Copy link
Contributor

@IFX-Anusha IFX-Anusha commented May 27, 2025 •
edited
Loading

  • BeginMulticast function - set up a UDP socket to receive UDP packets sent to a multicast group address.
  • Refactor the begin function

@IFX-Anusha IFX-Anusha force-pushed the WiFiUDP_multicast branch from fb42717 to 4d6b394 Compare May 27, 2025 12:10
jaenrig-ifx
jaenrig-ifx approved these changes May 30, 2025
View reviewed changes
Copy link
Member

@jaenrig-ifx jaenrig-ifx left a comment

Choose a reason for hiding this comment

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

All good if the manual tests are passing! 🥳

ramya-subramanyam
ramya-subramanyam approved these changes May 30, 2025
View reviewed changes
Copy link
Collaborator

@ramya-subramanyam ramya-subramanyam left a comment

Choose a reason for hiding this comment

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

Nice!

@IFX-Anusha IFX-Anusha force-pushed the WiFiUDP_multicast branch from 4d6b394 to bc2a13a Compare June 2, 2025 07:20
@ederjc ederjc self-requested a review June 6, 2025 06:39
ederjc
ederjc previously requested changes Jun 6, 2025
View reviewed changes
Copy link
Member

@ederjc ederjc left a comment

Choose a reason for hiding this comment

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

Needs manual testing as discussed

@IFX-Anusha
libraries/WiFi: BeginMulticast function.
938e823 
Signed-off-by: IFX-Anusha <Anusha.TR@infineon.com>
@IFX-Anusha
libraries/WiFi: Refactor begin function.
02c0218 
Signed-off-by: IFX-Anusha <Anusha.TR@infineon.com>
@IFX-Anusha
tests/arduino-core-tests: Added test case.
658d0cd 
Signed-off-by: IFX-Anusha <Anusha.TR@infineon.com>
@IFX-Anusha
libraries/WiFi: Fix in read function to read only available len.
dd1fc9f 
Signed-off-by: IFX-Anusha <Anusha.TR@infineon.com>
@IFX-Anusha IFX-Anusha requested a review from ederjc June 10, 2025 09:34
@IFX-Anusha IFX-Anusha dismissed ederjc’s stale review June 10, 2025 09:35

Manual testing is passing

@IFX-Anusha IFX-Anusha requested a review from jaenrig-ifx June 10, 2025 09:36
@IFX-Anusha
Copy link
Contributor Author

Needs manual testing as discussed

Done

jaenrig-ifx
jaenrig-ifx approved these changes Jun 10, 2025
View reviewed changes
Copy link
Member

@jaenrig-ifx jaenrig-ifx left a comment

Choose a reason for hiding this comment

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

👌 nice!

@IFX-Anusha IFX-Anusha merged commit 97f3487 into main Jun 10, 2025
12 checks passed
@IFX-Anusha IFX-Anusha deleted the WiFiUDP_multicast branch June 10, 2025 09:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jaenrig-ifx jaenrig-ifx jaenrig-ifx approved these changes

@ederjc ederjc Awaiting requested review from ederjc

@ramya-subramanyam ramya-subramanyam Awaiting requested review from ramya-subramanyam

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@IFX-Anusha @ederjc @jaenrig-ifx @ramya-subramanyam