net: convert multiple drivers to use netdev_for_each_mc_addr, part5 V2
[safe/jmp/linux-2.6] / drivers / net / netxen / netxen_nic_hw.c
index 37f4766..a945591 100644 (file)
@@ -19,7 +19,7 @@
  * MA  02111-1307, USA.
  *
  * The full GNU General Public License is included in this distribution
- * in the file called LICENSE.
+ * in the file called "COPYING".
  *
  */
 
@@ -31,6 +31,7 @@
 #define MASK(n) ((1ULL<<(n))-1)
 #define MN_WIN(addr) (((addr & 0x1fc0000) >> 1) | ((addr >> 25) & 0x3ff))
 #define OCM_WIN(addr) (((addr & 0x1ff0000) >> 1) | ((addr >> 25) & 0x3ff))
+#define OCM_WIN_P3P(addr) (addr & 0xffc0000)
 #define MS_WIN(addr) (addr & 0x0ffc0000)
 
 #define GET_MEM_OFFS_2M(addr) (addr & MASK(18))
 #define CRB_HI(off)    ((crb_hub_agt[CRB_BLK(off)] << 20) | ((off) & 0xf0000))
 #define CRB_INDIRECT_2M        (0x1e0000UL)
 
+static void netxen_nic_io_write_128M(struct netxen_adapter *adapter,
+               void __iomem *addr, u32 data);
+static u32 netxen_nic_io_read_128M(struct netxen_adapter *adapter,
+               void __iomem *addr);
+
 #ifndef readq
 static inline u64 readq(void __iomem *addr)
 {
@@ -326,7 +332,7 @@ netxen_pcie_sem_lock(struct netxen_adapter *adapter, int sem, u32 id_reg)
                if (done == 1)
                        break;
                if (++timeout >= NETXEN_PCIE_SEM_TIMEOUT)
-                       return -1;
+                       return -EIO;
                msleep(1);
        }
 
@@ -339,8 +345,7 @@ netxen_pcie_sem_lock(struct netxen_adapter *adapter, int sem, u32 id_reg)
 void
 netxen_pcie_sem_unlock(struct netxen_adapter *adapter, int sem)
 {
-       int val;
-       val = NXRD32(adapter, NETXEN_PCIE_REG(PCIE_SEM_UNLOCK(sem)));
+       NXRD32(adapter, NETXEN_PCIE_REG(PCIE_SEM_UNLOCK(sem)));
 }
 
 int netxen_niu_xg_init_port(struct netxen_adapter *adapter, int port)
@@ -383,24 +388,51 @@ int netxen_niu_disable_xg_port(struct netxen_adapter *adapter)
 
 int netxen_p2_nic_set_promisc(struct netxen_adapter *adapter, u32 mode)
 {
-       __u32 reg;
+       u32 mac_cfg;
+       u32 cnt = 0;
+       __u32 reg = 0x0200;
        u32 port = adapter->physical_port;
+       u16 board_type = adapter->ahw.board_type;
 
        if (port > NETXEN_NIU_MAX_XG_PORTS)
                return -EINVAL;
 
-       reg = NXRD32(adapter, NETXEN_NIU_XGE_CONFIG_1 + (0x10000 * port));
-       if (mode == NETXEN_NIU_PROMISC_MODE)
-               reg = (reg | 0x2000UL);
-       else
-               reg = (reg & ~0x2000UL);
+       mac_cfg = NXRD32(adapter, NETXEN_NIU_XGE_CONFIG_0 + (0x10000 * port));
+       mac_cfg &= ~0x4;
+       NXWR32(adapter, NETXEN_NIU_XGE_CONFIG_0 + (0x10000 * port), mac_cfg);
 
-       if (mode == NETXEN_NIU_ALLMULTI_MODE)
-               reg = (reg | 0x1000UL);
-       else
-               reg = (reg & ~0x1000UL);
+       if ((board_type == NETXEN_BRDTYPE_P2_SB31_10G_IMEZ) ||
+                       (board_type == NETXEN_BRDTYPE_P2_SB31_10G_HMEZ))
+               reg = (0x20 << port);
+
+       NXWR32(adapter, NETXEN_NIU_FRAME_COUNT_SELECT, reg);
+
+       mdelay(10);
+
+       while (NXRD32(adapter, NETXEN_NIU_FRAME_COUNT) && ++cnt < 20)
+               mdelay(10);
+
+       if (cnt < 20) {
+
+               reg = NXRD32(adapter,
+                       NETXEN_NIU_XGE_CONFIG_1 + (0x10000 * port));
+
+               if (mode == NETXEN_NIU_PROMISC_MODE)
+                       reg = (reg | 0x2000UL);
+               else
+                       reg = (reg & ~0x2000UL);
+
+               if (mode == NETXEN_NIU_ALLMULTI_MODE)
+                       reg = (reg | 0x1000UL);
+               else
+                       reg = (reg & ~0x1000UL);
 
-       NXWR32(adapter, NETXEN_NIU_XGE_CONFIG_1 + (0x10000 * port), reg);
+               NXWR32(adapter,
+                       NETXEN_NIU_XGE_CONFIG_1 + (0x10000 * port), reg);
+       }
+
+       mac_cfg |= 0x4;
+       NXWR32(adapter, NETXEN_NIU_XGE_CONFIG_0 + (0x10000 * port), mac_cfg);
 
        return 0;
 }
@@ -436,7 +468,7 @@ netxen_nic_enable_mcast_filter(struct netxen_adapter *adapter)
 {
        u32     val = 0;
        u16 port = adapter->physical_port;
-       u8 *addr = adapter->netdev->dev_addr;
+       u8 *addr = adapter->mac_addr;
 
        if (adapter->mc_enabled)
                return 0;
@@ -465,7 +497,7 @@ netxen_nic_disable_mcast_filter(struct netxen_adapter *adapter)
 {
        u32     val = 0;
        u16 port = adapter->physical_port;
-       u8 *addr = adapter->netdev->dev_addr;
+       u8 *addr = adapter->mac_addr;
 
        if (!adapter->mc_enabled)
                return 0;
@@ -507,7 +539,7 @@ void netxen_p2_nic_set_multi(struct net_device *netdev)
        struct netxen_adapter *adapter = netdev_priv(netdev);
        struct dev_mc_list *mc_ptr;
        u8 null_addr[6];
-       int index = 0;
+       int i;
 
        memset(null_addr, 0, 6);
 
@@ -522,7 +554,7 @@ void netxen_p2_nic_set_multi(struct net_device *netdev)
                return;
        }
 
-       if (netdev->mc_count == 0) {
+       if (netdev_mc_empty(netdev)) {
                adapter->set_promisc(adapter,
                                NETXEN_NIU_NON_PROMISC_MODE);
                netxen_nic_disable_mcast_filter(adapter);
@@ -531,23 +563,20 @@ void netxen_p2_nic_set_multi(struct net_device *netdev)
 
        adapter->set_promisc(adapter, NETXEN_NIU_ALLMULTI_MODE);
        if (netdev->flags & IFF_ALLMULTI ||
-                       netdev->mc_count > adapter->max_mc_count) {
+                       netdev_mc_count(netdev) > adapter->max_mc_count) {
                netxen_nic_disable_mcast_filter(adapter);
                return;
        }
 
        netxen_nic_enable_mcast_filter(adapter);
 
-       for (mc_ptr = netdev->mc_list; mc_ptr; mc_ptr = mc_ptr->next, index++)
-               netxen_nic_set_mcast_addr(adapter, index, mc_ptr->dmi_addr);
-
-       if (index != netdev->mc_count)
-               printk(KERN_WARNING "%s: %s multicast address count mismatch\n",
-                       netxen_nic_driver_name, netdev->name);
+       i = 0;
+       netdev_for_each_mc_addr(mc_ptr, netdev)
+               netxen_nic_set_mcast_addr(adapter, i++, mc_ptr->dmi_addr);
 
        /* Clear out remaining addresses */
-       for (; index < adapter->max_mc_count; index++)
-               netxen_nic_set_mcast_addr(adapter, index, null_addr);
+       while (i < adapter->max_mc_count)
+               netxen_nic_set_mcast_addr(adapter, i++, null_addr);
 }
 
 static int
@@ -658,9 +687,12 @@ void netxen_p3_nic_set_multi(struct net_device *netdev)
        struct list_head *head;
        nx_mac_list_t *cur;
 
+       if (adapter->is_up != NETXEN_ADAPTER_UP_MAGIC)
+               return;
+
        list_splice_tail_init(&adapter->mac_list, &del_list);
 
-       nx_p3_nic_add_mac(adapter, netdev->dev_addr, &del_list);
+       nx_p3_nic_add_mac(adapter, adapter->mac_addr, &del_list);
        nx_p3_nic_add_mac(adapter, bcast_addr, &del_list);
 
        if (netdev->flags & IFF_PROMISC) {
@@ -669,16 +701,14 @@ void netxen_p3_nic_set_multi(struct net_device *netdev)
        }
 
        if ((netdev->flags & IFF_ALLMULTI) ||
-                       (netdev->mc_count > adapter->max_mc_count)) {
+                       (netdev_mc_count(netdev) > adapter->max_mc_count)) {
                mode = VPORT_MISS_MODE_ACCEPT_MULTI;
                goto send_fw_cmd;
        }
 
-       if (netdev->mc_count > 0) {
-               for (mc_ptr = netdev->mc_list; mc_ptr;
-                    mc_ptr = mc_ptr->next) {
+       if (!netdev_mc_empty(netdev)) {
+               netdev_for_each_mc_addr(mc_ptr, netdev)
                        nx_p3_nic_add_mac(adapter, mc_ptr->dmi_addr, &del_list);
-               }
        }
 
 send_fw_cmd:
@@ -742,17 +772,20 @@ int netxen_p3_nic_set_mac_addr(struct netxen_adapter *adapter, u8 *addr)
 int netxen_config_intr_coalesce(struct netxen_adapter *adapter)
 {
        nx_nic_req_t req;
-       u64 word;
-       int rv;
+       u64 word[6];
+       int rv, i;
 
        memset(&req, 0, sizeof(nx_nic_req_t));
+       memset(word, 0, sizeof(word));
 
        req.qhdr = cpu_to_le64(NX_HOST_REQUEST << 23);
 
-       word = NETXEN_CONFIG_INTR_COALESCE | ((u64)adapter->portnum << 16);
-       req.req_hdr = cpu_to_le64(word);
+       word[0] = NETXEN_CONFIG_INTR_COALESCE | ((u64)adapter->portnum << 16);
+       req.req_hdr = cpu_to_le64(word[0]);
 
-       memcpy(&req.words[0], &adapter->coal, sizeof(adapter->coal));
+       memcpy(&word[0], &adapter->coal, sizeof(adapter->coal));
+       for (i = 0; i < 6; i++)
+               req.words[i] = cpu_to_le64(word[i]);
 
        rv = netxen_send_cmd_descs(adapter, (struct cmd_desc_type0 *)&req, 1);
        if (rv != 0) {
@@ -998,7 +1031,7 @@ static int netxen_get_flash_block(struct netxen_adapter *adapter, int base,
        return 0;
 }
 
-int netxen_get_flash_mac_addr(struct netxen_adapter *adapter, __le64 *mac)
+int netxen_get_flash_mac_addr(struct netxen_adapter *adapter, u64 *mac)
 {
        __le32 *pmac = (__le32 *) mac;
        u32 offset;
@@ -1023,7 +1056,7 @@ int netxen_get_flash_mac_addr(struct netxen_adapter *adapter, __le64 *mac)
        return 0;
 }
 
-int netxen_p3_get_mac_addr(struct netxen_adapter *adapter, __le64 *mac)
+int netxen_p3_get_mac_addr(struct netxen_adapter *adapter, u64 *mac)
 {
        uint32_t crbaddr, mac_hi, mac_lo;
        int pci_func = adapter->ahw.pci_func;
@@ -1077,46 +1110,40 @@ netxen_nic_pci_set_crbwindow_128M(struct netxen_adapter *adapter,
 }
 
 /*
- * Return -1 if off is not valid,
+ * Returns < 0 if off is not valid,
  *      1 if window access is needed. 'off' is set to offset from
  *        CRB space in 128M pci map
  *      0 if no window access is needed. 'off' is set to 2M addr
  * In: 'off' is offset from base in 128M pci map
  */
 static int
-netxen_nic_pci_get_crb_addr_2M(struct netxen_adapter *adapter, ulong *off)
+netxen_nic_pci_get_crb_addr_2M(struct netxen_adapter *adapter,
+               ulong off, void __iomem **addr)
 {
        crb_128M_2M_sub_block_map_t *m;
 
 
-       if (*off >= NETXEN_CRB_MAX)
-               return -1;
-
-       if (*off >= NETXEN_PCI_CAMQM && (*off < NETXEN_PCI_CAMQM_2M_END)) {
-               *off = (*off - NETXEN_PCI_CAMQM) + NETXEN_PCI_CAMQM_2M_BASE +
-                       (ulong)adapter->ahw.pci_base0;
-               return 0;
-       }
-
-       if (*off < NETXEN_PCI_CRBSPACE)
-               return -1;
+       if ((off >= NETXEN_CRB_MAX) || (off < NETXEN_PCI_CRBSPACE))
+               return -EINVAL;
 
-       *off -= NETXEN_PCI_CRBSPACE;
+       off -= NETXEN_PCI_CRBSPACE;
 
        /*
         * Try direct map
         */
-       m = &crb_128M_2M_map[CRB_BLK(*off)].sub_block[CRB_SUBBLK(*off)];
+       m = &crb_128M_2M_map[CRB_BLK(off)].sub_block[CRB_SUBBLK(off)];
 
-       if (m->valid && (m->start_128M <= *off) && (m->end_128M > *off)) {
-               *off = *off + m->start_2M - m->start_128M +
-                       (ulong)adapter->ahw.pci_base0;
+       if (m->valid && (m->start_128M <= off) && (m->end_128M > off)) {
+               *addr = adapter->ahw.pci_base0 + m->start_2M +
+                       (off - m->start_128M);
                return 0;
        }
 
        /*
         * Not in direct map, use crb window
         */
+       *addr = adapter->ahw.pci_base0 + CRB_INDIRECT_2M +
+               (off & MASK(16));
        return 1;
 }
 
@@ -1126,57 +1153,78 @@ netxen_nic_pci_get_crb_addr_2M(struct netxen_adapter *adapter, ulong *off)
  * side effect: lock crb window
  */
 static void
-netxen_nic_pci_set_crbwindow_2M(struct netxen_adapter *adapter, ulong *off)
+netxen_nic_pci_set_crbwindow_2M(struct netxen_adapter *adapter, ulong off)
 {
        u32 window;
        void __iomem *addr = adapter->ahw.pci_base0 + CRB_WINDOW_2M;
 
-       window = CRB_HI(*off);
+       off -= NETXEN_PCI_CRBSPACE;
+
+       window = CRB_HI(off);
 
        if (adapter->ahw.crb_win == window)
-               goto done;
+               return;
 
        writel(window, addr);
        if (readl(addr) != window) {
                if (printk_ratelimit())
                        dev_warn(&adapter->pdev->dev,
                                "failed to set CRB window to %d off 0x%lx\n",
-                               window, *off);
+                               window, off);
        }
        adapter->ahw.crb_win = window;
+}
 
-done:
-       *off = (*off & MASK(16)) + CRB_INDIRECT_2M +
-               (ulong)adapter->ahw.pci_base0;
+static void __iomem *
+netxen_nic_map_indirect_address_128M(struct netxen_adapter *adapter,
+               ulong win_off, void __iomem **mem_ptr)
+{
+       ulong off = win_off;
+       void __iomem *addr;
+       resource_size_t mem_base;
+
+       if (ADDR_IN_WINDOW1(win_off))
+               off = NETXEN_CRB_NORMAL(win_off);
+
+       addr = pci_base_offset(adapter, off);
+       if (addr)
+               return addr;
+
+       if (adapter->ahw.pci_len0 == 0)
+               off -= NETXEN_PCI_CRBSPACE;
+
+       mem_base = pci_resource_start(adapter->pdev, 0);
+       *mem_ptr = ioremap(mem_base + (off & PAGE_MASK), PAGE_SIZE);
+       if (*mem_ptr)
+               addr = *mem_ptr + (off & (PAGE_SIZE - 1));
+
+       return addr;
 }
 
 static int
 netxen_nic_hw_write_wx_128M(struct netxen_adapter *adapter, ulong off, u32 data)
 {
        unsigned long flags;
-       void __iomem *addr;
-
-       if (ADDR_IN_WINDOW1(off))
-               addr = NETXEN_CRB_NORMALIZE(adapter, off);
-       else
-               addr = pci_base_offset(adapter, off);
+       void __iomem *addr, *mem_ptr = NULL;
 
-       BUG_ON(!addr);
+       addr = netxen_nic_map_indirect_address_128M(adapter, off, &mem_ptr);
+       if (!addr)
+               return -EIO;
 
-       if (ADDR_IN_WINDOW1(off)) {     /* Window 1 */
-               read_lock(&adapter->adapter_lock);
-               writel(data, addr);
-               read_unlock(&adapter->adapter_lock);
-       } else {                /* Window 0 */
-               write_lock_irqsave(&adapter->adapter_lock, flags);
-               addr = pci_base_offset(adapter, off);
+       if (ADDR_IN_WINDOW1(off)) { /* Window 1 */
+               netxen_nic_io_write_128M(adapter, addr, data);
+       } else {        /* Window 0 */
+               write_lock_irqsave(&adapter->ahw.crb_lock, flags);
                netxen_nic_pci_set_crbwindow_128M(adapter, 0);
                writel(data, addr);
                netxen_nic_pci_set_crbwindow_128M(adapter,
                                NETXEN_WINDOW_ONE);
-               write_unlock_irqrestore(&adapter->adapter_lock, flags);
+               write_unlock_irqrestore(&adapter->ahw.crb_lock, flags);
        }
 
+       if (mem_ptr)
+               iounmap(mem_ptr);
+
        return 0;
 }
 
@@ -1184,29 +1232,27 @@ static u32
 netxen_nic_hw_read_wx_128M(struct netxen_adapter *adapter, ulong off)
 {
        unsigned long flags;
-       void __iomem *addr;
+       void __iomem *addr, *mem_ptr = NULL;
        u32 data;
 
-       if (ADDR_IN_WINDOW1(off))
-               addr = NETXEN_CRB_NORMALIZE(adapter, off);
-       else
-               addr = pci_base_offset(adapter, off);
-
-       BUG_ON(!addr);
+       addr = netxen_nic_map_indirect_address_128M(adapter, off, &mem_ptr);
+       if (!addr)
+               return -EIO;
 
-       if (ADDR_IN_WINDOW1(off)) {     /* Window 1 */
-               read_lock(&adapter->adapter_lock);
-               data = readl(addr);
-               read_unlock(&adapter->adapter_lock);
-       } else {                /* Window 0 */
-               write_lock_irqsave(&adapter->adapter_lock, flags);
+       if (ADDR_IN_WINDOW1(off)) { /* Window 1 */
+               data = netxen_nic_io_read_128M(adapter, addr);
+       } else {        /* Window 0 */
+               write_lock_irqsave(&adapter->ahw.crb_lock, flags);
                netxen_nic_pci_set_crbwindow_128M(adapter, 0);
                data = readl(addr);
                netxen_nic_pci_set_crbwindow_128M(adapter,
                                NETXEN_WINDOW_ONE);
-               write_unlock_irqrestore(&adapter->adapter_lock, flags);
+               write_unlock_irqrestore(&adapter->ahw.crb_lock, flags);
        }
 
+       if (mem_ptr)
+               iounmap(mem_ptr);
+
        return data;
 }
 
@@ -1215,28 +1261,30 @@ netxen_nic_hw_write_wx_2M(struct netxen_adapter *adapter, ulong off, u32 data)
 {
        unsigned long flags;
        int rv;
+       void __iomem *addr = NULL;
 
-       rv = netxen_nic_pci_get_crb_addr_2M(adapter, &off);
+       rv = netxen_nic_pci_get_crb_addr_2M(adapter, off, &addr);
 
-       if (rv == -1) {
-               printk(KERN_ERR "%s: invalid offset: 0x%016lx\n",
-                               __func__, off);
-               dump_stack();
-               return -1;
+       if (rv == 0) {
+               writel(data, addr);
+               return 0;
        }
 
-       if (rv == 1) {
-               write_lock_irqsave(&adapter->adapter_lock, flags);
+       if (rv > 0) {
+               /* indirect access */
+               write_lock_irqsave(&adapter->ahw.crb_lock, flags);
                crb_win_lock(adapter);
-               netxen_nic_pci_set_crbwindow_2M(adapter, &off);
-               writel(data, (void __iomem *)off);
+               netxen_nic_pci_set_crbwindow_2M(adapter, off);
+               writel(data, addr);
                crb_win_unlock(adapter);
-               write_unlock_irqrestore(&adapter->adapter_lock, flags);
-       } else
-               writel(data, (void __iomem *)off);
-
+               write_unlock_irqrestore(&adapter->ahw.crb_lock, flags);
+               return 0;
+       }
 
-       return 0;
+       dev_err(&adapter->pdev->dev,
+                       "%s: invalid offset: 0x%016lx\n", __func__, off);
+       dump_stack();
+       return -EIO;
 }
 
 static u32
@@ -1245,36 +1293,37 @@ netxen_nic_hw_read_wx_2M(struct netxen_adapter *adapter, ulong off)
        unsigned long flags;
        int rv;
        u32 data;
+       void __iomem *addr = NULL;
 
-       rv = netxen_nic_pci_get_crb_addr_2M(adapter, &off);
+       rv = netxen_nic_pci_get_crb_addr_2M(adapter, off, &addr);
 
-       if (rv == -1) {
-               printk(KERN_ERR "%s: invalid offset: 0x%016lx\n",
-                               __func__, off);
-               dump_stack();
-               return -1;
-       }
+       if (rv == 0)
+               return readl(addr);
 
-       if (rv == 1) {
-               write_lock_irqsave(&adapter->adapter_lock, flags);
+       if (rv > 0) {
+               /* indirect access */
+               write_lock_irqsave(&adapter->ahw.crb_lock, flags);
                crb_win_lock(adapter);
-               netxen_nic_pci_set_crbwindow_2M(adapter, &off);
-               data = readl((void __iomem *)off);
+               netxen_nic_pci_set_crbwindow_2M(adapter, off);
+               data = readl(addr);
                crb_win_unlock(adapter);
-               write_unlock_irqrestore(&adapter->adapter_lock, flags);
-       } else
-               data = readl((void __iomem *)off);
+               write_unlock_irqrestore(&adapter->ahw.crb_lock, flags);
+               return data;
+       }
 
-       return data;
+       dev_err(&adapter->pdev->dev,
+                       "%s: invalid offset: 0x%016lx\n", __func__, off);
+       dump_stack();
+       return -1;
 }
 
 /* window 1 registers only */
 static void netxen_nic_io_write_128M(struct netxen_adapter *adapter,
                void __iomem *addr, u32 data)
 {
-       read_lock(&adapter->adapter_lock);
+       read_lock(&adapter->ahw.crb_lock);
        writel(data, addr);
-       read_unlock(&adapter->adapter_lock);
+       read_unlock(&adapter->ahw.crb_lock);
 }
 
 static u32 netxen_nic_io_read_128M(struct netxen_adapter *adapter,
@@ -1282,9 +1331,9 @@ static u32 netxen_nic_io_read_128M(struct netxen_adapter *adapter,
 {
        u32 val;
 
-       read_lock(&adapter->adapter_lock);
+       read_lock(&adapter->ahw.crb_lock);
        val = readl(addr);
-       read_unlock(&adapter->adapter_lock);
+       read_unlock(&adapter->ahw.crb_lock);
 
        return val;
 }
@@ -1304,17 +1353,20 @@ static u32 netxen_nic_io_read_2M(struct netxen_adapter *adapter,
 void __iomem *
 netxen_get_ioaddr(struct netxen_adapter *adapter, u32 offset)
 {
-       ulong off = offset;
+       void __iomem *addr = NULL;
 
        if (NX_IS_REVISION_P2(adapter->ahw.revision_id)) {
-               if (offset < NETXEN_CRB_PCIX_HOST2 &&
-                               offset > NETXEN_CRB_PCIX_HOST)
-                       return PCI_OFFSET_SECOND_RANGE(adapter, offset);
-               return NETXEN_CRB_NORMALIZE(adapter, offset);
+               if ((offset < NETXEN_CRB_PCIX_HOST2) &&
+                               (offset > NETXEN_CRB_PCIX_HOST))
+                       addr = PCI_OFFSET_SECOND_RANGE(adapter, offset);
+               else
+                       addr = NETXEN_CRB_NORMALIZE(adapter, offset);
+       } else {
+               WARN_ON(netxen_nic_pci_get_crb_addr_2M(adapter,
+                                       offset, &addr));
        }
 
-       BUG_ON(netxen_nic_pci_get_crb_addr_2M(adapter, &off));
-       return (void __iomem *)off;
+       return addr;
 }
 
 static int
@@ -1337,7 +1389,7 @@ static int
 netxen_nic_pci_set_window_2M(struct netxen_adapter *adapter,
                u64 addr, u32 *start)
 {
-       u32 win_read, window;
+       u32 window;
        struct pci_dev *pdev = adapter->pdev;
 
        if ((addr & 0x00ff800) == 0xff800) {
@@ -1346,14 +1398,14 @@ netxen_nic_pci_set_window_2M(struct netxen_adapter *adapter,
                return -EIO;
        }
 
-       window = OCM_WIN(addr);
+       if (NX_IS_REVISION_P3P(adapter->ahw.revision_id))
+               window = OCM_WIN_P3P(addr);
+       else
+               window = OCM_WIN(addr);
+
        writel(window, adapter->ahw.ocm_win_crb);
-       win_read = readl(adapter->ahw.ocm_win_crb);
-       if ((win_read >> 7) != window) {
-               if (printk_ratelimit())
-                       dev_warn(&pdev->dev, "failed to set OCM window\n");
-               return -EIO;
-       }
+       /* read back to flush */
+       readl(adapter->ahw.ocm_win_crb);
 
        adapter->ahw.ocm_win = window;
        *start = NETXEN_PCI_OCM0_2M + GET_MEM_OFFS_2M(addr);
@@ -1366,11 +1418,10 @@ netxen_nic_pci_mem_access_direct(struct netxen_adapter *adapter, u64 off,
 {
        void __iomem *addr, *mem_ptr = NULL;
        resource_size_t mem_base;
-       unsigned long flags;
        int ret = -EIO;
        u32 start;
 
-       write_lock_irqsave(&adapter->adapter_lock, flags);
+       spin_lock(&adapter->ahw.mem_lock);
 
        ret = adapter->pci_set_window(adapter, off, &start);
        if (ret != 0)
@@ -1397,7 +1448,8 @@ noremap:
                writeq(*data, addr);
 
 unlock:
-       write_unlock_irqrestore(&adapter->adapter_lock, flags);
+       spin_unlock(&adapter->ahw.mem_lock);
+
        if (mem_ptr)
                iounmap(mem_ptr);
        return ret;
@@ -1409,7 +1461,6 @@ static int
 netxen_nic_pci_mem_write_128M(struct netxen_adapter *adapter,
                u64 off, u64 data)
 {
-       unsigned long   flags;
        int j, ret;
        u32 temp, off_lo, off_hi, addr_hi, data_hi, data_lo;
        void __iomem *mem_crb;
@@ -1453,7 +1504,7 @@ netxen_nic_pci_mem_write_128M(struct netxen_adapter *adapter,
        return -EIO;
 
 correct:
-       write_lock_irqsave(&adapter->adapter_lock, flags);
+       spin_lock(&adapter->ahw.mem_lock);
        netxen_nic_pci_set_crbwindow_128M(adapter, 0);
 
        writel(off_lo, (mem_crb + MIU_TEST_AGT_ADDR_LO));
@@ -1479,7 +1530,7 @@ correct:
                ret = 0;
 
        netxen_nic_pci_set_crbwindow_128M(adapter, NETXEN_WINDOW_ONE);
-       write_unlock_irqrestore(&adapter->adapter_lock, flags);
+       spin_unlock(&adapter->ahw.mem_lock);
        return ret;
 }
 
@@ -1487,7 +1538,6 @@ static int
 netxen_nic_pci_mem_read_128M(struct netxen_adapter *adapter,
                u64 off, u64 *data)
 {
-       unsigned long   flags;
        int j, ret;
        u32 temp, off_lo, off_hi, addr_hi, data_hi, data_lo;
        u64 val;
@@ -1532,7 +1582,7 @@ netxen_nic_pci_mem_read_128M(struct netxen_adapter *adapter,
        return -EIO;
 
 correct:
-       write_lock_irqsave(&adapter->adapter_lock, flags);
+       spin_lock(&adapter->ahw.mem_lock);
        netxen_nic_pci_set_crbwindow_128M(adapter, 0);
 
        writel(off_lo, (mem_crb + MIU_TEST_AGT_ADDR_LO));
@@ -1561,7 +1611,7 @@ correct:
        }
 
        netxen_nic_pci_set_crbwindow_128M(adapter, NETXEN_WINDOW_ONE);
-       write_unlock_irqrestore(&adapter->adapter_lock, flags);
+       spin_unlock(&adapter->ahw.mem_lock);
 
        return ret;
 }
@@ -1570,9 +1620,9 @@ static int
 netxen_nic_pci_mem_write_2M(struct netxen_adapter *adapter,
                u64 off, u64 data)
 {
-       unsigned long   flags;
-       int j, ret;
+       int i, j, ret;
        u32 temp, off8;
+       u64 stride;
        void __iomem *mem_crb;
 
        /* Only 64-bit aligned access */
@@ -1599,14 +1649,45 @@ netxen_nic_pci_mem_write_2M(struct netxen_adapter *adapter,
        return -EIO;
 
 correct:
-       off8 = off & MIU_TEST_AGT_ADDR_MASK;
+       stride = NX_IS_REVISION_P3P(adapter->ahw.revision_id) ? 16 : 8;
 
-       write_lock_irqsave(&adapter->adapter_lock, flags);
+       off8 = off & ~(stride-1);
+
+       spin_lock(&adapter->ahw.mem_lock);
 
        writel(off8, (mem_crb + MIU_TEST_AGT_ADDR_LO));
        writel(0, (mem_crb + MIU_TEST_AGT_ADDR_HI));
-       writel(data & 0xffffffff, mem_crb + MIU_TEST_AGT_WRDATA_LO);
-       writel((data >> 32) & 0xffffffff, mem_crb + MIU_TEST_AGT_WRDATA_HI);
+
+       i = 0;
+       if (stride == 16) {
+               writel(TA_CTL_ENABLE, (mem_crb + TEST_AGT_CTRL));
+               writel((TA_CTL_START | TA_CTL_ENABLE),
+                               (mem_crb + TEST_AGT_CTRL));
+
+               for (j = 0; j < MAX_CTL_CHECK; j++) {
+                       temp = readl(mem_crb + TEST_AGT_CTRL);
+                       if ((temp & TA_CTL_BUSY) == 0)
+                               break;
+               }
+
+               if (j >= MAX_CTL_CHECK) {
+                       ret = -EIO;
+                       goto done;
+               }
+
+               i = (off & 0xf) ? 0 : 2;
+               writel(readl(mem_crb + MIU_TEST_AGT_RDDATA(i)),
+                               mem_crb + MIU_TEST_AGT_WRDATA(i));
+               writel(readl(mem_crb + MIU_TEST_AGT_RDDATA(i+1)),
+                               mem_crb + MIU_TEST_AGT_WRDATA(i+1));
+               i = (off & 0xf) ? 2 : 0;
+       }
+
+       writel(data & 0xffffffff,
+                       mem_crb + MIU_TEST_AGT_WRDATA(i));
+       writel((data >> 32) & 0xffffffff,
+                       mem_crb + MIU_TEST_AGT_WRDATA(i+1));
+
        writel((TA_CTL_ENABLE | TA_CTL_WRITE), (mem_crb + TEST_AGT_CTRL));
        writel((TA_CTL_START | TA_CTL_ENABLE | TA_CTL_WRITE),
                        (mem_crb + TEST_AGT_CTRL));
@@ -1625,7 +1706,8 @@ correct:
        } else
                ret = 0;
 
-       write_unlock_irqrestore(&adapter->adapter_lock, flags);
+done:
+       spin_unlock(&adapter->ahw.mem_lock);
 
        return ret;
 }
@@ -1634,10 +1716,9 @@ static int
 netxen_nic_pci_mem_read_2M(struct netxen_adapter *adapter,
                u64 off, u64 *data)
 {
-       unsigned long   flags;
        int j, ret;
        u32 temp, off8;
-       u64 val;
+       u64 val, stride;
        void __iomem *mem_crb;
 
        /* Only 64-bit aligned access */
@@ -1666,9 +1747,11 @@ netxen_nic_pci_mem_read_2M(struct netxen_adapter *adapter,
        return -EIO;
 
 correct:
-       off8 = off & MIU_TEST_AGT_ADDR_MASK;
+       stride = NX_IS_REVISION_P3P(adapter->ahw.revision_id) ? 16 : 8;
+
+       off8 = off & ~(stride-1);
 
-       write_lock_irqsave(&adapter->adapter_lock, flags);
+       spin_lock(&adapter->ahw.mem_lock);
 
        writel(off8, (mem_crb + MIU_TEST_AGT_ADDR_LO));
        writel(0, (mem_crb + MIU_TEST_AGT_ADDR_HI));
@@ -1687,14 +1770,18 @@ correct:
                                        "failed to read through agent\n");
                ret = -EIO;
        } else {
-               temp = readl(mem_crb + MIU_TEST_AGT_RDDATA_HI);
+               off8 = MIU_TEST_AGT_RDDATA_LO;
+               if ((stride == 16) && (off & 0xf))
+                       off8 = MIU_TEST_AGT_RDDATA_UPPER_LO;
+
+               temp = readl(mem_crb + off8 + 4);
                val = (u64)temp << 32;
-               val |= readl(mem_crb + MIU_TEST_AGT_RDDATA_LO);
+               val |= readl(mem_crb + off8);
                *data = val;
                ret = 0;
        }
 
-       write_unlock_irqrestore(&adapter->adapter_lock, flags);
+       spin_unlock(&adapter->ahw.mem_lock);
 
        return ret;
 }
@@ -1740,22 +1827,16 @@ netxen_setup_hwops(struct netxen_adapter *adapter)
 
 int netxen_nic_get_board_info(struct netxen_adapter *adapter)
 {
-       int offset, board_type, magic, header_version;
+       int offset, board_type, magic;
        struct pci_dev *pdev = adapter->pdev;
 
        offset = NX_FW_MAGIC_OFFSET;
        if (netxen_rom_fast_read(adapter, offset, &magic))
                return -EIO;
 
-       offset = NX_HDR_VERSION_OFFSET;
-       if (netxen_rom_fast_read(adapter, offset, &header_version))
-               return -EIO;
-
-       if (magic != NETXEN_BDINFO_MAGIC ||
-                       header_version != NETXEN_BDINFO_VERSION) {
-               dev_err(&pdev->dev,
-                       "invalid board config, magic=%08x, version=%08x\n",
-                       magic, header_version);
+       if (magic != NETXEN_BDINFO_MAGIC) {
+               dev_err(&pdev->dev, "invalid board config, magic=%08x\n",
+                       magic);
                return -EIO;
        }
 
@@ -1855,10 +1936,10 @@ void netxen_nic_set_link_parameters(struct netxen_adapter *adapter)
                        return;
                }
 
-               if (adapter->phy_read
-                   && adapter->phy_read(adapter,
-                            NETXEN_NIU_GB_MII_MGMT_ADDR_PHY_STATUS,
-                            &status) == 0) {
+               if (adapter->phy_read &&
+                   adapter->phy_read(adapter,
+                                     NETXEN_NIU_GB_MII_MGMT_ADDR_PHY_STATUS,
+                                     &status) == 0) {
                        if (netxen_get_phy_link(status)) {
                                switch (netxen_get_phy_speed(status)) {
                                case 0:
@@ -1885,10 +1966,10 @@ void netxen_nic_set_link_parameters(struct netxen_adapter *adapter)
                                        adapter->link_duplex = -1;
                                        break;
                                }
-                               if (adapter->phy_read
-                                   && adapter->phy_read(adapter,
-                                            NETXEN_NIU_GB_MII_MGMT_ADDR_AUTONEG,
-                                            &autoneg) != 0)
+                               if (adapter->phy_read &&
+                                   adapter->phy_read(adapter,
+                                                     NETXEN_NIU_GB_MII_MGMT_ADDR_AUTONEG,
+                                                     &autoneg) != 0)
                                        adapter->link_autoneg = autoneg;
                        } else
                                goto link_down;