-
-
Notifications
You must be signed in to change notification settings - Fork 349
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
What are the lwip includes for? #391
Comments
In early days sming was using the openlwip library. |
Delete the folder and try to compile it, you'll get a big list of places! |
It looks like we use structures from those headers that don't exist in the Espressif headers, which is worrying. |
Can't check now but will tomorrow. |
Wow, no wonder we have network problems then! |
Of course not absolutely sure but LWIP is more open/standard than the espressif layer above it. |
I'm looking at that at the moment, but as usual from espressif there is no information about it at all, just a file dumped in a forum. |
I know, and as it at the base of Sming we should be very careful on this and not updating without severe testing. |
Don't worry, I can't even work out how you are supposed to compile it. |
Well got some stuff compiling, but not had any time to play with it further. Not sure how much time I'll have over the next week either, got night shifts this week and some house rewiring to do in between. |
I assume the espressif libraries use the function in their version of lwip, so if we go with a pure lwip I don't imagine their libraries will link against it. I'm assuming the new one they have released is normal lwip with their bits on top, but I haven't looked into it that much yet. |
@raburton : |
I got as far as compiling it, but I didn't go on to try and use it because you told me not to! |
I know, just asking what you did to make it compile, not to finish the integration. There is the directory : lwip_open_src_template_proj, which includes an application user_main.c |
I added my own make files to build the libraries. Try this: https://dl.dropboxusercontent.com/u/5500141/makes.zip |
Thx, Will give it a try later today. |
@raburton : Changing to the RTOS SDK does not work however. |
Found an issue when moving to sdk v1.4 that was is system/include/lwip/mem.h but I can't work out why we have all these includes at all! We don't use the libopenlwip that is included in sming so I've removed it, but we seem to use these headers and I don't understand why.
The text was updated successfully, but these errors were encountered: