O
> +static noinline void pasemi_mac_free_tx_resources(struct net_device *dev)

Why tell compiler how to do it's job.  marking it noinline?

> Index: merge/drivers/net/pasemi_mac.h
> ===================================================================
> --- /dev/null
> +++ merge/drivers/net/pasemi_mac.h
                                         PAS_IOB_DMA_COM_TIMEOUTCFG_TCNT_M)
> +
> +struct pas_dma_xct_descr {
> +     union {
> +             u64     mactx;
> +#define      XCT_MACTX_T             0x8000000000000000
> +#define      XCT_MACTX_ST            0x4000000000000000
> +#define XCT_MACTX_NORES              0x0000000000000000
> +#define XCT_MACTX_8BRES              0x1000000000000000
> +#define XCT_MACTX_24BRES     0x2000000000000000
> +#define XCT_MACTX_40BRES     0x3000000000000000
> +#define XCT_MACTX_I          0x0800000000000000
> +#define XCT_MACTX_O          0x0400000000000000
> +#define XCT_MACTX_E          0x0200000000000000
> +#define XCT_MACTX_VLAN_M     0x0180000000000000
> +#define XCT_MACTX_VLAN_NOP   0x0000000000000000
> +#define XCT_MACTX_VLAN_REMOVE        0x0080000000000000
> +#define XCT_MACTX_VLAN_INSERT   0x0100000000000000
> +#define XCT_MACTX_VLAN_REPLACE  0x0180000000000000
> +#define XCT_MACTX_CRC_M              0x0060000000000000
> +#define XCT_MACTX_CRC_NOP    0x0000000000000000
> +#define XCT_MACTX_CRC_INSERT 0x0020000000000000
> +#define XCT_MACTX_CRC_PAD    0x0040000000000000
> +#define XCT_MACTX_CRC_REPLACE        0x0060000000000000
> +#define XCT_MACTX_SS         0x0010000000000000
> +#define XCT_MACTX_LLEN_M     0x00007fff00000000
> +#define XCT_MACTX_LLEN_S     32ull
> +#define XCT_MACTX_LLEN(x)    ((((long)(x)) << XCT_MACTX_LLEN_S) & 
> XCT_MACTX_LLEN_M)
> +#define XCT_MACTX_IPH_M              0x00000000f8000000
> +#define XCT_MACTX_IPH_S              27ull
> +#define XCT_MACTX_IPH(x)     ((((long)(x)) << XCT_MACTX_IPH_S) & 
> XCT_MACTX_IPH_M)
> +#define XCT_MACTX_IPO_M              0x0000000007c00000
> +#define XCT_MACTX_IPO_S              22ull
> +#define XCT_MACTX_IPO(x)     ((((long)(x)) << XCT_MACTX_IPO_S) & 
> XCT_MACTX_IPO_M)
> +#define XCT_MACTX_CSUM_M     0x0000000000000060
> +#define XCT_MACTX_CSUM_NOP   0x0000000000000000
> +#define XCT_MACTX_CSUM_TCP   0x0000000000000040
> +#define XCT_MACTX_CSUM_UDP   0x0000000000000060
> +#define XCT_MACTX_V6         0x0000000000000010
> +#define XCT_MACTX_C          0x0000000000000004
> +#define XCT_MACTX_AL2                0x0000000000000002
> +             u64     macrx;

Mixing definitions and structure values gets confusing.

-- 
Stephen Hemminger <[EMAIL PROTECTED]>
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to