Linsys Contractor Amit S. Kale wrote:
=> +extern struct netxen_adapter *g_adapter;
+
+/*
+ * The basic unit of access when reading/writing control registers.
+ */
+
+typedef u32 netxen_crbword_t;    /* single word in CRB space */
+
+#define NETXEN_HW_H0_CH_HUB_ADR    0x05
+#define NETXEN_HW_H1_CH_HUB_ADR    0x0E
+#define NETXEN_HW_H2_CH_HUB_ADR    0x03
+#define NETXEN_HW_H3_CH_HUB_ADR    0x01
+#define NETXEN_HW_H4_CH_HUB_ADR    0x06
+#define NETXEN_HW_H5_CH_HUB_ADR    0x07
+#define NETXEN_HW_H6_CH_HUB_ADR    0x08
+
+/*  Hub 0 */
+#define NETXEN_HW_MN_CRB_AGT_ADR    0x15
+#define NETXEN_HW_MS_CRB_AGT_ADR    0x25
+
+/*  Hub 1 */
+#define NETXEN_HW_PS_CRB_AGT_ADR    0x73
+#define NETXEN_HW_SS_CRB_AGT_ADR    0x20
+#define NETXEN_HW_RPMX3_CRB_AGT_ADR    0x0b
+#define NETXEN_HW_QMS_CRB_AGT_ADR    0x00
+#define NETXEN_HW_SQGS0_CRB_AGT_ADR    0x01
+#define NETXEN_HW_SQGS1_CRB_AGT_ADR    0x02
+#define NETXEN_HW_SQGS2_CRB_AGT_ADR    0x03
+#define NETXEN_HW_SQGS3_CRB_AGT_ADR    0x04
+#define NETXEN_HW_C2C0_CRB_AGT_ADR    0x58
+#define NETXEN_HW_C2C1_CRB_AGT_ADR    0x59
+#define NETXEN_HW_C2C2_CRB_AGT_ADR    0x5a
+#define NETXEN_HW_RPMX2_CRB_AGT_ADR    0x0a
+#define NETXEN_HW_RPMX4_CRB_AGT_ADR    0x0c
+#define NETXEN_HW_RPMX7_CRB_AGT_ADR    0x0f
+#define NETXEN_HW_RPMX9_CRB_AGT_ADR    0x12
+#define NETXEN_HW_SMB_CRB_AGT_ADR    0x18


overall, enums are preferred over #define. It provides type info to the compiler, provides symbol info to debuggers and similar tools, and other benefits.

e.g.

enum {
        NETXEN_HW_RPMX9_CRB_AGT_ADR     = 0x12,
        NETXEN_HW_SMB_CRB_AGT_ADR       = 0x18,
};



+#define NETXEN_PCI_MAPSIZE    128
+#define NETXEN_PCI_DDR_NET    (unsigned long)0x00000000
+#define NETXEN_PCI_QDR_NET    (unsigned long)0x04000000
+#define NETXEN_PCI_DIRECT_CRB    (unsigned long)0x04400000
+#define NETXEN_PCI_CAMQM_MAX    (unsigned long)0x04ffffff
+#define NETXEN_PCI_OCM0        (unsigned long)0x05000000
+#define NETXEN_PCI_OCM0_MAX    (unsigned long)0x050fffff
+#define NETXEN_PCI_OCM1        (unsigned long)0x05100000
+#define NETXEN_PCI_OCM1_MAX    (unsigned long)0x051fffff
+#define NETXEN_PCI_CRBSPACE    (unsigned long)0x06000000

Use the 'UL' suffix, not a cast



+struct netxen_pcix_crb_window {
+    netxen_crbword_t rsvd1:25, addrbit:1,    /* bit 25 of CRB address */
+     rsvd2:6;
+};

see bitfields objections in another email

-
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