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

[Hans Petter Selasky]

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.

--HPS