Skip to content

pkg/lwip: add support for esp32 Ethernet device#12950

Merged
benpicco merged 5 commits intoRIOT-OS:masterfrom
gschorcht:cpu/esp32/lwip_eth
Dec 16, 2019
Merged

pkg/lwip: add support for esp32 Ethernet device#12950
benpicco merged 5 commits intoRIOT-OS:masterfrom
gschorcht:cpu/esp32/lwip_eth

Conversation

@gschorcht
Copy link
Copy Markdown
Contributor

Contribution description

This PR adds the support for ESP32 Ethernet device to pkg/lwip in same way as PR #12895 does. Addititionally, it adds the IPv4 support for the ESP32 Ethernet device as PR #12903.

Testing procedure

Flash tests/lwip with enabled module esp_wifi on any ESP32 board with an Ethernet interface (for example Olimex ESP32-EVB)

USEMODULE=esp_eth make BOARD=esp32-olimex-evb -C tests/lwip flash term

and execute ifconfig on ESP32 node and ping the node from any machine in the LAN.

Issues/PRs references

Related to PR #12903.

@gschorcht gschorcht added Area: pkg Area: External package ports CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Platform: ESP Platform: This PR/issue effects ESP-based platforms Type: enhancement The issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation Type: new feature The issue requests / The PR implemements a new feature for RIOT and removed Type: enhancement The issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation Type: new feature The issue requests / The PR implemements a new feature for RIOT labels Dec 14, 2019
Copy link
Copy Markdown
Contributor

@benpicco benpicco left a comment

Choose a reason for hiding this comment

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

Looks good, just follows the same pattern contrib/lwip.c.

@benpicco
Copy link
Copy Markdown
Contributor

Please squash!

@fjmolinas fjmolinas added this to the Release 2020.01 milestone Dec 15, 2019
@gschorcht
Copy link
Copy Markdown
Contributor Author

@benpicco Squashed.

@benpicco benpicco merged commit 8c4498a into RIOT-OS:master Dec 16, 2019
@gschorcht gschorcht deleted the cpu/esp32/lwip_eth branch December 16, 2019 09:54
@gschorcht
Copy link
Copy Markdown
Contributor Author

Thanks for reviewing and merging.

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

Labels

Area: pkg Area: External package ports CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Platform: ESP Platform: This PR/issue effects ESP-based platforms Type: enhancement The issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants