[riot-notifications] [RIOT-OS/RIOT] esp8266: esp_wifi netdev driver (#10792)

Sebastian Meiling notifications at github.com
Tue Jan 22 09:11:16 CET 2019


Hi @gschorcht 

first: please rebase, once again so we can get this merged 😄 
second, regarding stress testing: maybe increase the packet buffer a bit, its default is 6K (IIRC) but with 1.5K frames this might be a bit small, especially if once in a while some other packets (e.g. from neighbour discovery) are also around and take up some space.

In general, I don't see a problem in merging this even with these issues present. I mean this is still embedded hardware and under high load I would expect some problems. Nevertheless we should keep them on our list: maybe open a tracking issue for esp_wifi and work on it. But with this merged we will likely attract more people using it and also testing will be easier for others, not requiring them to merge/checkout this PR but rather use master.

@MichelRottleuthner: agreed?

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/RIOT-OS/RIOT/pull/10792#issuecomment-456307151
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190122/7441e449/attachment.html>


More information about the notifications mailing list