[riot-notifications] [RIOT-OS/RIOT] drivers/cc110x: Complete rewrite from scratch (#10340)

Marian Buschsieweke notifications at github.com
Mon Feb 4 12:03:53 CET 2019


@PeterKietzmann: Judging from a quick look it seems that using thread flags would be a way to work around that issue.

But https://github.com/RIOT-OS/RIOT/pull/8304 added 49 lines of code (62 - 13) and a dependency (`USEMODULE += core_thread_flags) and https://github.com/RIOT-OS/RIOT/pull/10403 added 15 (19 - 4) and no dependency. Keeping also in mind that waiting in GNRC for the driver would result in one place to implement it, while implementing it in the network device driver would result in duplication of this feature. (@miri64: Is it right that https://github.com/RIOT-OS/RIOT/pull/8304 could be basically reverted if (the simple version without MAC features of) https://github.com/RIOT-OS/RIOT/pull/8304 would be merged?)

There might be even more drivers suffering the issue of not being able to handle a second send request while the current transmission has not finished yet, if no one ever tried to send large enough frames to required 6LoWPAN fragmentation. If that is true, https://github.com/RIOT-OS/RIOT/pull/8304 would fix that without anyone ever noticing the problem :-)

-- 
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/10340#issuecomment-460207872
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190204/e9e13316/attachment.html>


More information about the notifications mailing list