[dpdk-dev] [PATCH 0/2] cxgbe: Minor bug fix and update license year

DPDK patches and discussions
 help / color / mirror / Atom feed

* [dpdk-dev] [PATCH 0/2] cxgbe: Minor bug fix and update license year
@ 2016-01-31 11:22 Rahul Lakkireddy
  2016-01-31 11:22 ` [dpdk-dev] [PATCH 1/2] cxgbe: fix to copy pci info to other ports under same PF Rahul Lakkireddy
                   ` (2 more replies)
  0 siblings, 3 replies; 7+ messages in thread
From: Rahul Lakkireddy @ 2016-01-31 11:22 UTC (permalink / raw)
  To: dev; +Cc: Felix Marti, Kumar Sanghvi, Nirranjan Kirubaharan

Patch 1 fixes an issue where pci device information was not being
copied to other ports on the Chelsio NIC.

Patch 2 updates the license year for CXGBE PMD to 2016.

Rahul Lakkireddy (2):
  cxgbe: fix to copy pci info to other ports under same PF
  cxgbe: update license year to 2016

 doc/guides/nics/cxgbe.rst               | 2 +-
 drivers/net/cxgbe/Makefile              | 2 +-
 drivers/net/cxgbe/base/adapter.h        | 2 +-
 drivers/net/cxgbe/base/common.h         | 2 +-
 drivers/net/cxgbe/base/t4_chip_type.h   | 2 +-
 drivers/net/cxgbe/base/t4_hw.c          | 2 +-
 drivers/net/cxgbe/base/t4_hw.h          | 2 +-
 drivers/net/cxgbe/base/t4_msg.h         | 2 +-
 drivers/net/cxgbe/base/t4_pci_id_tbl.h  | 2 +-
 drivers/net/cxgbe/base/t4_regs.h        | 2 +-
 drivers/net/cxgbe/base/t4_regs_values.h | 2 +-
 drivers/net/cxgbe/base/t4fw_interface.h | 2 +-
 drivers/net/cxgbe/cxgbe.h               | 2 +-
 drivers/net/cxgbe/cxgbe_compat.h        | 2 +-
 drivers/net/cxgbe/cxgbe_ethdev.c        | 4 +---
 drivers/net/cxgbe/cxgbe_main.c          | 5 ++++-
 drivers/net/cxgbe/sge.c                 | 2 +-
 17 files changed, 20 insertions(+), 19 deletions(-)

-- 
2.5.3

^ permalink raw reply	[flat|nested] 7+ messages in thread

* [dpdk-dev] [PATCH 1/2] cxgbe: fix to copy pci info to other ports under same PF
  2016-01-31 11:22 [dpdk-dev] [PATCH 0/2] cxgbe: Minor bug fix and update license year Rahul Lakkireddy
@ 2016-01-31 11:22 ` Rahul Lakkireddy
  2016-02-24 14:08   ` Bruce Richardson
  2016-01-31 11:22 ` [dpdk-dev] [PATCH 2/2] cxgbe: update license year to 2016 Rahul Lakkireddy
  2016-02-25  9:37 ` [dpdk-dev] [PATCH v2] cxgbe: fix to copy pci info to ports under same PF Rahul Lakkireddy
  2 siblings, 1 reply; 7+ messages in thread
From: Rahul Lakkireddy @ 2016-01-31 11:22 UTC (permalink / raw)
  To: dev; +Cc: Felix Marti, Kumar Sanghvi, Nirranjan Kirubaharan

Chelsio NIC ports share a single PF. Move rte_eth_copy_pci_info()
to copy the pci device information to the remaining ports as well.

Fixes: eeefe73f0af1 ("drivers: copy PCI device info to ethdev data")

Signed-off-by: Rahul Lakkireddy <rahul.lakkireddy@chelsio.com>
Signed-off-by: Kumar Sanghvi <kumaras@chelsio.com>
---
 drivers/net/cxgbe/cxgbe_ethdev.c | 2 --
 drivers/net/cxgbe/cxgbe_main.c   | 3 +++
 2 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/drivers/net/cxgbe/cxgbe_ethdev.c b/drivers/net/cxgbe/cxgbe_ethdev.c
index 97ef152..fd0eb1c 100644
--- a/drivers/net/cxgbe/cxgbe_ethdev.c
+++ b/drivers/net/cxgbe/cxgbe_ethdev.c
@@ -819,8 +819,6 @@ static int eth_cxgbe_dev_init(struct rte_eth_dev *eth_dev)
 
 	pci_dev = eth_dev->pci_dev;
 
-	rte_eth_copy_pci_info(eth_dev, pci_dev);
-
 	snprintf(name, sizeof(name), "cxgbeadapter%d", eth_dev->data->port_id);
 	adapter = rte_zmalloc(name, sizeof(*adapter), 0);
 	if (!adapter)
diff --git a/drivers/net/cxgbe/cxgbe_main.c b/drivers/net/cxgbe/cxgbe_main.c
index aff23d0..6c7eb7f 100644
--- a/drivers/net/cxgbe/cxgbe_main.c
+++ b/drivers/net/cxgbe/cxgbe_main.c
@@ -1166,6 +1166,9 @@ allocate_mac:
 		pi->eth_dev->dev_ops = adapter->eth_dev->dev_ops;
 		pi->eth_dev->tx_pkt_burst = adapter->eth_dev->tx_pkt_burst;
 		pi->eth_dev->rx_pkt_burst = adapter->eth_dev->rx_pkt_burst;
+
+		rte_eth_copy_pci_info(pi->eth_dev, pi->eth_dev->pci_dev);
+
 		TAILQ_INIT(&pi->eth_dev->link_intr_cbs);
 
 		pi->eth_dev->data->mac_addrs = rte_zmalloc(name,
-- 
2.5.3

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [dpdk-dev] [PATCH 1/2] cxgbe: fix to copy pci info to other ports under same PF
  2016-01-31 11:22 ` [dpdk-dev] [PATCH 1/2] cxgbe: fix to copy pci info to other ports under same PF Rahul Lakkireddy
@ 2016-02-24 14:08   ` Bruce Richardson
  0 siblings, 0 replies; 7+ messages in thread
From: Bruce Richardson @ 2016-02-24 14:08 UTC (permalink / raw)
  To: Rahul Lakkireddy; +Cc: dev, Felix Marti, Nirranjan Kirubaharan, Kumar Sanghvi

On Sun, Jan 31, 2016 at 04:52:49PM +0530, Rahul Lakkireddy wrote:
> Chelsio NIC ports share a single PF. Move rte_eth_copy_pci_info()
> to copy the pci device information to the remaining ports as well.
> 
> Fixes: eeefe73f0af1 ("drivers: copy PCI device info to ethdev data")
> 
> Signed-off-by: Rahul Lakkireddy <rahul.lakkireddy@chelsio.com>
> Signed-off-by: Kumar Sanghvi <kumaras@chelsio.com>
> ---

Hi,

can you perhaps submit this fix as a patch alone, without the copyright update
patch attached. [Feel free to upate the copyright year on the two files affected
here by this change, if you like.]

FYI: Also, the commit title is slightly too long. It should be around 50 characters
long (for me, vim highlights the correct length for me). It could be shortened
by dropping the word "other". :-)

/Bruce

