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

Martine Lenders notifications at github.com
Mon Jun 10 20:43:23 CEST 2019


miri64 commented on this pull request.

One last nit-pick ^^"

> @@ -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

Sorry, last minute thing I noticed. Shouldn't this also be called `EUI48_LOCAL_FLAG` for consistency? (You may squash immediatelly)

-- 
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#pullrequestreview-247777807
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190610/a5aac912/attachment-0001.html>


More information about the notifications mailing list