search

martin-ger / esp-open-lwip

ESP8266 lwIP library with NAT, SLIP, ENC28j60 Ethernet, and routing support
67 stars 25 forks source link

Fix so that the code actually does the data scoop. Prevents NPE. #16

Closed xxxajk closed 6 years ago

xxxajk commented 6 years ago

Typical 1 line fix. [00:19:29] 106173 KiB in 75696 KiB out 115072 B/s in 4212 B/s out 55688 free heap Nice thruput.

xxxajk commented 6 years ago

Don't touch the enc driver, I'm doing some new optimizations on it. I'll do a pull request when it is ready.

xxxajk commented 6 years ago

[00:16:12] 42921 KiB in 35831 KiB out 124632 B/s in 4368 B/s out 55984 free heap got an extra 10K/sec out of it... The only way I can think of getting more out of it is if the SPI speed is increased. That will require some minimal additional parts ... probabbly pull-up resistors on MISO/MOSI/SCLK to help reject noise, providing noise is really the issue.

martin-ger commented 6 years ago

Adding pullups would be easy in my breadboard setup.

martin-ger commented 6 years ago

Issues with the SDK 2.2: when I was trying to integrate the new stuff, I came across some issures with the new SDK:

Currently I will stay with the old SDK for the binaries of the main version.

xxxajk commented 6 years ago

New SDK has different setup. See the reference manual. If in doubt perform recalibration.

On Fri, Nov 9, 2018, 8:11 AM martin-ger <notifications@github.com wrote:

Issues with the SDK 2.2: when I was trying to integrate the new stuff, I came across some issures with the new SDK:

-

sometimes during WiFi-traffic (e.g. Web) a new message pops up: pb:0x40107ac0 pb:0x40107ac0 pb:0x40107ac0 pb:0x40107ac0 pb:0x40107ac0 pb:0x40107ac0 pb:0x40107ac0 No idea, but I guess some memory handling problem

also the esp_init_data_default seems to be corrupted/wrong sometimes - no boot at all. My mechanism in config_flash.c in user_rf_pre_init() to fix that seems to fail - it used to work pretty reliably in the old SDK.

Currently I will stay with the old SDK for the binaries of the main version.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/martin-ger/esp-open-lwip/pull/16#issuecomment-437355265, or mute the thread https://github.com/notifications/unsubscribe-auth/ADskzDYNBbHnJnHP70HbbFBZSZjrZJGIks5utX8WgaJpZM4YV3DI .