On Wed, Jun 26, 2019 at 12:49 PM David Miller <da...@davemloft.net> wrote: > > From: Catherine Sullivan <csu...@google.com> > Date: Wed, 26 Jun 2019 11:52:49 -0700 > > > +#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.
