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

Make gcc-built lwIP default one #1926

Closed
3 tasks done
igrr opened this issue Apr 17, 2016 · 2 comments
Closed
3 tasks done

Make gcc-built lwIP default one #1926

igrr opened this issue Apr 17, 2016 · 2 comments

Comments

@igrr
Copy link
Member

igrr commented Apr 17, 2016

In SDK 1.5.3 lwIP source code in included with the SDK release, which is a good indication that future SDK versions will include that as well. This issue tracks switching from Espressif-provided xcc-built lwIP to gcc-built one.

  • Remove workarounds made in ClientContext to work around return value passing bug
  • Update lwIP source code from the latest SDK and build new binary
  • Update boards.txt to use gcc built lwip by default

This will enable us to fix #1924, #1826, #1505, #1050, #1262.

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

@igrr igrr added this to the 2.2.0 milestone Apr 17, 2016
@igrr igrr mentioned this issue Apr 18, 2016
4 tasks
@igrr igrr modified the milestones: 2.2.0, 2.3.0 Apr 18, 2016
igrr added a commit that referenced this issue May 6, 2016
It is still possible to use xcc-built LwIP by selecting "Core Development" board from tools menu.
@andig
Copy link
Contributor

andig commented May 8, 2016

@igrr fyi- I remember you metioning that gcc code size is bugger, however according to my tests it became smaller when switching lwip.

@igrr
Copy link
Member Author

igrr commented May 8, 2016

Yep, I think I have used -O0 in my early tests.
@me-no-dev actually pointed out that gcc-prebuilt LwIP produces the smallest binary when he added an option to switch between LwIP versions from the IDE.

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

No branches or pull requests

2 participants