[riot-notifications] [RIOT-OS/RIOT] sys/net/iee802154/radio: add IEEE802154_RADIO_INDICATION_CRC_ERROR (#15397)

Francisco notifications at github.com
Wed Jan 13 14:10:35 CET 2021


@fjmolinas commented on this pull request.



> @@ -255,6 +282,7 @@ typedef struct {
      */
     uint8_t rssi;
     uint8_t lqi;    /**< LQI of the received frame */
+    uint8_t flags;  /**< flags as defined above */

I think in general it might be uneeded, I used in OpenWSN because of its internal logic:

- on a valid or invalid CRC a callback is called then `radio_getReceivedFrame` is called that then calls `ieee802154_radio_read` and so I used the `flags` to see if there was an invalid CRC

But if there is no other use case for this I can set a global variable for this and remove this extra `flags` variable. I'll do this since there is not other use currently.

-- 
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/15397#discussion_r556507710
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20210113/e2eb6fbc/attachment.htm>


More information about the notifications mailing list