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

Update ChibiOS lwip to 2.1.3 #2791

Merged
merged 14 commits into from
Nov 14, 2023
Merged

Conversation

networkfusion
Copy link
Member

@networkfusion networkfusion commented Oct 14, 2023

Description

Bumps LWIP to next minor version that includes a number of fixes. See: https://git.savannah.nongnu.org/cgit/lwip.git/log/?h=STABLE-2_1_3_RELEASE

Motivation and Context

This brings the version in line with that used on ESP32 devices.

Improves ability to run on certain networks (such as ubiquiti unifi).
Improves ability to add IPv6 support.

How Has This Been Tested?

Running this on an orgpal3 with the networkfusion orgpal3 demo which extensively uses networking has not shown up any issues. It is comfortably getting an IP and using MbedTLS to send data to AWS IoT.

Screenshots

Types of changes

  • Improvement (non-breaking change that improves a feature, code or algorithm)
  • Bug fix (non-breaking change which fixes an issue with code or algorithm)
  • New feature (non-breaking change which adds functionality to code)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Config and build (change in the configuration and build system, has no impact on code or features)
  • Dev Containers (changes related with Dev Containers, has no impact on code or features)
  • Dependencies (update dependencies and changes associated, has no impact on code or features)
  • Documentation (changes or updates in the documentation, has no impact on code or features)

Checklist

  • My code follows the code style of this project (only if there are changes in source code).
  • My changes require an update to the documentation (there are changes that require the docs website to be updated).
  • I have updated the documentation accordingly (the changes require an update on the docs in this repo).
  • I have read the CONTRIBUTING document.
  • I have tested everything locally and all new and existing tests passed (only if there are changes in source code).

Retains current version but unlocks ability to update.
@networkfusion networkfusion added ⚠️ DO NOT MERGE ⚠️ Platform: STM32 Everything related specifically with ChibiOS platform labels Oct 14, 2023
@networkfusion networkfusion marked this pull request as ready for review October 16, 2023 21:55
@networkfusion networkfusion changed the title DRAFT: Update ChibiOS lwip to 2.1.3 Update ChibiOS lwip to 2.1.3 Oct 16, 2023
Copy link
Member

@josesimoes josesimoes left a comment

Choose a reason for hiding this comment

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

LGTM

@nfbot nfbot added Area: Config-and-Build Type: dependencies Pull requests that update a dependency file(s) or version labels Nov 14, 2023
@josesimoes josesimoes merged commit 1980be4 into nanoframework:main Nov 14, 2023
8 checks passed
@networkfusion networkfusion deleted the update-lwip branch November 14, 2023 18:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: Config-and-Build Platform: STM32 Everything related specifically with ChibiOS platform Type: dependencies Pull requests that update a dependency file(s) or version
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants