[riot-notifications] [RIOT-OS/RIOT] drivers/at86rf2xx: Provide confirm_send() (#16273)

José Alamos notifications at github.com
Tue Apr 6 14:01:42 CEST 2021


@jia200x commented on this pull request.



> @@ -251,6 +251,7 @@ typedef struct {
     uint16_t flags;                         /**< Device specific flags */
     uint8_t state;                          /**< current state of the radio */
     uint8_t tx_frame_len;                   /**< length of the current TX frame */
+    uint8_t status;                         /**< Tx/Rx status (TRAC_STATUS) */

We could do the same in `_isr` when `#ifdef MODULE_OPENTHREAD`. As mentioned earlier, this status only tells what was the result of TX_ARET (it's not even read in BASIC_MODE).
 
Besides that, we could drop these lines if we do a minor adoption to the event callbacks of netdev. At least GNRC is able to differentiate TX_COMPLETE from TX_COMPLETE_DATA_PENDING.

-- 
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/16273#discussion_r607786262
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20210406/2719a907/attachment.htm>


More information about the notifications mailing list