[riot-notifications] [RIOT-OS/RIOT] drivers/mrf24j40: add Turbo Mode (#14047)

benpicco notifications at github.com
Tue May 19 11:20:19 CEST 2020


@benpicco commented on this pull request.



> +        case NETOPT_IEEE802154_PHY:
+            assert(max_len >= sizeof(int8_t));
+            *(uint8_t *)val = IEEE802154_PHY_OQPSK;
+            return sizeof(uint8_t);

I agree that this is a bit awkward.
I wanted a way to being able to disable it when it's not needed and as the `high_rate` option is only available with `netdev_ieee802154_oqpsk` I thought that's the way to go.

However this is explicitly a non-IEEE802.15.4 mode, so maybe having a separate `netdev_ieee802154_oqpsk_high_rate` pseudo-module would be better here.

-- 
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/14047#discussion_r427155847
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20200519/e6eb9adb/attachment-0001.htm>


More information about the notifications mailing list