[riot-notifications] [RIOT-OS/RIOT] drivers/nrf24l01p: Netdev driver for nrf24l01p (#13743)

benpicco notifications at github.com
Mon May 4 14:02:07 CEST 2020


@benpicco commented on this pull request.



> +/**
+ * @brief   Reset value of TX_ADDR register
+ */
+#define NRF24L01P_NG_DEFAULT_TX_ADDR        { 0xE7, 0xE7, 0xE7, 0xE7, 0xE7 }
+
+/**
+ * @brief   Reset value of RX_ADDR_P0 register
+ */
+#define NRF24L01P_NG_DEFAULT_ADDR_P0        { 0xE7, 0xE7, 0xE7, 0xE7, 0xE7 }
+
+/**
+ * @brief   Reset value of RX_ADDR_P1 register
+ */
+#define NRF24L01P_NG_DEFAULT_ADDR_P1        { 0xC2, 0xC2, 0xC2, 0xC2, 0xC2 }

Why call `luid_base()` *and* `luid_get()`?
The latter is just a wrapper for the former - why is calling `luid_get(dev->params.urxaddr.rxaddrpx.rx_p1, addr_size)` not enough?

-- 
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/13743#discussion_r419385116
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20200504/96489da2/attachment.htm>


More information about the notifications mailing list