[riot-notifications] [RIOT-OS/RIOT] radios: Remove default radio flags from init (#10355)

Koen Zandberg notifications at github.com
Fri Nov 9 14:21:21 CET 2018


bergzand commented on this pull request.



> @@ -104,8 +104,6 @@ void kw2xrf_reset_phy(kw2xrf_t *dev)
 
     kw2xrf_set_rx_watermark(dev, 1);
 
-    kw2xrf_set_option(dev, KW2XRF_OPT_AUTOACK, true);
-    kw2xrf_set_option(dev, KW2XRF_OPT_ACK_REQ, true);
     kw2xrf_set_option(dev, KW2XRF_OPT_AUTOCCA, true);

At the moment the kw2xrf is the only radio using the `NETOPT_AUTOCCA` option, so I'd rather leave it to a follow up PR if we want this change.

I was thinking, if we combine this CCA feature with some exponential back off timer, we effectively have CSMA right? Would this be something that we want to have for this radio?

-- 
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/10355#discussion_r232250356
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20181109/99d5ac23/attachment.html>


More information about the notifications mailing list