Skip to content

Upgrade to lwIP 2.2.1#66

Open
mitchellwaite wants to merge 9 commits intoFree60Project:masterfrom
mitchellwaite:lwipUpgradeThatActuallyWorked
Open

Upgrade to lwIP 2.2.1#66
mitchellwaite wants to merge 9 commits intoFree60Project:masterfrom
mitchellwaite:lwipUpgradeThatActuallyWorked

Conversation

@mitchellwaite
Copy link

@mitchellwaite mitchellwaite commented Feb 13, 2026

Who doesn't love an upgrade to a modern version of a dependency? Here we've got lwIP 2.2.1 (Released Feb 6, 2025).... 1.4.1 was released in ~2012 or so (14 years ago)

Main changes (other than replacing the lwip sources) are as follows:

  • Makefile was updated to include the correct set of lwip object files, and make the install command happy by creating the proper directory structure. There were some changes to the source files in newer versions of lwip
  • sys_check_timeouts() was fixed, so we don't need to manually check and fire each timer (of which there are more to worry about in LWIP 2.2.1)
  • netif_set_link_up and netif_set_up are called in the appropriate places, lwip < 1.4.1 is more strict about the interface being up before it will transmit packets
  • types provided by stdint.h were removed from cc.h
  • console "waiting for DHCP" message was updated so we don't print as many dots to the screen, rather we flash a dot while waiting. DHCP init takes a bit longer in 2.2.1

NOTE: Free60Project/xell-reloaded#35 is a prerequisite to this for XeLL, commit Free60Project/xell-reloaded@9c357a7 contains an update to tftp.c that resolves compile issues and allows it to compile on both 1.4.1 and 2.2.1

ELF for testing:
xenon.elf.zip

DHCP_DOES_ARP_CHECK replaced with LWIP_DHCP_DOES_ACD_CHECK in lwip commit 7d1c26cc0c18d02cc72f348f9189dd1c3a06bfca

Update lwipopts to indicate this.

Also, update network_init to not print *so many* dots to the console output. Rather, it will flash one dot on the sceen while waiting for DHCP
@mitchellwaite mitchellwaite marked this pull request as ready for review February 14, 2026 07:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant