On Wed, Jun 26, 2019 at 12:49 PM David Miller [off-list ref] wrote:

From: Catherine Sullivan <redacted>
Date: Wed, 26 Jun 2019 11:52:49 -0700

quoted

+#ifdef __LITTLE_ENDIAN
+#define GVE_SEQNO(x) ((((__force u16)x) >> 8) & 0x7)
+#else
+#define      GVE_SEQNO(x) ((__force u16)(x) & 0x7)
+#endif

This can be simply "le16_to_cpu(x) & 0x7" or similar.  No need to
messy ifdefs.

Will fix in v2.