-
Notifications
You must be signed in to change notification settings - Fork 3k
Disabled lwip ethernet ipv6 multicast filter for STM boards #4513
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.
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
Conversation
@@ -0,0 +1,48 @@ | |||
#include "stm32f7xx_hal.h" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please add license to the new files
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Corrected.
.VLANTagIdentifier = 0x0 | ||
}; | ||
|
||
if (heth->Init.ChecksumMode == ETH_CHECKSUM_BY_HARDWARE) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please fix the formatting
if () {
//code here
} else {
}
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Corrected.
How was this tested? Before the patch - any failures, after this one - what is fixed ? |
Verified with UBLOX EVK ODIN W2 board that ipv6 IP address allocation is working with this fix and mbed-os-example-application is able to connect to mbed device connector. Without this fix as in the original error multicast packets are not received by lwip stack and ipv6 address cannot be configured resulting -3010 error. |
Hi |
/morph test |
Result: SUCCESSYour command has finished executing! Here's what you wrote!
OutputAll builds and test passed! |
Description
Disabled lwip ethernet ipv6 multicast filter for STM boards.
This is correction to #4432
Status
READY
Migrations
NO
Related PRs
None
Todos
Deploy notes
None
Steps to test or reproduce
None