pure lwip2 integration proposal#271
Conversation
|
It has to be noted that there is not and won't be any ICACHE_FLASH_ATTR modifiers in original lwip source repository. So unlike with esp-open-lwip, liblwip2.a must appear in ldscripts to put symbols in the appropriate sections. I also use some objcopy tweaks for keeping some symbols in iram. |
|
@d-a-v : Thanks for taking effort to separate your solution to a separate project and to post a patch here! As a quick reply, I'm very busy these weeks, and small time I have, goes towards low-hanging or long-waiting things in other projects (that's why I didn't have chance to reply to your mail, sorry). So, I apologize in advance if it may take weeks to look into this in detail. (I hope other interested folks will look into it in the meantime, @davydnorris et al) |
|
@pfalcon: some updates here. I am nearly successful in compiling micropython (changes) with this PR. I am missing a "millis()"-like function. Instead of me going to pick one somewhere, is there some kind of similar function I could use in esp-open-sdk (or in micropython) ? That and ldscripts update for lwip2 and it will be tryable. |
|
In MicroPython, there's mp_hal_ticks_ms() which returns number of milliseconds since arbitrary point of time. esp-open-sdk doesn't offer any functions on its own, it's just a Makefile. ESP8266 system however itself offers system_get_time() which returns number of microseconds since arbitrary point of time. |
|
@pfalcon - I took some time to make changes to @d-a-v's repo to get it to compile cleanly with |
Sorry, wanted to post this reply earlier, but it slipped. But anyway, I don't do much of esp8266 hacking lately. I'd like to get back to it, but don't know when that would be. I see that @d-a-v continues doing some great work there, so I suggest to cooperate with him directly. |
|
@someburner I suggest you create a new pull-request here, made with your walkthrough. A way for esp-open-sdk users to easily test it should be provided If testing is as easy as this, hopefully @pfalcon would be able to run some basic validating tests. |
This is a proposal, following this thread.