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

Koen Zandberg notifications at github.com
Mon Jun 10 21:08:22 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

> I know that this is about the uniqueness of the address, not the scope ;-).

I know that you're aware of that, but the reader might not be. I agree that the names should match. Anyway, enough discussion, You've convinced me. I've renamed the define

-- 
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_r292148728
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190610/2c05150e/attachment.html>


More information about the notifications mailing list