Patchwork [1/1] be2net: add two new pci device ids to pci device table

login
register
mail settings
Submitter Ajit Khaparde
Date May 18, 2009, 10:33 p.m.
Message ID <20090518223344.GA25192@serverengines.com>
Download mbox | patch
Permalink /patch/27379/
State Accepted
Delegated to: David Miller
Headers show

Comments

Ajit Khaparde - May 18, 2009, 10:33 p.m.
David, thanks for the reply. There is a slight sense of urgency to see this 
patch in the upstream kernel. Hence I am sending this patch which I have
spun again - against the net-2.6 tree. Please accept. I hope it applies this
time.

Thanks
Ajit

Signed-off-by: Ajit Khaparde <ajitk@serverengines.com>
---
 drivers/net/benet/be.h      |   14 ++++++++++++++
 drivers/net/benet/be_main.c |   10 +++++-----
 2 files changed, 19 insertions(+), 5 deletions(-)
David Miller - May 18, 2009, 10:38 p.m.
From: Ajit Khaparde <ajitk@serverengines.com>
Date: Tue, 19 May 2009 04:03:46 +0530

> David, thanks for the reply. There is a slight sense of urgency to see this 
> patch in the upstream kernel. Hence I am sending this patch which I have
> spun again - against the net-2.6 tree. Please accept. I hope it applies this
> time.

I'll apply this to net-2.6 sure, but I wasn't saying this
patch didn't apply correctly.

I was talking about the patch that moved the NAPI stuff out the TX
work stuff in this driver.  That's the patch that wouldn't apply
correctly.

This one:

	http://patchwork.ozlabs.org/patch/27312/
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Ajit Khaparde - May 19, 2009, 1:22 a.m.
Thanks David. I will check the other patch.

On 18/05/09 15:38 -0700, David Miller wrote:
> From: Ajit Khaparde <ajitk@serverengines.com>
> Date: Tue, 19 May 2009 04:03:46 +0530
> 
> > David, thanks for the reply. There is a slight sense of urgency to see this 
> > patch in the upstream kernel. Hence I am sending this patch which I have
> > spun again - against the net-2.6 tree. Please accept. I hope it applies this
> > time.
> 
> I'll apply this to net-2.6 sure, but I wasn't saying this
> patch didn't apply correctly.
> 
> I was talking about the patch that moved the NAPI stuff out the TX
> work stuff in this driver.  That's the patch that wouldn't apply
> correctly.
> 
> This one:
> 
> 	http://patchwork.ozlabs.org/patch/27312/
> --
> To unsubscribe from this list: send the line "unsubscribe netdev" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Patch

diff --git a/drivers/net/benet/be.h b/drivers/net/benet/be.h
index c49ddd0..b4bb06f 100644
--- a/drivers/net/benet/be.h
+++ b/drivers/net/benet/be.h
@@ -35,8 +35,22 @@ 
 #define DRV_VER			"2.0.348"
 #define DRV_NAME		"be2net"
 #define BE_NAME			"ServerEngines BladeEngine2 10Gbps NIC"
+#define OC_NAME			"Emulex OneConnect 10Gbps NIC"
 #define DRV_DESC		BE_NAME "Driver"
 
+#define BE_VENDOR_ID 		0x19a2
+#define BE_DEVICE_ID1		0x211
+#define OC_DEVICE_ID1		0x700
+#define OC_DEVICE_ID2		0x701
+
+static inline char *nic_name(struct pci_dev *pdev)
+{
+	if (pdev->device == OC_DEVICE_ID1 || pdev->device == OC_DEVICE_ID2)
+		return OC_NAME;
+	else
+		return BE_NAME;
+}
+
 /* Number of bytes of an RX frame that are copied to skb->data */
 #define BE_HDR_LEN 		64
 #define BE_MAX_JUMBO_FRAME_SIZE	9018
diff --git a/drivers/net/benet/be_main.c b/drivers/net/benet/be_main.c
index 30d0c81..5c378b5 100644
--- a/drivers/net/benet/be_main.c
+++ b/drivers/net/benet/be_main.c
@@ -28,10 +28,10 @@  static unsigned int rx_frag_size = 2048;
 module_param(rx_frag_size, uint, S_IRUGO);
 MODULE_PARM_DESC(rx_frag_size, "Size of a fragment that holds rcvd data.");
 
-#define BE_VENDOR_ID 		0x19a2
-#define BE2_DEVICE_ID_1 	0x0211
 static DEFINE_PCI_DEVICE_TABLE(be_dev_ids) = {
-	{ PCI_DEVICE(BE_VENDOR_ID, BE2_DEVICE_ID_1) },
+	{ PCI_DEVICE(BE_VENDOR_ID, BE_DEVICE_ID1) },
+	{ PCI_DEVICE(BE_VENDOR_ID, OC_DEVICE_ID1) },
+	{ PCI_DEVICE(BE_VENDOR_ID, OC_DEVICE_ID2) },
 	{ 0 }
 };
 MODULE_DEVICE_TABLE(pci, be_dev_ids);
@@ -1859,7 +1859,7 @@  static int __devinit be_probe(struct pci_dev *pdev,
 	if (status != 0)
 		goto stats_clean;
 
-	dev_info(&pdev->dev, BE_NAME " port %d\n", adapter->port_num);
+	dev_info(&pdev->dev, "%s port %d\n", nic_name(pdev), adapter->port_num);
 	return 0;
 
 stats_clean:
@@ -1873,7 +1873,7 @@  rel_reg:
 disable_dev:
 	pci_disable_device(pdev);
 do_none:
-	dev_warn(&pdev->dev, BE_NAME " initialization failed\n");
+	dev_err(&pdev->dev, "%s initialization failed\n", nic_name(pdev));
 	return status;
 }