Skip to content
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

Disabled lwip ethernet ipv6 multicast filter for STM boards #4513

Merged
merged 1 commit into from
Jun 9, 2017
Merged

Disabled lwip ethernet ipv6 multicast filter for STM boards #4513

merged 1 commit into from
Jun 9, 2017

Conversation

mikaleppanen
Copy link

Description

Disabled lwip ethernet ipv6 multicast filter for STM boards.

This is correction to #4432

Status

READY

Migrations

NO

Related PRs

None

Todos

  • Tests
  • Documentation

Deploy notes

None

Steps to test or reproduce

None

@@ -0,0 +1,48 @@
#include "stm32f7xx_hal.h"
Copy link
Contributor

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

Copy link
Author

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)
Copy link
Contributor

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 {
}

Copy link
Author

Choose a reason for hiding this comment

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

Corrected.

@0xc0170
Copy link
Contributor

0xc0170 commented Jun 9, 2017

@0xc0170
Copy link
Contributor

0xc0170 commented Jun 9, 2017

How was this tested? Before the patch - any failures, after this one - what is fixed ?

@mikaleppanen
Copy link
Author

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.

@0xc0170
Copy link
Contributor

0xc0170 commented Jun 9, 2017

@jeromecoutant
Copy link
Collaborator

Hi
Tested OK with NUCLEO-F429ZI
This solves also #4433

@0xc0170
Copy link
Contributor

0xc0170 commented Jun 9, 2017

/morph test

@mbed-bot
Copy link

mbed-bot commented Jun 9, 2017

Result: SUCCESS

Your command has finished executing! Here's what you wrote!

/morph test

Output

mbed Build Number: 519

All builds and test passed!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants