svn commit: r304439 - head/sys/dev/usb/net

[YongHyeon PYUN]

On Fri, Aug 19, 2016 at 10:50:47AM +0200, Hans Petter Selasky wrote:
> On 08/19/16 02:50, Pyun YongHyeon wrote:
> >Modified: head/sys/dev/usb/net/if_axgereg.h
> >==============================================================================
> >--- head/sys/dev/usb/net/if_axgereg.h	Fri Aug 19 00:03:41 2016 (r304438)
> >+++ head/sys/dev/usb/net/if_axgereg.h	Fri Aug 19 00:50:32 2016 (r304439)
> >@@ -156,19 +156,20 @@ enum {
> > struct axge_frame_txhdr {
> > #if BYTE_ORDER == LITTLE_ENDIAN
> > 	uint32_t		len;
> >-#define	AXGE_TXLEN_MASK		0x0001FFFF
> >-#define	AXGE_VLAN_INSERT	0x20000000
> >-#define	AXGE_CSUM_DISABLE	0x80000000
> > 	uint32_t		mss;
> >-#define	AXGE_MSS_MASK		0x00003FFF
> >-#define	AXGE_PADDING		0x80008000
> >-#define	AXGE_VLAN_TAG_MASK	0xFFFF0000
> > #else
> > 	uint32_t		mss;
> > 	uint32_t		len;
> > #endif
> > } __packed;
> >
> 
> Hi,
> 
> Is it correct to switch the order of mss and len variables for 
> bit/little endian? Looks buggy to me.
> 

I think the order is right but it was not tested on big-endian
systems.