[riot-notifications] [RIOT-OS/RIOT] EUI48: Add address flag helper functions (#11666)

Koen Zandberg notifications at github.com
Mon Jun 10 20:48:11 CEST 2019


bergzand commented on this pull request.



> @@ -56,6 +56,24 @@ static inline void eui48_to_eui64(eui64_t *eui64, const eui48_t *addr)
     eui64->uint8[7] = addr->uint8[5];
 }
 
+/**
+ * @name EUI-48 bit flags contained in the first octet
+ *
+ * @see IEEE 802-2001 section 9.2
+ * @{
+ */
+
+/**
+ * @brief Locally administered address.
+ */
+#define EUI48_LOCAL_ADMIN_FLAG  0x02

Quoting the IEEE 802 spec:
> The Individual/Group (I/G) address bit

and
> The Universally or Locally administered (U/L) address

I think I disagree with you here as the address itself is not local, but the address is locally administered (shortened to `LOCAL_ADMIN`) whereas with the other case the address is a group address. Although I don't mind shortening it to `EUI48_LOCAL_FLAG` if you insist.

-- 
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/11666#discussion_r292140968
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190610/5a2ca3b0/attachment.html>


More information about the notifications mailing list