On Fri, 15 Sep 2006 11:51:46 -0700 (PDT) Amit S. Kale wrote:

> diff -Narup -X linux-2.6.18-rc7.orig/Documentation/dontdiff 
> linux-2.6.18-rc7.orig/drivers/net/netxen/netxen_nic.h 
> linux-2.6.18-rc7/drivers/net/netxen/netxen_nic.h
> --- linux-2.6.18-rc7.orig/drivers/net/netxen/netxen_nic.h     1969-12-31 
> 16:00:00.000000000 -0800
> +++ linux-2.6.18-rc7/drivers/net/netxen/netxen_nic.h  2006-09-15 
> 10:51:59.000000000 -0700

> +#include <linux/config.h>
> +#include <linux/module.h>
> +#include <linux/kernel.h>
> +#include <linux/types.h>
> +#include <linux/compiler.h>
> +#include <linux/slab.h>
> +#include <linux/delay.h>
> +#include <linux/init.h>
> +#include <linux/ioport.h>
> +#include <linux/pci.h>
> +#include <linux/netdevice.h>
> +#include <linux/etherdevice.h>
> +#include <linux/ip.h>
> +#include <linux/in.h>
> +#include <linux/tcp.h>
> +#include <linux/skbuff.h>
> +#include <linux/version.h>
> +
> +#include <linux/ethtool.h>
> +#include <linux/mii.h>
> +#include <linux/interrupt.h>
> +#include <linux/timer.h>
> +
> +#include <linux/mm.h>
> +#include <linux/mman.h>
> +
> +#include <asm/system.h>
> +#include <asm/io.h>
> +#include <asm/byteorder.h>
> +#include <asm/uaccess.h>
> +#include <asm/pgtable.h>
> +
> +#include <linux/skbuff.h>

This one is already #included above.

> +#include "netxen_nic_hw.h"
> +
> +#define NETXEN_NIC_BUILD_NO     "232"
> +#define _NETXEN_NIC_LINUX_MAJOR 2
> +#define _NETXEN_NIC_LINUX_MINOR 3
> +#define _NETXEN_NIC_LINUX_SUBVERSION 57
> +#define NETXEN_NIC_LINUX_VERSIONID  "2.3.57"
> +#define NETXEN_NIC_FW_VERSIONID "2.3.57"

> +#define MAX_CMD_DESCRIPTORS          1024
> +#define MAX_RCV_DESCRIPTORS          32768
> +#define MAX_JUMBO_RCV_DESCRIPTORS    1024
> +#define MAX_RCVSTATUS_DESCRIPTORS    MAX_RCV_DESCRIPTORS
> +#define MAX_JUMBO_RCV_DESC   MAX_JUMBO_RCV_DESCRIPTORS
> +#define MAX_RCV_DESC         MAX_RCV_DESCRIPTORS
> +#define MAX_RCVSTATUS_DESC   MAX_RCV_DESCRIPTORS
> +#define NUM_RCV_DESC         (MAX_RCV_DESC + MAX_JUMBO_RCV_DESCRIPTORS)
> +#define MAX_EPG_DESCRIPTORS  MAX_CMD_DESCRIPTORS * 8

Parentheses around the expression above, please.

> +/**
> + * Following data structures describe the descriptors that will be used.
> + * Added fileds of tcpHdrSize and ipHdrSize, The driver needs to do it only 
> when
> + * we are doing LSO (above the 1500 size packet) only.
> + **/

Please don't use "/**" for any comments other than kernel-doc
formatted comments.

> +/**
> + * The size of reference handle been changed to 16 bits to pass the MSS 
> fields
> + * for the LSO packet
> + **/

Drop the "_t" on struct names (here and elsewhere).

> +struct cmd_desc_type0_t {
> +     u64 netxen_next;        /* for fragments handled by Phantom */
> +     union {
> +             struct {
> +                     u32 addr_low_part2;
> +                     u32 addr_high_part2;
> +             };
> +             u64 addr_buffer2;
> +     };

> +/* this structure by all ports on the adapter */

eh?

---
~Randy
-
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