netxen: fix error codes in for tools access
authorDhananjay Phadke <dhananjay@netxen.com>
Fri, 16 Oct 2009 15:50:10 +0000 (15:50 +0000)
committerDavid S. Miller <davem@davemloft.net>
Sun, 18 Oct 2009 06:44:44 +0000 (23:44 -0700)
Use -EIO or -EINVAL as error codes, these can get passed up
to applications (tools).

Signed-off-by: Dhananjay Phadke <dhananjay@netxen.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/netxen/netxen_nic_hw.c
drivers/net/netxen/netxen_nic_init.c

index a63324613430c9d803404c975f8c36bdb5eb3e12..e43cbbd5bec182f8a72fa1c2983c2208565d9332 100644 (file)
@@ -332,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);
        }
 
@@ -1083,7 +1083,7 @@ 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
@@ -1096,7 +1096,7 @@ netxen_nic_pci_get_crb_addr_2M(struct netxen_adapter *adapter, ulong *off)
 
 
        if (*off >= NETXEN_CRB_MAX)
-               return -1;
+               return -EINVAL;
 
        if (*off >= NETXEN_PCI_CAMQM && (*off < NETXEN_PCI_CAMQM_2M_END)) {
                *off = (*off - NETXEN_PCI_CAMQM) + NETXEN_PCI_CAMQM_2M_BASE +
@@ -1105,7 +1105,7 @@ netxen_nic_pci_get_crb_addr_2M(struct netxen_adapter *adapter, ulong *off)
        }
 
        if (*off < NETXEN_PCI_CRBSPACE)
-               return -1;
+               return -EINVAL;
 
        *off -= NETXEN_PCI_CRBSPACE;
 
@@ -1220,25 +1220,26 @@ netxen_nic_hw_write_wx_2M(struct netxen_adapter *adapter, ulong off, u32 data)
 
        rv = netxen_nic_pci_get_crb_addr_2M(adapter, &off);
 
-       if (rv == -1) {
-               printk(KERN_ERR "%s: invalid offset: 0x%016lx\n",
-                               __func__, off);
-               dump_stack();
-               return -1;
+       if (rv == 0) {
+               writel(data, (void __iomem *)off);
+               return 0;
        }
 
-       if (rv == 1) {
+       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);
                crb_win_unlock(adapter);
                write_unlock_irqrestore(&adapter->ahw.crb_lock, flags);
-       } else
-               writel(data, (void __iomem *)off);
-
+               return 0;
+       }
 
-       return 0;
+       dev_err(&adapter->pdev->dev,
+                       "%s: invalid offset: 0x%016lx\n", __func__, off);
+       dump_stack();
+       return -EIO;
 }
 
 static u32
@@ -1250,24 +1251,24 @@ netxen_nic_hw_read_wx_2M(struct netxen_adapter *adapter, ulong off)
 
        rv = netxen_nic_pci_get_crb_addr_2M(adapter, &off);
 
-       if (rv == -1) {
-               printk(KERN_ERR "%s: invalid offset: 0x%016lx\n",
-                               __func__, off);
-               dump_stack();
-               return -1;
-       }
+       if (rv == 0)
+               return readl((void __iomem *)off);
 
-       if (rv == 1) {
+       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);
                crb_win_unlock(adapter);
                write_unlock_irqrestore(&adapter->ahw.crb_lock, flags);
-       } else
-               data = readl((void __iomem *)off);
+               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 */
index a5248447789b4a218d0f56563cbad1d1014aff3c..d8c4b70e35ba2c151f87fe734c9e332a6c6b1d9f 100644 (file)
@@ -832,6 +832,12 @@ void netxen_request_firmware(struct netxen_adapter *adapter)
                goto request_fw;
        }
 
+       if (NX_IS_REVISION_P3P(adapter->ahw.revision_id)) {
+               /* No file firmware for the time being */
+               fw_type = NX_FLASH_ROMIMAGE;
+               goto done;
+       }
+
        fw_type = netxen_p3_has_mn(adapter) ?
                NX_P3_MN_ROMIMAGE : NX_P3_CT_ROMIMAGE;