Bill Fischofer(Bill-Fischofer-Linaro) replied on github web page:

helper/include/odp/helper/vxlan.h
line 42
@@ -0,0 +1,58 @@
+/* Copyright (c) 2017, Linaro Limited
+ * All rights reserved.
+ *
+ * SPDX-License-Identifier:     BSD-3-Clause
+ */
+
+/**
+ * @file
+ *
+ * ODPH VxLAN header
+ */
+
+#ifndef ODPH_VXLAN_H_
+#define ODPH_VXLAN_H_
+
+#ifdef __cplusplus
+extern "C" {
+#endif
+
+#include <odp_api.h>
+
+/** @addtogroup odp_header ODP HEADER
+ *  @{
+ */
+
+/** VXLAN header length */
+#define ODPH_VXLANHDR_LEN 8
+
+/** VNI from VXLAN header format */
+#define ODPH_VXLAN_VNI(vxlan_vni) ((vxlan_vni & 0xffffff00) >> 8)
+
+/** UDP VXLAN destination port */
+#define ODPH_UDP_VXLAN_PORT 4789
+
+/** VNI to VXLAN header format */
+#define ODPH_VNI_VXLAN(vxlan_vni) (vxlan_vni << 8)
+
+/** VXLAN header */
+typedef struct ODP_PACKED {
+       uint8_t flags; /**< Vxlan flags */
+       uint8_t reserve1; /**< reserved bits */
+       uint16_t reseve2; /**< reserved bits */


Comment:
Typo: reserve, not reseve? Perhaps better for these to be anonymous bitfields 
if they are truly unreferenced?

https://github.com/Linaro/odp/pull/345#discussion_r157107756
updated_at 2017-12-15 02:13:15

Reply via email to