>  drivers/net/cxgbe/cxgbe_ethdev.c | 2 --
>  drivers/net/cxgbe/cxgbe_main.c   | 3 +++
>  2 files changed, 3 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/net/cxgbe/cxgbe_ethdev.c b/drivers/net/cxgbe/cxgbe_ethdev.c
> index 97ef152..fd0eb1c 100644
> --- a/drivers/net/cxgbe/cxgbe_ethdev.c
> +++ b/drivers/net/cxgbe/cxgbe_ethdev.c
> @@ -819,8 +819,6 @@ static int eth_cxgbe_dev_init(struct rte_eth_dev *eth_dev)
>  
>  	pci_dev = eth_dev->pci_dev;
>  
> -	rte_eth_copy_pci_info(eth_dev, pci_dev);
> -
>  	snprintf(name, sizeof(name), "cxgbeadapter%d", eth_dev->data->port_id);
>  	adapter = rte_zmalloc(name, sizeof(*adapter), 0);
>  	if (!adapter)
> diff --git a/drivers/net/cxgbe/cxgbe_main.c b/drivers/net/cxgbe/cxgbe_main.c
> index aff23d0..6c7eb7f 100644
> --- a/drivers/net/cxgbe/cxgbe_main.c
> +++ b/drivers/net/cxgbe/cxgbe_main.c
> @@ -1166,6 +1166,9 @@ allocate_mac:
>  		pi->eth_dev->dev_ops = adapter->eth_dev->dev_ops;
>  		pi->eth_dev->tx_pkt_burst = adapter->eth_dev->tx_pkt_burst;
>  		pi->eth_dev->rx_pkt_burst = adapter->eth_dev->rx_pkt_burst;
> +
> +		rte_eth_copy_pci_info(pi->eth_dev, pi->eth_dev->pci_dev);
> +
>  		TAILQ_INIT(&pi->eth_dev->link_intr_cbs);
>  
>  		pi->eth_dev->data->mac_addrs = rte_zmalloc(name,
> -- 
> 2.5.3
> 

^ permalink raw reply	[flat|nested] 7+ messages in thread

* [dpdk-dev] [PATCH 2/2] cxgbe: update license year to 2016
  2016-01-31 11:22 [dpdk-dev] [PATCH 0/2] cxgbe: Minor bug fix and update license year Rahul Lakkireddy
  2016-01-31 11:22 ` [dpdk-dev] [PATCH 1/2] cxgbe: fix to copy pci info to other ports under same PF Rahul Lakkireddy
@ 2016-01-31 11:22 ` Rahul Lakkireddy
  2016-02-24 13:59   ` Bruce Richardson
  2016-02-25  9:37 ` [dpdk-dev] [PATCH v2] cxgbe: fix to copy pci info to ports under same PF Rahul Lakkireddy
  2 siblings, 1 reply; 7+ messages in thread
From: Rahul Lakkireddy @ 2016-01-31 11:22 UTC (permalink / raw)
  To: dev; +Cc: Felix Marti, Kumar Sanghvi, Nirranjan Kirubaharan

Update CXGBE PMD license year to 2016.

Signed-off-by: Rahul Lakkireddy <rahul.lakkireddy@chelsio.com>
Signed-off-by: Kumar Sanghvi <kumaras@chelsio.com>
---
 doc/guides/nics/cxgbe.rst               | 2 +-
 drivers/net/cxgbe/Makefile              | 2 +-
 drivers/net/cxgbe/base/adapter.h        | 2 +-
 drivers/net/cxgbe/base/common.h         | 2 +-
 drivers/net/cxgbe/base/t4_chip_type.h   | 2 +-
 drivers/net/cxgbe/base/t4_hw.c          | 2 +-
 drivers/net/cxgbe/base/t4_hw.h          | 2 +-
 drivers/net/cxgbe/base/t4_msg.h         | 2 +-
 drivers/net/cxgbe/base/t4_pci_id_tbl.h  | 2 +-
 drivers/net/cxgbe/base/t4_regs.h        | 2 +-
 drivers/net/cxgbe/base/t4_regs_values.h | 2 +-
 drivers/net/cxgbe/base/t4fw_interface.h | 2 +-
 drivers/net/cxgbe/cxgbe.h               | 2 +-
 drivers/net/cxgbe/cxgbe_compat.h        | 2 +-
 drivers/net/cxgbe/cxgbe_ethdev.c        | 2 +-
 drivers/net/cxgbe/cxgbe_main.c          | 2 +-
 drivers/net/cxgbe/sge.c                 | 2 +-
 17 files changed, 17 insertions(+), 17 deletions(-)

diff --git a/doc/guides/nics/cxgbe.rst b/doc/guides/nics/cxgbe.rst
index d718f19..ac0a02c 100644
--- a/doc/guides/nics/cxgbe.rst
+++ b/doc/guides/nics/cxgbe.rst
@@ -1,5 +1,5 @@
 ..  BSD LICENSE
-    Copyright 2015 Chelsio Communications.
+    Copyright 2015-2016 Chelsio Communications.
     All rights reserved.
 
     Redistribution and use in source and binary forms, with or without
diff --git a/drivers/net/cxgbe/Makefile b/drivers/net/cxgbe/Makefile
index 0711976..404154d 100644
--- a/drivers/net/cxgbe/Makefile
+++ b/drivers/net/cxgbe/Makefile
@@ -1,6 +1,6 @@
 #   BSD LICENSE
 #
-#   Copyright(c) 2014-2015 Chelsio Communications.
+#   Copyright(c) 2014-2016 Chelsio Communications.
 #   All rights reserved.
 #
 #   Redistribution and use in source and binary forms, with or without
diff --git a/drivers/net/cxgbe/base/adapter.h b/drivers/net/cxgbe/base/adapter.h
index a5225c0..9c54691 100644
--- a/drivers/net/cxgbe/base/adapter.h
+++ b/drivers/net/cxgbe/base/adapter.h
@@ -1,7 +1,7 @@
 /*-
  *   BSD LICENSE
  *
- *   Copyright(c) 2014-2015 Chelsio Communications.
+ *   Copyright(c) 2014-2016 Chelsio Communications.
  *   All rights reserved.
  *
  *   Redistribution and use in source and binary forms, with or without
diff --git a/drivers/net/cxgbe/base/common.h b/drivers/net/cxgbe/base/common.h
index cf2e82d..e166279 100644
--- a/drivers/net/cxgbe/base/common.h
+++ b/drivers/net/cxgbe/base/common.h
@@ -1,7 +1,7 @@
 /*-
  *   BSD LICENSE
  *
- *   Copyright(c) 2014-2015 Chelsio Communications.
+ *   Copyright(c) 2014-2016 Chelsio Communications.
  *   All rights reserved.
  *
  *   Redistribution and use in source and binary forms, with or without
diff --git a/drivers/net/cxgbe/base/t4_chip_type.h b/drivers/net/cxgbe/base/t4_chip_type.h
index 1ca6803..3135733 100644
--- a/drivers/net/cxgbe/base/t4_chip_type.h
+++ b/drivers/net/cxgbe/base/t4_chip_type.h
@@ -1,7 +1,7 @@
 /*-
  *   BSD LICENSE
  *
- *   Copyright(c) 2014-2015 Chelsio Communications.
+ *   Copyright(c) 2014-2016 Chelsio Communications.
  *   All rights reserved.
  *
  *   Redistribution and use in source and binary forms, with or without
diff --git a/drivers/net/cxgbe/base/t4_hw.c b/drivers/net/cxgbe/base/t4_hw.c
index 884d2cf..f4099cc 100644
--- a/drivers/net/cxgbe/base/t4_hw.c
+++ b/drivers/net/cxgbe/base/t4_hw.c
@@ -1,7 +1,7 @@
 /*-
  *   BSD LICENSE
  *
- *   Copyright(c) 2014-2015 Chelsio Communications.
+ *   Copyright(c) 2014-2016 Chelsio Communications.
  *   All rights reserved.
  *
  *   Redistribution and use in source and binary forms, with or without
diff --git a/drivers/net/cxgbe/base/t4_hw.h b/drivers/net/cxgbe/base/t4_hw.h
index bf623cf..c4b24ba 100644
--- a/drivers/net/cxgbe/base/t4_hw.h
+++ b/drivers/net/cxgbe/base/t4_hw.h
@@ -1,7 +1,7 @@
 /*-
  *   BSD LICENSE
  *
- *   Copyright(c) 2014-2015 Chelsio Communications.
+ *   Copyright(c) 2014-2016 Chelsio Communications.
  *   All rights reserved.
  *
  *   Redistribution and use in source and binary forms, with or without
diff --git a/drivers/net/cxgbe/base/t4_msg.h b/drivers/net/cxgbe/base/t4_msg.h
index 4b04cd0..62c503d 100644
--- a/drivers/net/cxgbe/base/t4_msg.h
+++ b/drivers/net/cxgbe/base/t4_msg.h
@@ -1,7 +1,7 @@
 /*-
  *   BSD LICENSE
  *
- *   Copyright(c) 2014-2015 Chelsio Communications.
+ *   Copyright(c) 2014-2016 Chelsio Communications.
  *   All rights reserved.
  *
  *   Redistribution and use in source and binary forms, with or without
diff --git a/drivers/net/cxgbe/base/t4_pci_id_tbl.h b/drivers/net/cxgbe/base/t4_pci_id_tbl.h
index 110fadb..252ec4e 100644
--- a/drivers/net/cxgbe/base/t4_pci_id_tbl.h
+++ b/drivers/net/cxgbe/base/t4_pci_id_tbl.h
@@ -1,7 +1,7 @@
 /*-
  *   BSD LICENSE
  *
- *   Copyright(c) 2014-2015 Chelsio Communications.
+ *   Copyright(c) 2014-2016 Chelsio Communications.
  *   All rights reserved.
  *
  *   Redistribution and use in source and binary forms, with or without
diff --git a/drivers/net/cxgbe/base/t4_regs.h b/drivers/net/cxgbe/base/t4_regs.h
index 9057e40..6a85771 100644
--- a/drivers/net/cxgbe/base/t4_regs.h
+++ b/drivers/net/cxgbe/base/t4_regs.h
@@ -1,7 +1,7 @@
 /*-
  *   BSD LICENSE
  *
- *   Copyright(c) 2014-2015 Chelsio Communications.
+ *   Copyright(c) 2014-2016 Chelsio Communications.
  *   All rights reserved.
  *
  *   Redistribution and use in source and binary forms, with or without
diff --git a/drivers/net/cxgbe/base/t4_regs_values.h b/drivers/net/cxgbe/base/t4_regs_values.h
index d7d3144..aa079ef 100644
--- a/drivers/net/cxgbe/base/t4_regs_values.h
+++ b/drivers/net/cxgbe/base/t4_regs_values.h
@@ -1,7 +1,7 @@
 /*-
  *   BSD LICENSE
  *
- *   Copyright(c) 2014-2015 Chelsio Communications.
+ *   Copyright(c) 2014-2016 Chelsio Communications.
  *   All rights reserved.
  *
  *   Redistribution and use in source and binary forms, with or without
diff --git a/drivers/net/cxgbe/base/t4fw_interface.h b/drivers/net/cxgbe/base/t4fw_interface.h
index 74f19fe..22b328d 100644
--- a/drivers/net/cxgbe/base/t4fw_interface.h
+++ b/drivers/net/cxgbe/base/t4fw_interface.h
@@ -1,7 +1,7 @@
 /*-
  *   BSD LICENSE
  *
- *   Copyright(c) 2014-2015 Chelsio Communications.
+ *   Copyright(c) 2014-2016 Chelsio Communications.
  *   All rights reserved.
  *
  *   Redistribution and use in source and binary forms, with or without
diff --git a/drivers/net/cxgbe/cxgbe.h b/drivers/net/cxgbe/cxgbe.h
index 0201c99..6cb9d1e 100644
--- a/drivers/net/cxgbe/cxgbe.h
+++ b/drivers/net/cxgbe/cxgbe.h
@@ -1,7 +1,7 @@
 /*-
  *   BSD LICENSE
  *
- *   Copyright(c) 2014-2015 Chelsio Communications.
+ *   Copyright(c) 2014-2016 Chelsio Communications.
  *   All rights reserved.
  *
  *   Redistribution and use in source and binary forms, with or without
diff --git a/drivers/net/cxgbe/cxgbe_compat.h b/drivers/net/cxgbe/cxgbe_compat.h
index e68f8f5..b5aeff7 100644
--- a/drivers/net/cxgbe/cxgbe_compat.h
+++ b/drivers/net/cxgbe/cxgbe_compat.h
@@ -1,7 +1,7 @@
 /*-
  *   BSD LICENSE
  *
- *   Copyright(c) 2014-2015 Chelsio Communications.
+ *   Copyright(c) 2014-2016 Chelsio Communications.
  *   All rights reserved.
  *
  *   Redistribution and use in source and binary forms, with or without
diff --git a/drivers/net/cxgbe/cxgbe_ethdev.c b/drivers/net/cxgbe/cxgbe_ethdev.c
index fd0eb1c..8b63e16 100644
--- a/drivers/net/cxgbe/cxgbe_ethdev.c
+++ b/drivers/net/cxgbe/cxgbe_ethdev.c
@@ -1,7 +1,7 @@
 /*-
  *   BSD LICENSE
  *
- *   Copyright(c) 2014-2015 Chelsio Communications.
+ *   Copyright(c) 2014-2016 Chelsio Communications.
  *   All rights reserved.
  *
  *   Redistribution and use in source and binary forms, with or without
diff --git a/drivers/net/cxgbe/cxgbe_main.c b/drivers/net/cxgbe/cxgbe_main.c
index 6c7eb7f..6783d0b 100644
--- a/drivers/net/cxgbe/cxgbe_main.c
+++ b/drivers/net/cxgbe/cxgbe_main.c
@@ -1,7 +1,7 @@
 /*-
  *   BSD LICENSE
  *
- *   Copyright(c) 2014-2015 Chelsio Communications.
+ *   Copyright(c) 2014-2016 Chelsio Communications.
  *   All rights reserved.
  *
  *   Redistribution and use in source and binary forms, with or without
diff --git a/drivers/net/cxgbe/sge.c b/drivers/net/cxgbe/sge.c
index 3c62d03..fd24753 100644
--- a/drivers/net/cxgbe/sge.c
+++ b/drivers/net/cxgbe/sge.c
@@ -1,7 +1,7 @@
 /*-
  *   BSD LICENSE
  *
- *   Copyright(c) 2014-2015 Chelsio Communications.
+ *   Copyright(c) 2014-2016 Chelsio Communications.
  *   All rights reserved.
  *
  *   Redistribution and use in source and binary forms, with or without
-- 
2.5.3

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [dpdk-dev] [PATCH 2/2] cxgbe: update license year to 2016
  2016-01-31 11:22 ` [dpdk-dev] [PATCH 2/2] cxgbe: update license year to 2016 Rahul Lakkireddy
@ 2016-02-24 13:59   ` Bruce Richardson
  0 siblings, 0 replies; 7+ messages in thread
From: Bruce Richardson @ 2016-02-24 13:59 UTC (permalink / raw)
  To: Rahul Lakkireddy; +Cc: dev, Felix Marti, Nirranjan Kirubaharan, Kumar Sanghvi

On Sun, Jan 31, 2016 at 04:52:50PM +0530, Rahul Lakkireddy wrote:
> Update CXGBE PMD license year to 2016.
> 
> Signed-off-by: Rahul Lakkireddy <rahul.lakkireddy@chelsio.com>
> Signed-off-by: Kumar Sanghvi <kumaras@chelsio.com>
> ---

Although I don't think it's officially documented, in DPDK - as in many open
source projects - the license year is only updated once a modification is made
to the file in question.

/Bruce

^ permalink raw reply	[flat|nested] 7+ messages in thread

* [dpdk-dev] [PATCH v2] cxgbe: fix to copy pci info to ports under same PF
  2016-01-31 11:22 [dpdk-dev] [PATCH 0/2] cxgbe: Minor bug fix and update license year Rahul Lakkireddy
  2016-01-31 11:22 ` [dpdk-dev] [PATCH 1/2] cxgbe: fix to copy pci info to other ports under same PF Rahul Lakkireddy
  2016-01-31 11:22 ` [dpdk-dev] [PATCH 2/2] cxgbe: update license year to 2016 Rahul Lakkireddy
@ 2016-02-25  9:37 ` Rahul Lakkireddy
  2016-02-25 13:39   ` Bruce Richardson
  2 siblings, 1 reply; 7+ messages in thread
From: Rahul Lakkireddy @ 2016-02-25  9:37 UTC (permalink / raw)
  To: dev; +Cc: Kumar Sanghvi, Nirranjan Kirubaharan

Chelsio NIC ports share a single PF. Move rte_eth_copy_pci_info()
to copy the pci device information to the remaining ports as well.

Also update license year to 2016.

Fixes: eeefe73f0af1 ("drivers: copy PCI device info to ethdev data")

Signed-off-by: Rahul Lakkireddy <rahul.lakkireddy@chelsio.com>
Signed-off-by: Kumar Sanghvi <kumaras@chelsio.com>
---
v2:
- Dropped the separate patch that updated license year for unmodified
  CXGBE related files.

 drivers/net/cxgbe/cxgbe_ethdev.c | 4 +---
 drivers/net/cxgbe/cxgbe_main.c   | 5 ++++-
 2 files changed, 5 insertions(+), 4 deletions(-)

diff --git a/drivers/net/cxgbe/cxgbe_ethdev.c b/drivers/net/cxgbe/cxgbe_ethdev.c
index 97ef152..8b63e16 100644
--- a/drivers/net/cxgbe/cxgbe_ethdev.c
+++ b/drivers/net/cxgbe/cxgbe_ethdev.c
@@ -1,7 +1,7 @@
 /*-
  *   BSD LICENSE
  *
- *   Copyright(c) 2014-2015 Chelsio Communications.
+ *   Copyright(c) 2014-2016 Chelsio Communications.
  *   All rights reserved.
  *
  *   Redistribution and use in source and binary forms, with or without
@@ -819,8 +819,6 @@ static int eth_cxgbe_dev_init(struct rte_eth_dev *eth_dev)
 
 	pci_dev = eth_dev->pci_dev;
 
-	rte_eth_copy_pci_info(eth_dev, pci_dev);
-
 	snprintf(name, sizeof(name), "cxgbeadapter%d", eth_dev->data->port_id);
 	adapter = rte_zmalloc(name, sizeof(*adapter), 0);
 	if (!adapter)
diff --git a/drivers/net/cxgbe/cxgbe_main.c b/drivers/net/cxgbe/cxgbe_main.c
index 649b941..9321cbe 100644
--- a/drivers/net/cxgbe/cxgbe_main.c
+++ b/drivers/net/cxgbe/cxgbe_main.c
@@ -1,7 +1,7 @@
 /*-
  *   BSD LICENSE
  *
- *   Copyright(c) 2014-2015 Chelsio Communications.
+ *   Copyright(c) 2014-2016 Chelsio Communications.
  *   All rights reserved.
  *
  *   Redistribution and use in source and binary forms, with or without
@@ -1166,6 +1166,9 @@ allocate_mac:
 		pi->eth_dev->dev_ops = adapter->eth_dev->dev_ops;
 		pi->eth_dev->tx_pkt_burst = adapter->eth_dev->tx_pkt_burst;
 		pi->eth_dev->rx_pkt_burst = adapter->eth_dev->rx_pkt_burst;
+
+		rte_eth_copy_pci_info(pi->eth_dev, pi->eth_dev->pci_dev);
+
 		TAILQ_INIT(&pi->eth_dev->link_intr_cbs);
 
 		pi->eth_dev->data->mac_addrs = rte_zmalloc(name,
-- 
2.5.3

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [dpdk-dev] [PATCH v2] cxgbe: fix to copy pci info to ports under same PF
  2016-02-25  9:37 ` [dpdk-dev] [PATCH v2] cxgbe: fix to copy pci info to ports under same PF Rahul Lakkireddy
@ 2016-02-25 13:39   ` Bruce Richardson
  0 siblings, 0 replies; 7+ messages in thread
From: Bruce Richardson @ 2016-02-25 13:39 UTC (permalink / raw)
  To: Rahul Lakkireddy; +Cc: dev, Kumar Sanghvi, Nirranjan Kirubaharan

On Thu, Feb 25, 2016 at 03:07:53PM +0530, Rahul Lakkireddy wrote:
> Chelsio NIC ports share a single PF. Move rte_eth_copy_pci_info()
> to copy the pci device information to the remaining ports as well.
> 
> Also update license year to 2016.
> 
> Fixes: eeefe73f0af1 ("drivers: copy PCI device info to ethdev data")
> 
> Signed-off-by: Rahul Lakkireddy <rahul.lakkireddy@chelsio.com>
> Signed-off-by: Kumar Sanghvi <kumaras@chelsio.com>

Applied to dpdk-next-net/rel_16_04

Thanks,
/Bruce

^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2016-02-25 13:40 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-01-31 11:22 [dpdk-dev] [PATCH 0/2] cxgbe: Minor bug fix and update license year Rahul Lakkireddy
2016-01-31 11:22 ` [dpdk-dev] [PATCH 1/2] cxgbe: fix to copy pci info to other ports under same PF Rahul Lakkireddy
2016-02-24 14:08   ` Bruce Richardson
2016-01-31 11:22 ` [dpdk-dev] [PATCH 2/2] cxgbe: update license year to 2016 Rahul Lakkireddy
2016-02-24 13:59   ` Bruce Richardson
2016-02-25  9:37 ` [dpdk-dev] [PATCH v2] cxgbe: fix to copy pci info to ports under same PF Rahul Lakkireddy
2016-02-25 13:39   ` Bruce Richardson

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).