[riot-notifications] [RIOT-OS/RIOT] gnrc_lorawan : Add ADR support (#15995)

Akshai M (Augustus Diode) notifications at github.com
Fri Jul 16 14:09:04 CEST 2021


@akshaim commented on this pull request.



> +    for (; index < len; index += GNRC_LORAWAN_FOPT_LINK_ADR_REQ_SIZE ) {
+        if ( p[index] == GNRC_LORAWAN_CID_LINK_ADR_REQ )
+        {
+            mac->mlme.adr_req_cnt++;
+        }
+    }

As discussed offline , In case of  `LinkAdrReq1 - LinkAdrReq2 - AnotherCommand - LinkAdrReq3`. The idea was  to accommodate `LinkAdrReq3`. Since the standard is vague and Semtech stack rejects it, we may discard the third one. Shall correct it.


-- 
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/15995#discussion_r671193393
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20210716/a5666d97/attachment.htm>


More information about the notifications mailing list