* [dpdk-dev] [PATCH] net/cxgbe: update Chelsio T5/T6 NIC device ids
@ 2019-04-19 9:22 Rahul Lakkireddy
2019-04-19 9:22 ` Rahul Lakkireddy
` (3 more replies)
0 siblings, 4 replies; 24+ messages in thread
From: Rahul Lakkireddy @ 2019-04-19 9:22 UTC (permalink / raw)
To: dev; +Cc: nirranjan, indranil, stable
Fixes: 04868e5bfddd ("net/cxgbe: add support to run Chelsio T6 cards")
Cc: stable@dpdk.org
Signed-off-by: Rahul Lakkireddy <rahul.lakkireddy@chelsio.com>
---
drivers/net/cxgbe/base/t4_pci_id_tbl.h | 52 +++++++++++++++++++++++++-
1 file changed, 51 insertions(+), 1 deletion(-)
diff --git a/drivers/net/cxgbe/base/t4_pci_id_tbl.h b/drivers/net/cxgbe/base/t4_pci_id_tbl.h
index 5f5cbe048..f5f027a2e 100644
--- a/drivers/net/cxgbe/base/t4_pci_id_tbl.h
+++ b/drivers/net/cxgbe/base/t4_pci_id_tbl.h
@@ -1,5 +1,5 @@
/* SPDX-License-Identifier: BSD-3-Clause
- * Copyright(c) 2014-2018 Chelsio Communications.
+ * Copyright(c) 2014-2019 Chelsio Communications.
* All rights reserved.
*/
@@ -103,6 +103,12 @@ CH_PCI_DEVICE_ID_TABLE_DEFINE_BEGIN
CH_PCI_ID_TABLE_FENTRY(0x5013), /* T580-chr */
CH_PCI_ID_TABLE_FENTRY(0x5014), /* T580-so */
CH_PCI_ID_TABLE_FENTRY(0x5015), /* T502-bt */
+ CH_PCI_ID_TABLE_FENTRY(0x5016), /* T580-OCP-SO */
+ CH_PCI_ID_TABLE_FENTRY(0x5017), /* T520-OCP-SO */
+ CH_PCI_ID_TABLE_FENTRY(0x5018), /* T540-BT */
+ CH_PCI_ID_TABLE_FENTRY(0x5019), /* T540-LP-BT */
+ CH_PCI_ID_TABLE_FENTRY(0x501a), /* T540-SO-BT */
+ CH_PCI_ID_TABLE_FENTRY(0x501b), /* T540-SO-CR */
CH_PCI_ID_TABLE_FENTRY(0x5080), /* Custom T540-cr */
CH_PCI_ID_TABLE_FENTRY(0x5081), /* Custom T540-LL-cr */
CH_PCI_ID_TABLE_FENTRY(0x5082), /* Custom T504-cr */
@@ -116,19 +122,63 @@ CH_PCI_DEVICE_ID_TABLE_DEFINE_BEGIN
CH_PCI_ID_TABLE_FENTRY(0x5090), /* Custom T540-CR */
CH_PCI_ID_TABLE_FENTRY(0x5091), /* Custom T522-CR */
CH_PCI_ID_TABLE_FENTRY(0x5092), /* Custom T520-CR */
+ CH_PCI_ID_TABLE_FENTRY(0x5093), /* Custom T580-LP-CR */
+ CH_PCI_ID_TABLE_FENTRY(0x5094), /* Custom T540-CR */
+ CH_PCI_ID_TABLE_FENTRY(0x5095), /* Custom T540-CR-SO */
+ CH_PCI_ID_TABLE_FENTRY(0x5096), /* Custom T580-CR */
+ CH_PCI_ID_TABLE_FENTRY(0x5097), /* Custom T520-KR */
+ CH_PCI_ID_TABLE_FENTRY(0x5098), /* Custom 2x40G QSFP */
+ CH_PCI_ID_TABLE_FENTRY(0x5099), /* Custom 2x40G QSFP */
+ CH_PCI_ID_TABLE_FENTRY(0x509A), /* Custom T520-CR */
+ CH_PCI_ID_TABLE_FENTRY(0x509B), /* Custom T540-CR LOM */
+ CH_PCI_ID_TABLE_FENTRY(0x509c), /* Custom T520-CR SFP+ LOM */
+ CH_PCI_ID_TABLE_FENTRY(0x509d), /* Custom T540-CR SFP+ */
+ CH_PCI_ID_TABLE_FENTRY(0x509e), /* Custom T520-CR */
+ CH_PCI_ID_TABLE_FENTRY(0x509f), /* Custom T540-CR */
+ CH_PCI_ID_TABLE_FENTRY(0x50a0), /* Custom T540-CR */
+ CH_PCI_ID_TABLE_FENTRY(0x50a1), /* Custom T540-CR */
+ CH_PCI_ID_TABLE_FENTRY(0x50a2), /* Custom T580-KR4 */
+ CH_PCI_ID_TABLE_FENTRY(0x50a3), /* Custom T580-KR4 */
+ CH_PCI_ID_TABLE_FENTRY(0x50a4), /* Custom 2x T540-CR */
+ CH_PCI_ID_TABLE_FENTRY(0x50a5), /* Custom T522-BT */
+ CH_PCI_ID_TABLE_FENTRY(0x50a6), /* Custom T522-BT-SO */
+ CH_PCI_ID_TABLE_FENTRY(0x50a7), /* Custom T580-CR */
+ CH_PCI_ID_TABLE_FENTRY(0x50a8), /* Custom T580-KR */
+ CH_PCI_ID_TABLE_FENTRY(0x50a9), /* Custom T580-KR */
+ CH_PCI_ID_TABLE_FENTRY(0x50aa), /* Custom T580-CR */
+ CH_PCI_ID_TABLE_FENTRY(0x50ab), /* Custom T520-CR */
+ CH_PCI_ID_TABLE_FENTRY(0x50ac), /* Custom T540-BT */
+ CH_PCI_ID_TABLE_FENTRY(0x50ad), /* Custom T520-CR */
+ CH_PCI_ID_TABLE_FENTRY(0x50ae), /* Custom T540-XL-SO */
+ CH_PCI_ID_TABLE_FENTRY(0x50af), /* Custom T580-KR-SO */
+ CH_PCI_ID_TABLE_FENTRY(0x50b0), /* Custom T520-CR-LOM */
/* T6 adapter */
CH_PCI_ID_TABLE_FENTRY(0x6001), /* T6225-CR */
CH_PCI_ID_TABLE_FENTRY(0x6002), /* T6225-SO-CR */
CH_PCI_ID_TABLE_FENTRY(0x6003), /* T6425-CR */
+ CH_PCI_ID_TABLE_FENTRY(0x6004), /* T6425-SO-CR */
CH_PCI_ID_TABLE_FENTRY(0x6005), /* T6225-OCP */
+ CH_PCI_ID_TABLE_FENTRY(0x6006), /* T62100-OCP-SO */
CH_PCI_ID_TABLE_FENTRY(0x6007), /* T62100-LP-CR */
CH_PCI_ID_TABLE_FENTRY(0x6008), /* T62100-SO-CR */
+ CH_PCI_ID_TABLE_FENTRY(0x6009), /* T6210-BT */
CH_PCI_ID_TABLE_FENTRY(0x600d), /* T62100-CR */
CH_PCI_ID_TABLE_FENTRY(0x6011), /* T6225-LL-CR */
CH_PCI_ID_TABLE_FENTRY(0x6014), /* T61100-OCP-SO */
+ CH_PCI_ID_TABLE_FENTRY(0x6015), /* T6201-BT */
CH_PCI_ID_TABLE_FENTRY(0x6080), /* Custom T6225-CR SFP28 */
CH_PCI_ID_TABLE_FENTRY(0x6081), /* Custom T62100-CR */
+ CH_PCI_ID_TABLE_FENTRY(0x6082), /* Custom T6225-CR */
+ CH_PCI_ID_TABLE_FENTRY(0x6083), /* Custom T62100-CR */
+ CH_PCI_ID_TABLE_FENTRY(0x6084), /* Custom T64100-CR */
+ CH_PCI_ID_TABLE_FENTRY(0x6085), /* Custom T6240-SO */
+ CH_PCI_ID_TABLE_FENTRY(0x6086), /* Custom T6225-SO-CR */
+ CH_PCI_ID_TABLE_FENTRY(0x6087), /* Custom T6225-CR */
+ CH_PCI_ID_TABLE_FENTRY(0x6088), /* Custom T62100-CR */
+ CH_PCI_ID_TABLE_FENTRY(0x6089), /* Custom T62100-KR */
+ CH_PCI_ID_TABLE_FENTRY(0x608a), /* Custom T62100-CR */
+ CH_PCI_ID_TABLE_FENTRY(0x608b), /* Custom T6225-CR */
CH_PCI_DEVICE_ID_TABLE_DEFINE_END;
#endif /* CH_PCI_DEVICE_ID_TABLE_DEFINE_BEGIN */
--
2.18.0
^ permalink raw reply [flat|nested] 24+ messages in thread
* [dpdk-dev] [PATCH] net/cxgbe: update Chelsio T5/T6 NIC device ids
2019-04-19 9:22 [dpdk-dev] [PATCH] net/cxgbe: update Chelsio T5/T6 NIC device ids Rahul Lakkireddy
@ 2019-04-19 9:22 ` Rahul Lakkireddy
2019-04-19 9:30 ` Ferruh Yigit
` (2 subsequent siblings)
3 siblings, 0 replies; 24+ messages in thread
From: Rahul Lakkireddy @ 2019-04-19 9:22 UTC (permalink / raw)
To: dev; +Cc: nirranjan, indranil, stable
Fixes: 04868e5bfddd ("net/cxgbe: add support to run Chelsio T6 cards")
Cc: stable@dpdk.org
Signed-off-by: Rahul Lakkireddy <rahul.lakkireddy@chelsio.com>
---
drivers/net/cxgbe/base/t4_pci_id_tbl.h | 52 +++++++++++++++++++++++++-
1 file changed, 51 insertions(+), 1 deletion(-)
diff --git a/drivers/net/cxgbe/base/t4_pci_id_tbl.h b/drivers/net/cxgbe/base/t4_pci_id_tbl.h
index 5f5cbe048..f5f027a2e 100644
--- a/drivers/net/cxgbe/base/t4_pci_id_tbl.h
+++ b/drivers/net/cxgbe/base/t4_pci_id_tbl.h
@@ -1,5 +1,5 @@
/* SPDX-License-Identifier: BSD-3-Clause
- * Copyright(c) 2014-2018 Chelsio Communications.
+ * Copyright(c) 2014-2019 Chelsio Communications.
* All rights reserved.
*/
@@ -103,6 +103,12 @@ CH_PCI_DEVICE_ID_TABLE_DEFINE_BEGIN
CH_PCI_ID_TABLE_FENTRY(0x5013), /* T580-chr */
CH_PCI_ID_TABLE_FENTRY(0x5014), /* T580-so */
CH_PCI_ID_TABLE_FENTRY(0x5015), /* T502-bt */
+ CH_PCI_ID_TABLE_FENTRY(0x5016), /* T580-OCP-SO */
+ CH_PCI_ID_TABLE_FENTRY(0x5017), /* T520-OCP-SO */
+ CH_PCI_ID_TABLE_FENTRY(0x5018), /* T540-BT */
+ CH_PCI_ID_TABLE_FENTRY(0x5019), /* T540-LP-BT */
+ CH_PCI_ID_TABLE_FENTRY(0x501a), /* T540-SO-BT */
+ CH_PCI_ID_TABLE_FENTRY(0x501b), /* T540-SO-CR */
CH_PCI_ID_TABLE_FENTRY(0x5080), /* Custom T540-cr */
CH_PCI_ID_TABLE_FENTRY(0x5081), /* Custom T540-LL-cr */
CH_PCI_ID_TABLE_FENTRY(0x5082), /* Custom T504-cr */
@@ -116,19 +122,63 @@ CH_PCI_DEVICE_ID_TABLE_DEFINE_BEGIN
CH_PCI_ID_TABLE_FENTRY(0x5090), /* Custom T540-CR */
CH_PCI_ID_TABLE_FENTRY(0x5091), /* Custom T522-CR */
CH_PCI_ID_TABLE_FENTRY(0x5092), /* Custom T520-CR */
+ CH_PCI_ID_TABLE_FENTRY(0x5093), /* Custom T580-LP-CR */
+ CH_PCI_ID_TABLE_FENTRY(0x5094), /* Custom T540-CR */
+ CH_PCI_ID_TABLE_FENTRY(0x5095), /* Custom T540-CR-SO */
+ CH_PCI_ID_TABLE_FENTRY(0x5096), /* Custom T580-CR */
+ CH_PCI_ID_TABLE_FENTRY(0x5097), /* Custom T520-KR */
+ CH_PCI_ID_TABLE_FENTRY(0x5098), /* Custom 2x40G QSFP */
+ CH_PCI_ID_TABLE_FENTRY(0x5099), /* Custom 2x40G QSFP */
+ CH_PCI_ID_TABLE_FENTRY(0x509A), /* Custom T520-CR */
+ CH_PCI_ID_TABLE_FENTRY(0x509B), /* Custom T540-CR LOM */
+ CH_PCI_ID_TABLE_FENTRY(0x509c), /* Custom T520-CR SFP+ LOM */
+ CH_PCI_ID_TABLE_FENTRY(0x509d), /* Custom T540-CR SFP+ */
+ CH_PCI_ID_TABLE_FENTRY(0x509e), /* Custom T520-CR */
+ CH_PCI_ID_TABLE_FENTRY(0x509f), /* Custom T540-CR */
+ CH_PCI_ID_TABLE_FENTRY(0x50a0), /* Custom T540-CR */
+ CH_PCI_ID_TABLE_FENTRY(0x50a1), /* Custom T540-CR */
+ CH_PCI_ID_TABLE_FENTRY(0x50a2), /* Custom T580-KR4 */
+ CH_PCI_ID_TABLE_FENTRY(0x50a3), /* Custom T580-KR4 */
+ CH_PCI_ID_TABLE_FENTRY(0x50a4), /* Custom 2x T540-CR */
+ CH_PCI_ID_TABLE_FENTRY(0x50a5), /* Custom T522-BT */
+ CH_PCI_ID_TABLE_FENTRY(0x50a6), /* Custom T522-BT-SO */
+ CH_PCI_ID_TABLE_FENTRY(0x50a7), /* Custom T580-CR */
+ CH_PCI_ID_TABLE_FENTRY(0x50a8), /* Custom T580-KR */
+ CH_PCI_ID_TABLE_FENTRY(0x50a9), /* Custom T580-KR */
+ CH_PCI_ID_TABLE_FENTRY(0x50aa), /* Custom T580-CR */
+ CH_PCI_ID_TABLE_FENTRY(0x50ab), /* Custom T520-CR */
+ CH_PCI_ID_TABLE_FENTRY(0x50ac), /* Custom T540-BT */
+ CH_PCI_ID_TABLE_FENTRY(0x50ad), /* Custom T520-CR */
+ CH_PCI_ID_TABLE_FENTRY(0x50ae), /* Custom T540-XL-SO */
+ CH_PCI_ID_TABLE_FENTRY(0x50af), /* Custom T580-KR-SO */
+ CH_PCI_ID_TABLE_FENTRY(0x50b0), /* Custom T520-CR-LOM */
/* T6 adapter */
CH_PCI_ID_TABLE_FENTRY(0x6001), /* T6225-CR */
CH_PCI_ID_TABLE_FENTRY(0x6002), /* T6225-SO-CR */
CH_PCI_ID_TABLE_FENTRY(0x6003), /* T6425-CR */
+ CH_PCI_ID_TABLE_FENTRY(0x6004), /* T6425-SO-CR */
CH_PCI_ID_TABLE_FENTRY(0x6005), /* T6225-OCP */
+ CH_PCI_ID_TABLE_FENTRY(0x6006), /* T62100-OCP-SO */
CH_PCI_ID_TABLE_FENTRY(0x6007), /* T62100-LP-CR */
CH_PCI_ID_TABLE_FENTRY(0x6008), /* T62100-SO-CR */
+ CH_PCI_ID_TABLE_FENTRY(0x6009), /* T6210-BT */
CH_PCI_ID_TABLE_FENTRY(0x600d), /* T62100-CR */
CH_PCI_ID_TABLE_FENTRY(0x6011), /* T6225-LL-CR */
CH_PCI_ID_TABLE_FENTRY(0x6014), /* T61100-OCP-SO */
+ CH_PCI_ID_TABLE_FENTRY(0x6015), /* T6201-BT */
CH_PCI_ID_TABLE_FENTRY(0x6080), /* Custom T6225-CR SFP28 */
CH_PCI_ID_TABLE_FENTRY(0x6081), /* Custom T62100-CR */
+ CH_PCI_ID_TABLE_FENTRY(0x6082), /* Custom T6225-CR */
+ CH_PCI_ID_TABLE_FENTRY(0x6083), /* Custom T62100-CR */
+ CH_PCI_ID_TABLE_FENTRY(0x6084), /* Custom T64100-CR */
+ CH_PCI_ID_TABLE_FENTRY(0x6085), /* Custom T6240-SO */
+ CH_PCI_ID_TABLE_FENTRY(0x6086), /* Custom T6225-SO-CR */
+ CH_PCI_ID_TABLE_FENTRY(0x6087), /* Custom T6225-CR */
+ CH_PCI_ID_TABLE_FENTRY(0x6088), /* Custom T62100-CR */
+ CH_PCI_ID_TABLE_FENTRY(0x6089), /* Custom T62100-KR */
+ CH_PCI_ID_TABLE_FENTRY(0x608a), /* Custom T62100-CR */
+ CH_PCI_ID_TABLE_FENTRY(0x608b), /* Custom T6225-CR */
CH_PCI_DEVICE_ID_TABLE_DEFINE_END;
#endif /* CH_PCI_DEVICE_ID_TABLE_DEFINE_BEGIN */
--
2.18.0
^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [dpdk-dev] [PATCH] net/cxgbe: update Chelsio T5/T6 NIC device ids
2019-04-19 9:22 [dpdk-dev] [PATCH] net/cxgbe: update Chelsio T5/T6 NIC device ids Rahul Lakkireddy
2019-04-19 9:22 ` Rahul Lakkireddy
@ 2019-04-19 9:30 ` Ferruh Yigit
2019-04-19 9:30 ` Ferruh Yigit
2019-04-19 9:49 ` Rahul Lakkireddy
2019-04-19 12:24 ` Ferruh Yigit
2019-05-09 8:59 ` Kevin Traynor
3 siblings, 2 replies; 24+ messages in thread
From: Ferruh Yigit @ 2019-04-19 9:30 UTC (permalink / raw)
To: Rahul Lakkireddy, dev
Cc: nirranjan, indranil, stable, Kevin Traynor, Luca Boccassi, Yongseok Koh
On 4/19/2019 10:22 AM, Rahul Lakkireddy wrote:
> Fixes: 04868e5bfddd ("net/cxgbe: add support to run Chelsio T6 cards")
Why this is a fix, isn't this just adding new device ids?
> Cc: stable@dpdk.org
What about keeping 'stable' tag to request the backport but remove the fix?
>
> Signed-off-by: Rahul Lakkireddy <rahul.lakkireddy@chelsio.com>
^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [dpdk-dev] [PATCH] net/cxgbe: update Chelsio T5/T6 NIC device ids
2019-04-19 9:30 ` Ferruh Yigit
@ 2019-04-19 9:30 ` Ferruh Yigit
2019-04-19 9:49 ` Rahul Lakkireddy
1 sibling, 0 replies; 24+ messages in thread
From: Ferruh Yigit @ 2019-04-19 9:30 UTC (permalink / raw)
To: Rahul Lakkireddy, dev
Cc: nirranjan, indranil, stable, Kevin Traynor, Luca Boccassi, Yongseok Koh
On 4/19/2019 10:22 AM, Rahul Lakkireddy wrote:
> Fixes: 04868e5bfddd ("net/cxgbe: add support to run Chelsio T6 cards")
Why this is a fix, isn't this just adding new device ids?
> Cc: stable@dpdk.org
What about keeping 'stable' tag to request the backport but remove the fix?
>
> Signed-off-by: Rahul Lakkireddy <rahul.lakkireddy@chelsio.com>
^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [dpdk-dev] [PATCH] net/cxgbe: update Chelsio T5/T6 NIC device ids
2019-04-19 9:30 ` Ferruh Yigit
2019-04-19 9:30 ` Ferruh Yigit
@ 2019-04-19 9:49 ` Rahul Lakkireddy
2019-04-19 9:49 ` Rahul Lakkireddy
2019-04-19 11:18 ` Ferruh Yigit
1 sibling, 2 replies; 24+ messages in thread
From: Rahul Lakkireddy @ 2019-04-19 9:49 UTC (permalink / raw)
To: Ferruh Yigit
Cc: dev, Nirranjan Kirubaharan, Indranil Choudhury, stable,
Kevin Traynor, Luca Boccassi, Yongseok Koh
Hi Ferruh,
On Friday, April 04/19/19, 2019 at 15:00:32 +0530, Ferruh Yigit wrote:
> On 4/19/2019 10:22 AM, Rahul Lakkireddy wrote:
> > Fixes: 04868e5bfddd ("net/cxgbe: add support to run Chelsio T6 cards")
>
> Why this is a fix, isn't this just adding new device ids?
>
>
> > Cc: stable@dpdk.org
>
> What about keeping 'stable' tag to request the backport but remove the fix?
>
It's fixing probe for these new devices. Let me know if this particular
case doesn't require the fixes line and I will send v2 with only the
stable tag.
> >
> > Signed-off-by: Rahul Lakkireddy <rahul.lakkireddy@chelsio.com>
>
Thanks,
Rahul
^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [dpdk-dev] [PATCH] net/cxgbe: update Chelsio T5/T6 NIC device ids
2019-04-19 9:49 ` Rahul Lakkireddy
@ 2019-04-19 9:49 ` Rahul Lakkireddy
2019-04-19 11:18 ` Ferruh Yigit
1 sibling, 0 replies; 24+ messages in thread
From: Rahul Lakkireddy @ 2019-04-19 9:49 UTC (permalink / raw)
To: Ferruh Yigit
Cc: dev, Nirranjan Kirubaharan, Indranil Choudhury, stable,
Kevin Traynor, Luca Boccassi, Yongseok Koh
Hi Ferruh,
On Friday, April 04/19/19, 2019 at 15:00:32 +0530, Ferruh Yigit wrote:
> On 4/19/2019 10:22 AM, Rahul Lakkireddy wrote:
> > Fixes: 04868e5bfddd ("net/cxgbe: add support to run Chelsio T6 cards")
>
> Why this is a fix, isn't this just adding new device ids?
>
>
> > Cc: stable@dpdk.org
>
> What about keeping 'stable' tag to request the backport but remove the fix?
>
It's fixing probe for these new devices. Let me know if this particular
case doesn't require the fixes line and I will send v2 with only the
stable tag.
> >
> > Signed-off-by: Rahul Lakkireddy <rahul.lakkireddy@chelsio.com>
>
Thanks,
Rahul
^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [dpdk-dev] [PATCH] net/cxgbe: update Chelsio T5/T6 NIC device ids
2019-04-19 9:49 ` Rahul Lakkireddy
2019-04-19 9:49 ` Rahul Lakkireddy
@ 2019-04-19 11:18 ` Ferruh Yigit
2019-04-19 11:18 ` Ferruh Yigit
2019-04-19 12:10 ` Rahul Lakkireddy
1 sibling, 2 replies; 24+ messages in thread
From: Ferruh Yigit @ 2019-04-19 11:18 UTC (permalink / raw)
To: Rahul Lakkireddy
Cc: dev, Nirranjan Kirubaharan, Indranil Choudhury, stable,
Kevin Traynor, Luca Boccassi, Yongseok Koh
On 4/19/2019 10:49 AM, Rahul Lakkireddy wrote:
> Hi Ferruh,
>
> On Friday, April 04/19/19, 2019 at 15:00:32 +0530, Ferruh Yigit wrote:
>> On 4/19/2019 10:22 AM, Rahul Lakkireddy wrote:
>>> Fixes: 04868e5bfddd ("net/cxgbe: add support to run Chelsio T6 cards")
>>
>> Why this is a fix, isn't this just adding new device ids?
>>
>>
>>> Cc: stable@dpdk.org
>>
>> What about keeping 'stable' tag to request the backport but remove the fix?
>>
>
> It's fixing probe for these new devices. Let me know if this particular
> case doesn't require the fixes line and I will send v2 with only the
> stable tag.
No need a v2, if you are agree I can remove while merging.
The commit is not fixing any old code in DPDK, if the intention of making this
patch a fix is to backport the patch, we can do same via stable tag, fix not needed.
If these devices were ready at that time, in 17.08, but forgotten in that commit
to be added, than I see the fix but this looks unlikely since it is almost two
years ago.
>
>>>
>>> Signed-off-by: Rahul Lakkireddy <rahul.lakkireddy@chelsio.com>
>>
>
> Thanks,
> Rahul
>
^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [dpdk-dev] [PATCH] net/cxgbe: update Chelsio T5/T6 NIC device ids
2019-04-19 11:18 ` Ferruh Yigit
@ 2019-04-19 11:18 ` Ferruh Yigit
2019-04-19 12:10 ` Rahul Lakkireddy
1 sibling, 0 replies; 24+ messages in thread
From: Ferruh Yigit @ 2019-04-19 11:18 UTC (permalink / raw)
To: Rahul Lakkireddy
Cc: dev, Nirranjan Kirubaharan, Indranil Choudhury, stable,
Kevin Traynor, Luca Boccassi, Yongseok Koh
On 4/19/2019 10:49 AM, Rahul Lakkireddy wrote:
> Hi Ferruh,
>
> On Friday, April 04/19/19, 2019 at 15:00:32 +0530, Ferruh Yigit wrote:
>> On 4/19/2019 10:22 AM, Rahul Lakkireddy wrote:
>>> Fixes: 04868e5bfddd ("net/cxgbe: add support to run Chelsio T6 cards")
>>
>> Why this is a fix, isn't this just adding new device ids?
>>
>>
>>> Cc: stable@dpdk.org
>>
>> What about keeping 'stable' tag to request the backport but remove the fix?
>>
>
> It's fixing probe for these new devices. Let me know if this particular
> case doesn't require the fixes line and I will send v2 with only the
> stable tag.
No need a v2, if you are agree I can remove while merging.
The commit is not fixing any old code in DPDK, if the intention of making this
patch a fix is to backport the patch, we can do same via stable tag, fix not needed.
If these devices were ready at that time, in 17.08, but forgotten in that commit
to be added, than I see the fix but this looks unlikely since it is almost two
years ago.
>
>>>
>>> Signed-off-by: Rahul Lakkireddy <rahul.lakkireddy@chelsio.com>
>>
>
> Thanks,
> Rahul
>
^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [dpdk-dev] [PATCH] net/cxgbe: update Chelsio T5/T6 NIC device ids
2019-04-19 11:18 ` Ferruh Yigit
2019-04-19 11:18 ` Ferruh Yigit
@ 2019-04-19 12:10 ` Rahul Lakkireddy
2019-04-19 12:10 ` Rahul Lakkireddy
1 sibling, 1 reply; 24+ messages in thread
From: Rahul Lakkireddy @ 2019-04-19 12:10 UTC (permalink / raw)
To: Ferruh Yigit
Cc: dev, Nirranjan Kirubaharan, Indranil Choudhury, stable,
Kevin Traynor, Luca Boccassi, Yongseok Koh
On Friday, April 04/19/19, 2019 at 16:48:55 +0530, Ferruh Yigit wrote:
> On 4/19/2019 10:49 AM, Rahul Lakkireddy wrote:
> > Hi Ferruh,
> >
> > On Friday, April 04/19/19, 2019 at 15:00:32 +0530, Ferruh Yigit wrote:
> >> On 4/19/2019 10:22 AM, Rahul Lakkireddy wrote:
> >>> Fixes: 04868e5bfddd ("net/cxgbe: add support to run Chelsio T6 cards")
> >>
> >> Why this is a fix, isn't this just adding new device ids?
> >>
> >>
> >>> Cc: stable@dpdk.org
> >>
> >> What about keeping 'stable' tag to request the backport but remove the fix?
> >>
> >
> > It's fixing probe for these new devices. Let me know if this particular
> > case doesn't require the fixes line and I will send v2 with only the
> > stable tag.
>
> No need a v2, if you are agree I can remove while merging.
>
> The commit is not fixing any old code in DPDK, if the intention of making this
> patch a fix is to backport the patch, we can do same via stable tag, fix not needed.
> If these devices were ready at that time, in 17.08, but forgotten in that commit
> to be added, than I see the fix but this looks unlikely since it is almost two
> years ago.
>
Ok, please remove the fixes line during merge.
> >
> >>>
> >>> Signed-off-by: Rahul Lakkireddy <rahul.lakkireddy@chelsio.com>
> >>
> >
Thanks,
Rahul
^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [dpdk-dev] [PATCH] net/cxgbe: update Chelsio T5/T6 NIC device ids
2019-04-19 12:10 ` Rahul Lakkireddy
@ 2019-04-19 12:10 ` Rahul Lakkireddy
0 siblings, 0 replies; 24+ messages in thread
From: Rahul Lakkireddy @ 2019-04-19 12:10 UTC (permalink / raw)
To: Ferruh Yigit
Cc: dev, Nirranjan Kirubaharan, Indranil Choudhury, stable,
Kevin Traynor, Luca Boccassi, Yongseok Koh
On Friday, April 04/19/19, 2019 at 16:48:55 +0530, Ferruh Yigit wrote:
> On 4/19/2019 10:49 AM, Rahul Lakkireddy wrote:
> > Hi Ferruh,
> >
> > On Friday, April 04/19/19, 2019 at 15:00:32 +0530, Ferruh Yigit wrote:
> >> On 4/19/2019 10:22 AM, Rahul Lakkireddy wrote:
> >>> Fixes: 04868e5bfddd ("net/cxgbe: add support to run Chelsio T6 cards")
> >>
> >> Why this is a fix, isn't this just adding new device ids?
> >>
> >>
> >>> Cc: stable@dpdk.org
> >>
> >> What about keeping 'stable' tag to request the backport but remove the fix?
> >>
> >
> > It's fixing probe for these new devices. Let me know if this particular
> > case doesn't require the fixes line and I will send v2 with only the
> > stable tag.
>
> No need a v2, if you are agree I can remove while merging.
>
> The commit is not fixing any old code in DPDK, if the intention of making this
> patch a fix is to backport the patch, we can do same via stable tag, fix not needed.
> If these devices were ready at that time, in 17.08, but forgotten in that commit
> to be added, than I see the fix but this looks unlikely since it is almost two
> years ago.
>
Ok, please remove the fixes line during merge.
> >
> >>>
> >>> Signed-off-by: Rahul Lakkireddy <rahul.lakkireddy@chelsio.com>
> >>
> >
Thanks,
Rahul
^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [dpdk-dev] [PATCH] net/cxgbe: update Chelsio T5/T6 NIC device ids
2019-04-19 9:22 [dpdk-dev] [PATCH] net/cxgbe: update Chelsio T5/T6 NIC device ids Rahul Lakkireddy
2019-04-19 9:22 ` Rahul Lakkireddy
2019-04-19 9:30 ` Ferruh Yigit
@ 2019-04-19 12:24 ` Ferruh Yigit
2019-04-19 12:24 ` Ferruh Yigit
2019-05-09 8:59 ` Kevin Traynor
3 siblings, 1 reply; 24+ messages in thread
From: Ferruh Yigit @ 2019-04-19 12:24 UTC (permalink / raw)
To: Rahul Lakkireddy, dev; +Cc: nirranjan, indranil, stable
On 4/19/2019 10:22 AM, Rahul Lakkireddy wrote:
> Fixes: 04868e5bfddd ("net/cxgbe: add support to run Chelsio T6 cards")
> Cc: stable@dpdk.org
>
> Signed-off-by: Rahul Lakkireddy <rahul.lakkireddy@chelsio.com>
Applied to dpdk-next-net/master, thanks.
(Dropped the fixes line while merging)
^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [dpdk-dev] [PATCH] net/cxgbe: update Chelsio T5/T6 NIC device ids
2019-04-19 12:24 ` Ferruh Yigit
@ 2019-04-19 12:24 ` Ferruh Yigit
0 siblings, 0 replies; 24+ messages in thread
From: Ferruh Yigit @ 2019-04-19 12:24 UTC (permalink / raw)
To: Rahul Lakkireddy, dev; +Cc: nirranjan, indranil, stable
On 4/19/2019 10:22 AM, Rahul Lakkireddy wrote:
> Fixes: 04868e5bfddd ("net/cxgbe: add support to run Chelsio T6 cards")
> Cc: stable@dpdk.org
>
> Signed-off-by: Rahul Lakkireddy <rahul.lakkireddy@chelsio.com>
Applied to dpdk-next-net/master, thanks.
(Dropped the fixes line while merging)
^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [dpdk-dev] [PATCH] net/cxgbe: update Chelsio T5/T6 NIC device ids
2019-04-19 9:22 [dpdk-dev] [PATCH] net/cxgbe: update Chelsio T5/T6 NIC device ids Rahul Lakkireddy
` (2 preceding siblings ...)
2019-04-19 12:24 ` Ferruh Yigit
@ 2019-05-09 8:59 ` Kevin Traynor
2019-05-09 8:59 ` Kevin Traynor
2019-05-09 10:03 ` Rahul Lakkireddy
3 siblings, 2 replies; 24+ messages in thread
From: Kevin Traynor @ 2019-05-09 8:59 UTC (permalink / raw)
To: Rahul Lakkireddy, dev
Cc: nirranjan, indranil, stable, Ferruh Yigit, Yongseok Koh, Luca Boccassi
On 19/04/2019 10:22, Rahul Lakkireddy wrote:
> Fixes: 04868e5bfddd ("net/cxgbe: add support to run Chelsio T6 cards")
> Cc: stable@dpdk.org
>
Hi Rahul,
Which stable release(s) is this change requested for? Have you tested
these devices on those branches to ensure they work with the cxgbe pmd
on those branches?
Kevin.
> Signed-off-by: Rahul Lakkireddy <rahul.lakkireddy@chelsio.com>
> ---
> drivers/net/cxgbe/base/t4_pci_id_tbl.h | 52 +++++++++++++++++++++++++-
> 1 file changed, 51 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/net/cxgbe/base/t4_pci_id_tbl.h b/drivers/net/cxgbe/base/t4_pci_id_tbl.h
> index 5f5cbe048..f5f027a2e 100644
> --- a/drivers/net/cxgbe/base/t4_pci_id_tbl.h
> +++ b/drivers/net/cxgbe/base/t4_pci_id_tbl.h
> @@ -1,5 +1,5 @@
> /* SPDX-License-Identifier: BSD-3-Clause
> - * Copyright(c) 2014-2018 Chelsio Communications.
> + * Copyright(c) 2014-2019 Chelsio Communications.
> * All rights reserved.
> */
>
> @@ -103,6 +103,12 @@ CH_PCI_DEVICE_ID_TABLE_DEFINE_BEGIN
> CH_PCI_ID_TABLE_FENTRY(0x5013), /* T580-chr */
> CH_PCI_ID_TABLE_FENTRY(0x5014), /* T580-so */
> CH_PCI_ID_TABLE_FENTRY(0x5015), /* T502-bt */
> + CH_PCI_ID_TABLE_FENTRY(0x5016), /* T580-OCP-SO */
> + CH_PCI_ID_TABLE_FENTRY(0x5017), /* T520-OCP-SO */
> + CH_PCI_ID_TABLE_FENTRY(0x5018), /* T540-BT */
> + CH_PCI_ID_TABLE_FENTRY(0x5019), /* T540-LP-BT */
> + CH_PCI_ID_TABLE_FENTRY(0x501a), /* T540-SO-BT */
> + CH_PCI_ID_TABLE_FENTRY(0x501b), /* T540-SO-CR */
> CH_PCI_ID_TABLE_FENTRY(0x5080), /* Custom T540-cr */
> CH_PCI_ID_TABLE_FENTRY(0x5081), /* Custom T540-LL-cr */
> CH_PCI_ID_TABLE_FENTRY(0x5082), /* Custom T504-cr */
> @@ -116,19 +122,63 @@ CH_PCI_DEVICE_ID_TABLE_DEFINE_BEGIN
> CH_PCI_ID_TABLE_FENTRY(0x5090), /* Custom T540-CR */
> CH_PCI_ID_TABLE_FENTRY(0x5091), /* Custom T522-CR */
> CH_PCI_ID_TABLE_FENTRY(0x5092), /* Custom T520-CR */
> + CH_PCI_ID_TABLE_FENTRY(0x5093), /* Custom T580-LP-CR */
> + CH_PCI_ID_TABLE_FENTRY(0x5094), /* Custom T540-CR */
> + CH_PCI_ID_TABLE_FENTRY(0x5095), /* Custom T540-CR-SO */
> + CH_PCI_ID_TABLE_FENTRY(0x5096), /* Custom T580-CR */
> + CH_PCI_ID_TABLE_FENTRY(0x5097), /* Custom T520-KR */
> + CH_PCI_ID_TABLE_FENTRY(0x5098), /* Custom 2x40G QSFP */
> + CH_PCI_ID_TABLE_FENTRY(0x5099), /* Custom 2x40G QSFP */
> + CH_PCI_ID_TABLE_FENTRY(0x509A), /* Custom T520-CR */
> + CH_PCI_ID_TABLE_FENTRY(0x509B), /* Custom T540-CR LOM */
> + CH_PCI_ID_TABLE_FENTRY(0x509c), /* Custom T520-CR SFP+ LOM */
> + CH_PCI_ID_TABLE_FENTRY(0x509d), /* Custom T540-CR SFP+ */
> + CH_PCI_ID_TABLE_FENTRY(0x509e), /* Custom T520-CR */
> + CH_PCI_ID_TABLE_FENTRY(0x509f), /* Custom T540-CR */
> + CH_PCI_ID_TABLE_FENTRY(0x50a0), /* Custom T540-CR */
> + CH_PCI_ID_TABLE_FENTRY(0x50a1), /* Custom T540-CR */
> + CH_PCI_ID_TABLE_FENTRY(0x50a2), /* Custom T580-KR4 */
> + CH_PCI_ID_TABLE_FENTRY(0x50a3), /* Custom T580-KR4 */
> + CH_PCI_ID_TABLE_FENTRY(0x50a4), /* Custom 2x T540-CR */
> + CH_PCI_ID_TABLE_FENTRY(0x50a5), /* Custom T522-BT */
> + CH_PCI_ID_TABLE_FENTRY(0x50a6), /* Custom T522-BT-SO */
> + CH_PCI_ID_TABLE_FENTRY(0x50a7), /* Custom T580-CR */
> + CH_PCI_ID_TABLE_FENTRY(0x50a8), /* Custom T580-KR */
> + CH_PCI_ID_TABLE_FENTRY(0x50a9), /* Custom T580-KR */
> + CH_PCI_ID_TABLE_FENTRY(0x50aa), /* Custom T580-CR */
> + CH_PCI_ID_TABLE_FENTRY(0x50ab), /* Custom T520-CR */
> + CH_PCI_ID_TABLE_FENTRY(0x50ac), /* Custom T540-BT */
> + CH_PCI_ID_TABLE_FENTRY(0x50ad), /* Custom T520-CR */
> + CH_PCI_ID_TABLE_FENTRY(0x50ae), /* Custom T540-XL-SO */
> + CH_PCI_ID_TABLE_FENTRY(0x50af), /* Custom T580-KR-SO */
> + CH_PCI_ID_TABLE_FENTRY(0x50b0), /* Custom T520-CR-LOM */
>
> /* T6 adapter */
> CH_PCI_ID_TABLE_FENTRY(0x6001), /* T6225-CR */
> CH_PCI_ID_TABLE_FENTRY(0x6002), /* T6225-SO-CR */
> CH_PCI_ID_TABLE_FENTRY(0x6003), /* T6425-CR */
> + CH_PCI_ID_TABLE_FENTRY(0x6004), /* T6425-SO-CR */
> CH_PCI_ID_TABLE_FENTRY(0x6005), /* T6225-OCP */
> + CH_PCI_ID_TABLE_FENTRY(0x6006), /* T62100-OCP-SO */
> CH_PCI_ID_TABLE_FENTRY(0x6007), /* T62100-LP-CR */
> CH_PCI_ID_TABLE_FENTRY(0x6008), /* T62100-SO-CR */
> + CH_PCI_ID_TABLE_FENTRY(0x6009), /* T6210-BT */
> CH_PCI_ID_TABLE_FENTRY(0x600d), /* T62100-CR */
> CH_PCI_ID_TABLE_FENTRY(0x6011), /* T6225-LL-CR */
> CH_PCI_ID_TABLE_FENTRY(0x6014), /* T61100-OCP-SO */
> + CH_PCI_ID_TABLE_FENTRY(0x6015), /* T6201-BT */
> CH_PCI_ID_TABLE_FENTRY(0x6080), /* Custom T6225-CR SFP28 */
> CH_PCI_ID_TABLE_FENTRY(0x6081), /* Custom T62100-CR */
> + CH_PCI_ID_TABLE_FENTRY(0x6082), /* Custom T6225-CR */
> + CH_PCI_ID_TABLE_FENTRY(0x6083), /* Custom T62100-CR */
> + CH_PCI_ID_TABLE_FENTRY(0x6084), /* Custom T64100-CR */
> + CH_PCI_ID_TABLE_FENTRY(0x6085), /* Custom T6240-SO */
> + CH_PCI_ID_TABLE_FENTRY(0x6086), /* Custom T6225-SO-CR */
> + CH_PCI_ID_TABLE_FENTRY(0x6087), /* Custom T6225-CR */
> + CH_PCI_ID_TABLE_FENTRY(0x6088), /* Custom T62100-CR */
> + CH_PCI_ID_TABLE_FENTRY(0x6089), /* Custom T62100-KR */
> + CH_PCI_ID_TABLE_FENTRY(0x608a), /* Custom T62100-CR */
> + CH_PCI_ID_TABLE_FENTRY(0x608b), /* Custom T6225-CR */
> CH_PCI_DEVICE_ID_TABLE_DEFINE_END;
>
> #endif /* CH_PCI_DEVICE_ID_TABLE_DEFINE_BEGIN */
>
^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [dpdk-dev] [PATCH] net/cxgbe: update Chelsio T5/T6 NIC device ids
2019-05-09 8:59 ` Kevin Traynor
@ 2019-05-09 8:59 ` Kevin Traynor
2019-05-09 10:03 ` Rahul Lakkireddy
1 sibling, 0 replies; 24+ messages in thread
From: Kevin Traynor @ 2019-05-09 8:59 UTC (permalink / raw)
To: Rahul Lakkireddy, dev
Cc: nirranjan, indranil, stable, Ferruh Yigit, Yongseok Koh, Luca Boccassi
On 19/04/2019 10:22, Rahul Lakkireddy wrote:
> Fixes: 04868e5bfddd ("net/cxgbe: add support to run Chelsio T6 cards")
> Cc: stable@dpdk.org
>
Hi Rahul,
Which stable release(s) is this change requested for? Have you tested
these devices on those branches to ensure they work with the cxgbe pmd
on those branches?
Kevin.
> Signed-off-by: Rahul Lakkireddy <rahul.lakkireddy@chelsio.com>
> ---
> drivers/net/cxgbe/base/t4_pci_id_tbl.h | 52 +++++++++++++++++++++++++-
> 1 file changed, 51 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/net/cxgbe/base/t4_pci_id_tbl.h b/drivers/net/cxgbe/base/t4_pci_id_tbl.h
> index 5f5cbe048..f5f027a2e 100644
> --- a/drivers/net/cxgbe/base/t4_pci_id_tbl.h
> +++ b/drivers/net/cxgbe/base/t4_pci_id_tbl.h
> @@ -1,5 +1,5 @@
> /* SPDX-License-Identifier: BSD-3-Clause
> - * Copyright(c) 2014-2018 Chelsio Communications.
> + * Copyright(c) 2014-2019 Chelsio Communications.
> * All rights reserved.
> */
>
> @@ -103,6 +103,12 @@ CH_PCI_DEVICE_ID_TABLE_DEFINE_BEGIN
> CH_PCI_ID_TABLE_FENTRY(0x5013), /* T580-chr */
> CH_PCI_ID_TABLE_FENTRY(0x5014), /* T580-so */
> CH_PCI_ID_TABLE_FENTRY(0x5015), /* T502-bt */
> + CH_PCI_ID_TABLE_FENTRY(0x5016), /* T580-OCP-SO */
> + CH_PCI_ID_TABLE_FENTRY(0x5017), /* T520-OCP-SO */
> + CH_PCI_ID_TABLE_FENTRY(0x5018), /* T540-BT */
> + CH_PCI_ID_TABLE_FENTRY(0x5019), /* T540-LP-BT */
> + CH_PCI_ID_TABLE_FENTRY(0x501a), /* T540-SO-BT */
> + CH_PCI_ID_TABLE_FENTRY(0x501b), /* T540-SO-CR */
> CH_PCI_ID_TABLE_FENTRY(0x5080), /* Custom T540-cr */
> CH_PCI_ID_TABLE_FENTRY(0x5081), /* Custom T540-LL-cr */
> CH_PCI_ID_TABLE_FENTRY(0x5082), /* Custom T504-cr */
> @@ -116,19 +122,63 @@ CH_PCI_DEVICE_ID_TABLE_DEFINE_BEGIN
> CH_PCI_ID_TABLE_FENTRY(0x5090), /* Custom T540-CR */
> CH_PCI_ID_TABLE_FENTRY(0x5091), /* Custom T522-CR */
> CH_PCI_ID_TABLE_FENTRY(0x5092), /* Custom T520-CR */
> + CH_PCI_ID_TABLE_FENTRY(0x5093), /* Custom T580-LP-CR */
> + CH_PCI_ID_TABLE_FENTRY(0x5094), /* Custom T540-CR */
> + CH_PCI_ID_TABLE_FENTRY(0x5095), /* Custom T540-CR-SO */
> + CH_PCI_ID_TABLE_FENTRY(0x5096), /* Custom T580-CR */
> + CH_PCI_ID_TABLE_FENTRY(0x5097), /* Custom T520-KR */
> + CH_PCI_ID_TABLE_FENTRY(0x5098), /* Custom 2x40G QSFP */
> + CH_PCI_ID_TABLE_FENTRY(0x5099), /* Custom 2x40G QSFP */
> + CH_PCI_ID_TABLE_FENTRY(0x509A), /* Custom T520-CR */
> + CH_PCI_ID_TABLE_FENTRY(0x509B), /* Custom T540-CR LOM */
> + CH_PCI_ID_TABLE_FENTRY(0x509c), /* Custom T520-CR SFP+ LOM */
> + CH_PCI_ID_TABLE_FENTRY(0x509d), /* Custom T540-CR SFP+ */
> + CH_PCI_ID_TABLE_FENTRY(0x509e), /* Custom T520-CR */
> + CH_PCI_ID_TABLE_FENTRY(0x509f), /* Custom T540-CR */
> + CH_PCI_ID_TABLE_FENTRY(0x50a0), /* Custom T540-CR */
> + CH_PCI_ID_TABLE_FENTRY(0x50a1), /* Custom T540-CR */
> + CH_PCI_ID_TABLE_FENTRY(0x50a2), /* Custom T580-KR4 */
> + CH_PCI_ID_TABLE_FENTRY(0x50a3), /* Custom T580-KR4 */
> + CH_PCI_ID_TABLE_FENTRY(0x50a4), /* Custom 2x T540-CR */
> + CH_PCI_ID_TABLE_FENTRY(0x50a5), /* Custom T522-BT */
> + CH_PCI_ID_TABLE_FENTRY(0x50a6), /* Custom T522-BT-SO */
> + CH_PCI_ID_TABLE_FENTRY(0x50a7), /* Custom T580-CR */
> + CH_PCI_ID_TABLE_FENTRY(0x50a8), /* Custom T580-KR */
> + CH_PCI_ID_TABLE_FENTRY(0x50a9), /* Custom T580-KR */
> + CH_PCI_ID_TABLE_FENTRY(0x50aa), /* Custom T580-CR */
> + CH_PCI_ID_TABLE_FENTRY(0x50ab), /* Custom T520-CR */
> + CH_PCI_ID_TABLE_FENTRY(0x50ac), /* Custom T540-BT */
> + CH_PCI_ID_TABLE_FENTRY(0x50ad), /* Custom T520-CR */
> + CH_PCI_ID_TABLE_FENTRY(0x50ae), /* Custom T540-XL-SO */
> + CH_PCI_ID_TABLE_FENTRY(0x50af), /* Custom T580-KR-SO */
> + CH_PCI_ID_TABLE_FENTRY(0x50b0), /* Custom T520-CR-LOM */
>
> /* T6 adapter */
> CH_PCI_ID_TABLE_FENTRY(0x6001), /* T6225-CR */
> CH_PCI_ID_TABLE_FENTRY(0x6002), /* T6225-SO-CR */
> CH_PCI_ID_TABLE_FENTRY(0x6003), /* T6425-CR */
> + CH_PCI_ID_TABLE_FENTRY(0x6004), /* T6425-SO-CR */
> CH_PCI_ID_TABLE_FENTRY(0x6005), /* T6225-OCP */
> + CH_PCI_ID_TABLE_FENTRY(0x6006), /* T62100-OCP-SO */
> CH_PCI_ID_TABLE_FENTRY(0x6007), /* T62100-LP-CR */
> CH_PCI_ID_TABLE_FENTRY(0x6008), /* T62100-SO-CR */
> + CH_PCI_ID_TABLE_FENTRY(0x6009), /* T6210-BT */
> CH_PCI_ID_TABLE_FENTRY(0x600d), /* T62100-CR */
> CH_PCI_ID_TABLE_FENTRY(0x6011), /* T6225-LL-CR */
> CH_PCI_ID_TABLE_FENTRY(0x6014), /* T61100-OCP-SO */
> + CH_PCI_ID_TABLE_FENTRY(0x6015), /* T6201-BT */
> CH_PCI_ID_TABLE_FENTRY(0x6080), /* Custom T6225-CR SFP28 */
> CH_PCI_ID_TABLE_FENTRY(0x6081), /* Custom T62100-CR */
> + CH_PCI_ID_TABLE_FENTRY(0x6082), /* Custom T6225-CR */
> + CH_PCI_ID_TABLE_FENTRY(0x6083), /* Custom T62100-CR */
> + CH_PCI_ID_TABLE_FENTRY(0x6084), /* Custom T64100-CR */
> + CH_PCI_ID_TABLE_FENTRY(0x6085), /* Custom T6240-SO */
> + CH_PCI_ID_TABLE_FENTRY(0x6086), /* Custom T6225-SO-CR */
> + CH_PCI_ID_TABLE_FENTRY(0x6087), /* Custom T6225-CR */
> + CH_PCI_ID_TABLE_FENTRY(0x6088), /* Custom T62100-CR */
> + CH_PCI_ID_TABLE_FENTRY(0x6089), /* Custom T62100-KR */
> + CH_PCI_ID_TABLE_FENTRY(0x608a), /* Custom T62100-CR */
> + CH_PCI_ID_TABLE_FENTRY(0x608b), /* Custom T6225-CR */
> CH_PCI_DEVICE_ID_TABLE_DEFINE_END;
>
> #endif /* CH_PCI_DEVICE_ID_TABLE_DEFINE_BEGIN */
>
^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [dpdk-dev] [PATCH] net/cxgbe: update Chelsio T5/T6 NIC device ids
2019-05-09 8:59 ` Kevin Traynor
2019-05-09 8:59 ` Kevin Traynor
@ 2019-05-09 10:03 ` Rahul Lakkireddy
2019-05-09 10:03 ` Rahul Lakkireddy
2019-05-09 10:38 ` Kevin Traynor
1 sibling, 2 replies; 24+ messages in thread
From: Rahul Lakkireddy @ 2019-05-09 10:03 UTC (permalink / raw)
To: Kevin Traynor
Cc: dev, Nirranjan Kirubaharan, Indranil Choudhury, stable,
Ferruh Yigit, Yongseok Koh, Luca Boccassi
On Thursday, May 05/09/19, 2019 at 14:29:05 +0530, Kevin Traynor wrote:
> On 19/04/2019 10:22, Rahul Lakkireddy wrote:
> > Fixes: 04868e5bfddd ("net/cxgbe: add support to run Chelsio T6 cards")
> > Cc: stable@dpdk.org
> >
>
> Hi Rahul,
>
> Which stable release(s) is this change requested for? Have you tested
> these devices on those branches to ensure they work with the cxgbe pmd
> on those branches?
>
> Kevin.
Hi Kevin,
Please target 18.11 and 19.02 branches. All these devices work fine
in these branches.
Thanks,
Rahul
>
> > Signed-off-by: Rahul Lakkireddy <rahul.lakkireddy@chelsio.com>
> > ---
> > drivers/net/cxgbe/base/t4_pci_id_tbl.h | 52 +++++++++++++++++++++++++-
> > 1 file changed, 51 insertions(+), 1 deletion(-)
> >
> > diff --git a/drivers/net/cxgbe/base/t4_pci_id_tbl.h b/drivers/net/cxgbe/base/t4_pci_id_tbl.h
> > index 5f5cbe048..f5f027a2e 100644
> > --- a/drivers/net/cxgbe/base/t4_pci_id_tbl.h
> > +++ b/drivers/net/cxgbe/base/t4_pci_id_tbl.h
> > @@ -1,5 +1,5 @@
> > /* SPDX-License-Identifier: BSD-3-Clause
> > - * Copyright(c) 2014-2018 Chelsio Communications.
> > + * Copyright(c) 2014-2019 Chelsio Communications.
> > * All rights reserved.
> > */
> >
> > @@ -103,6 +103,12 @@ CH_PCI_DEVICE_ID_TABLE_DEFINE_BEGIN
> > CH_PCI_ID_TABLE_FENTRY(0x5013), /* T580-chr */
> > CH_PCI_ID_TABLE_FENTRY(0x5014), /* T580-so */
> > CH_PCI_ID_TABLE_FENTRY(0x5015), /* T502-bt */
> > + CH_PCI_ID_TABLE_FENTRY(0x5016), /* T580-OCP-SO */
> > + CH_PCI_ID_TABLE_FENTRY(0x5017), /* T520-OCP-SO */
> > + CH_PCI_ID_TABLE_FENTRY(0x5018), /* T540-BT */
> > + CH_PCI_ID_TABLE_FENTRY(0x5019), /* T540-LP-BT */
> > + CH_PCI_ID_TABLE_FENTRY(0x501a), /* T540-SO-BT */
> > + CH_PCI_ID_TABLE_FENTRY(0x501b), /* T540-SO-CR */
> > CH_PCI_ID_TABLE_FENTRY(0x5080), /* Custom T540-cr */
> > CH_PCI_ID_TABLE_FENTRY(0x5081), /* Custom T540-LL-cr */
> > CH_PCI_ID_TABLE_FENTRY(0x5082), /* Custom T504-cr */
> > @@ -116,19 +122,63 @@ CH_PCI_DEVICE_ID_TABLE_DEFINE_BEGIN
> > CH_PCI_ID_TABLE_FENTRY(0x5090), /* Custom T540-CR */
> > CH_PCI_ID_TABLE_FENTRY(0x5091), /* Custom T522-CR */
> > CH_PCI_ID_TABLE_FENTRY(0x5092), /* Custom T520-CR */
> > + CH_PCI_ID_TABLE_FENTRY(0x5093), /* Custom T580-LP-CR */
> > + CH_PCI_ID_TABLE_FENTRY(0x5094), /* Custom T540-CR */
> > + CH_PCI_ID_TABLE_FENTRY(0x5095), /* Custom T540-CR-SO */
> > + CH_PCI_ID_TABLE_FENTRY(0x5096), /* Custom T580-CR */
> > + CH_PCI_ID_TABLE_FENTRY(0x5097), /* Custom T520-KR */
> > + CH_PCI_ID_TABLE_FENTRY(0x5098), /* Custom 2x40G QSFP */
> > + CH_PCI_ID_TABLE_FENTRY(0x5099), /* Custom 2x40G QSFP */
> > + CH_PCI_ID_TABLE_FENTRY(0x509A), /* Custom T520-CR */
> > + CH_PCI_ID_TABLE_FENTRY(0x509B), /* Custom T540-CR LOM */
> > + CH_PCI_ID_TABLE_FENTRY(0x509c), /* Custom T520-CR SFP+ LOM */
> > + CH_PCI_ID_TABLE_FENTRY(0x509d), /* Custom T540-CR SFP+ */
> > + CH_PCI_ID_TABLE_FENTRY(0x509e), /* Custom T520-CR */
> > + CH_PCI_ID_TABLE_FENTRY(0x509f), /* Custom T540-CR */
> > + CH_PCI_ID_TABLE_FENTRY(0x50a0), /* Custom T540-CR */
> > + CH_PCI_ID_TABLE_FENTRY(0x50a1), /* Custom T540-CR */
> > + CH_PCI_ID_TABLE_FENTRY(0x50a2), /* Custom T580-KR4 */
> > + CH_PCI_ID_TABLE_FENTRY(0x50a3), /* Custom T580-KR4 */
> > + CH_PCI_ID_TABLE_FENTRY(0x50a4), /* Custom 2x T540-CR */
> > + CH_PCI_ID_TABLE_FENTRY(0x50a5), /* Custom T522-BT */
> > + CH_PCI_ID_TABLE_FENTRY(0x50a6), /* Custom T522-BT-SO */
> > + CH_PCI_ID_TABLE_FENTRY(0x50a7), /* Custom T580-CR */
> > + CH_PCI_ID_TABLE_FENTRY(0x50a8), /* Custom T580-KR */
> > + CH_PCI_ID_TABLE_FENTRY(0x50a9), /* Custom T580-KR */
> > + CH_PCI_ID_TABLE_FENTRY(0x50aa), /* Custom T580-CR */
> > + CH_PCI_ID_TABLE_FENTRY(0x50ab), /* Custom T520-CR */
> > + CH_PCI_ID_TABLE_FENTRY(0x50ac), /* Custom T540-BT */
> > + CH_PCI_ID_TABLE_FENTRY(0x50ad), /* Custom T520-CR */
> > + CH_PCI_ID_TABLE_FENTRY(0x50ae), /* Custom T540-XL-SO */
> > + CH_PCI_ID_TABLE_FENTRY(0x50af), /* Custom T580-KR-SO */
> > + CH_PCI_ID_TABLE_FENTRY(0x50b0), /* Custom T520-CR-LOM */
> >
> > /* T6 adapter */
> > CH_PCI_ID_TABLE_FENTRY(0x6001), /* T6225-CR */
> > CH_PCI_ID_TABLE_FENTRY(0x6002), /* T6225-SO-CR */
> > CH_PCI_ID_TABLE_FENTRY(0x6003), /* T6425-CR */
> > + CH_PCI_ID_TABLE_FENTRY(0x6004), /* T6425-SO-CR */
> > CH_PCI_ID_TABLE_FENTRY(0x6005), /* T6225-OCP */
> > + CH_PCI_ID_TABLE_FENTRY(0x6006), /* T62100-OCP-SO */
> > CH_PCI_ID_TABLE_FENTRY(0x6007), /* T62100-LP-CR */
> > CH_PCI_ID_TABLE_FENTRY(0x6008), /* T62100-SO-CR */
> > + CH_PCI_ID_TABLE_FENTRY(0x6009), /* T6210-BT */
> > CH_PCI_ID_TABLE_FENTRY(0x600d), /* T62100-CR */
> > CH_PCI_ID_TABLE_FENTRY(0x6011), /* T6225-LL-CR */
> > CH_PCI_ID_TABLE_FENTRY(0x6014), /* T61100-OCP-SO */
> > + CH_PCI_ID_TABLE_FENTRY(0x6015), /* T6201-BT */
> > CH_PCI_ID_TABLE_FENTRY(0x6080), /* Custom T6225-CR SFP28 */
> > CH_PCI_ID_TABLE_FENTRY(0x6081), /* Custom T62100-CR */
> > + CH_PCI_ID_TABLE_FENTRY(0x6082), /* Custom T6225-CR */
> > + CH_PCI_ID_TABLE_FENTRY(0x6083), /* Custom T62100-CR */
> > + CH_PCI_ID_TABLE_FENTRY(0x6084), /* Custom T64100-CR */
> > + CH_PCI_ID_TABLE_FENTRY(0x6085), /* Custom T6240-SO */
> > + CH_PCI_ID_TABLE_FENTRY(0x6086), /* Custom T6225-SO-CR */
> > + CH_PCI_ID_TABLE_FENTRY(0x6087), /* Custom T6225-CR */
> > + CH_PCI_ID_TABLE_FENTRY(0x6088), /* Custom T62100-CR */
> > + CH_PCI_ID_TABLE_FENTRY(0x6089), /* Custom T62100-KR */
> > + CH_PCI_ID_TABLE_FENTRY(0x608a), /* Custom T62100-CR */
> > + CH_PCI_ID_TABLE_FENTRY(0x608b), /* Custom T6225-CR */
> > CH_PCI_DEVICE_ID_TABLE_DEFINE_END;
> >
> > #endif /* CH_PCI_DEVICE_ID_TABLE_DEFINE_BEGIN */
> >
>
^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [dpdk-dev] [PATCH] net/cxgbe: update Chelsio T5/T6 NIC device ids
2019-05-09 10:03 ` Rahul Lakkireddy
@ 2019-05-09 10:03 ` Rahul Lakkireddy
2019-05-09 10:38 ` Kevin Traynor
1 sibling, 0 replies; 24+ messages in thread
From: Rahul Lakkireddy @ 2019-05-09 10:03 UTC (permalink / raw)
To: Kevin Traynor
Cc: dev, Nirranjan Kirubaharan, Indranil Choudhury, stable,
Ferruh Yigit, Yongseok Koh, Luca Boccassi
On Thursday, May 05/09/19, 2019 at 14:29:05 +0530, Kevin Traynor wrote:
> On 19/04/2019 10:22, Rahul Lakkireddy wrote:
> > Fixes: 04868e5bfddd ("net/cxgbe: add support to run Chelsio T6 cards")
> > Cc: stable@dpdk.org
> >
>
> Hi Rahul,
>
> Which stable release(s) is this change requested for? Have you tested
> these devices on those branches to ensure they work with the cxgbe pmd
> on those branches?
>
> Kevin.
Hi Kevin,
Please target 18.11 and 19.02 branches. All these devices work fine
in these branches.
Thanks,
Rahul
>
> > Signed-off-by: Rahul Lakkireddy <rahul.lakkireddy@chelsio.com>
> > ---
> > drivers/net/cxgbe/base/t4_pci_id_tbl.h | 52 +++++++++++++++++++++++++-
> > 1 file changed, 51 insertions(+), 1 deletion(-)
> >
> > diff --git a/drivers/net/cxgbe/base/t4_pci_id_tbl.h b/drivers/net/cxgbe/base/t4_pci_id_tbl.h
> > index 5f5cbe048..f5f027a2e 100644
> > --- a/drivers/net/cxgbe/base/t4_pci_id_tbl.h
> > +++ b/drivers/net/cxgbe/base/t4_pci_id_tbl.h
> > @@ -1,5 +1,5 @@
> > /* SPDX-License-Identifier: BSD-3-Clause
> > - * Copyright(c) 2014-2018 Chelsio Communications.
> > + * Copyright(c) 2014-2019 Chelsio Communications.
> > * All rights reserved.
> > */
> >
> > @@ -103,6 +103,12 @@ CH_PCI_DEVICE_ID_TABLE_DEFINE_BEGIN
> > CH_PCI_ID_TABLE_FENTRY(0x5013), /* T580-chr */
> > CH_PCI_ID_TABLE_FENTRY(0x5014), /* T580-so */
> > CH_PCI_ID_TABLE_FENTRY(0x5015), /* T502-bt */
> > + CH_PCI_ID_TABLE_FENTRY(0x5016), /* T580-OCP-SO */
> > + CH_PCI_ID_TABLE_FENTRY(0x5017), /* T520-OCP-SO */
> > + CH_PCI_ID_TABLE_FENTRY(0x5018), /* T540-BT */
> > + CH_PCI_ID_TABLE_FENTRY(0x5019), /* T540-LP-BT */
> > + CH_PCI_ID_TABLE_FENTRY(0x501a), /* T540-SO-BT */
> > + CH_PCI_ID_TABLE_FENTRY(0x501b), /* T540-SO-CR */
> > CH_PCI_ID_TABLE_FENTRY(0x5080), /* Custom T540-cr */
> > CH_PCI_ID_TABLE_FENTRY(0x5081), /* Custom T540-LL-cr */
> > CH_PCI_ID_TABLE_FENTRY(0x5082), /* Custom T504-cr */
> > @@ -116,19 +122,63 @@ CH_PCI_DEVICE_ID_TABLE_DEFINE_BEGIN
> > CH_PCI_ID_TABLE_FENTRY(0x5090), /* Custom T540-CR */
> > CH_PCI_ID_TABLE_FENTRY(0x5091), /* Custom T522-CR */
> > CH_PCI_ID_TABLE_FENTRY(0x5092), /* Custom T520-CR */
> > + CH_PCI_ID_TABLE_FENTRY(0x5093), /* Custom T580-LP-CR */
> > + CH_PCI_ID_TABLE_FENTRY(0x5094), /* Custom T540-CR */
> > + CH_PCI_ID_TABLE_FENTRY(0x5095), /* Custom T540-CR-SO */
> > + CH_PCI_ID_TABLE_FENTRY(0x5096), /* Custom T580-CR */
> > + CH_PCI_ID_TABLE_FENTRY(0x5097), /* Custom T520-KR */
> > + CH_PCI_ID_TABLE_FENTRY(0x5098), /* Custom 2x40G QSFP */
> > + CH_PCI_ID_TABLE_FENTRY(0x5099), /* Custom 2x40G QSFP */
> > + CH_PCI_ID_TABLE_FENTRY(0x509A), /* Custom T520-CR */
> > + CH_PCI_ID_TABLE_FENTRY(0x509B), /* Custom T540-CR LOM */
> > + CH_PCI_ID_TABLE_FENTRY(0x509c), /* Custom T520-CR SFP+ LOM */
> > + CH_PCI_ID_TABLE_FENTRY(0x509d), /* Custom T540-CR SFP+ */
> > + CH_PCI_ID_TABLE_FENTRY(0x509e), /* Custom T520-CR */
> > + CH_PCI_ID_TABLE_FENTRY(0x509f), /* Custom T540-CR */
> > + CH_PCI_ID_TABLE_FENTRY(0x50a0), /* Custom T540-CR */
> > + CH_PCI_ID_TABLE_FENTRY(0x50a1), /* Custom T540-CR */
> > + CH_PCI_ID_TABLE_FENTRY(0x50a2), /* Custom T580-KR4 */
> > + CH_PCI_ID_TABLE_FENTRY(0x50a3), /* Custom T580-KR4 */
> > + CH_PCI_ID_TABLE_FENTRY(0x50a4), /* Custom 2x T540-CR */
> > + CH_PCI_ID_TABLE_FENTRY(0x50a5), /* Custom T522-BT */
> > + CH_PCI_ID_TABLE_FENTRY(0x50a6), /* Custom T522-BT-SO */
> > + CH_PCI_ID_TABLE_FENTRY(0x50a7), /* Custom T580-CR */
> > + CH_PCI_ID_TABLE_FENTRY(0x50a8), /* Custom T580-KR */
> > + CH_PCI_ID_TABLE_FENTRY(0x50a9), /* Custom T580-KR */
> > + CH_PCI_ID_TABLE_FENTRY(0x50aa), /* Custom T580-CR */
> > + CH_PCI_ID_TABLE_FENTRY(0x50ab), /* Custom T520-CR */
> > + CH_PCI_ID_TABLE_FENTRY(0x50ac), /* Custom T540-BT */
> > + CH_PCI_ID_TABLE_FENTRY(0x50ad), /* Custom T520-CR */
> > + CH_PCI_ID_TABLE_FENTRY(0x50ae), /* Custom T540-XL-SO */
> > + CH_PCI_ID_TABLE_FENTRY(0x50af), /* Custom T580-KR-SO */
> > + CH_PCI_ID_TABLE_FENTRY(0x50b0), /* Custom T520-CR-LOM */
> >
> > /* T6 adapter */
> > CH_PCI_ID_TABLE_FENTRY(0x6001), /* T6225-CR */
> > CH_PCI_ID_TABLE_FENTRY(0x6002), /* T6225-SO-CR */
> > CH_PCI_ID_TABLE_FENTRY(0x6003), /* T6425-CR */
> > + CH_PCI_ID_TABLE_FENTRY(0x6004), /* T6425-SO-CR */
> > CH_PCI_ID_TABLE_FENTRY(0x6005), /* T6225-OCP */
> > + CH_PCI_ID_TABLE_FENTRY(0x6006), /* T62100-OCP-SO */
> > CH_PCI_ID_TABLE_FENTRY(0x6007), /* T62100-LP-CR */
> > CH_PCI_ID_TABLE_FENTRY(0x6008), /* T62100-SO-CR */
> > + CH_PCI_ID_TABLE_FENTRY(0x6009), /* T6210-BT */
> > CH_PCI_ID_TABLE_FENTRY(0x600d), /* T62100-CR */
> > CH_PCI_ID_TABLE_FENTRY(0x6011), /* T6225-LL-CR */
> > CH_PCI_ID_TABLE_FENTRY(0x6014), /* T61100-OCP-SO */
> > + CH_PCI_ID_TABLE_FENTRY(0x6015), /* T6201-BT */
> > CH_PCI_ID_TABLE_FENTRY(0x6080), /* Custom T6225-CR SFP28 */
> > CH_PCI_ID_TABLE_FENTRY(0x6081), /* Custom T62100-CR */
> > + CH_PCI_ID_TABLE_FENTRY(0x6082), /* Custom T6225-CR */
> > + CH_PCI_ID_TABLE_FENTRY(0x6083), /* Custom T62100-CR */
> > + CH_PCI_ID_TABLE_FENTRY(0x6084), /* Custom T64100-CR */
> > + CH_PCI_ID_TABLE_FENTRY(0x6085), /* Custom T6240-SO */
> > + CH_PCI_ID_TABLE_FENTRY(0x6086), /* Custom T6225-SO-CR */
> > + CH_PCI_ID_TABLE_FENTRY(0x6087), /* Custom T6225-CR */
> > + CH_PCI_ID_TABLE_FENTRY(0x6088), /* Custom T62100-CR */
> > + CH_PCI_ID_TABLE_FENTRY(0x6089), /* Custom T62100-KR */
> > + CH_PCI_ID_TABLE_FENTRY(0x608a), /* Custom T62100-CR */
> > + CH_PCI_ID_TABLE_FENTRY(0x608b), /* Custom T6225-CR */
> > CH_PCI_DEVICE_ID_TABLE_DEFINE_END;
> >
> > #endif /* CH_PCI_DEVICE_ID_TABLE_DEFINE_BEGIN */
> >
>
^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [dpdk-dev] [PATCH] net/cxgbe: update Chelsio T5/T6 NIC device ids
2019-05-09 10:03 ` Rahul Lakkireddy
2019-05-09 10:03 ` Rahul Lakkireddy
@ 2019-05-09 10:38 ` Kevin Traynor
2019-05-09 10:38 ` Kevin Traynor
2019-05-09 11:11 ` Rahul Lakkireddy
1 sibling, 2 replies; 24+ messages in thread
From: Kevin Traynor @ 2019-05-09 10:38 UTC (permalink / raw)
To: Rahul Lakkireddy
Cc: dev, Nirranjan Kirubaharan, Indranil Choudhury, stable,
Ferruh Yigit, Yongseok Koh, Luca Boccassi
On 09/05/2019 11:03, Rahul Lakkireddy wrote:
> On Thursday, May 05/09/19, 2019 at 14:29:05 +0530, Kevin Traynor wrote:
>> On 19/04/2019 10:22, Rahul Lakkireddy wrote:
>>> Fixes: 04868e5bfddd ("net/cxgbe: add support to run Chelsio T6 cards")
>>> Cc: stable@dpdk.org
>>>
>>
>> Hi Rahul,
>>
>> Which stable release(s) is this change requested for? Have you tested
>> these devices on those branches to ensure they work with the cxgbe pmd
>> on those branches?
>>
>> Kevin.
>
> Hi Kevin,
>
> Please target 18.11 and 19.02 branches. All these devices work fine
> in these branches.
>
Sorry, but this is not clear. There is no 19.02 branch and for 18.11
branch, I don't know if you tested some/all of the devices with that
branch or are making assumption based on the cxgbe pmd in 18.11.
I think these need to have some testing to be in 18.11.2. I will apply
it, but requesting you to do some testing to check they're working as
you expect when 18.11.2 RC is available (ETA for RC-1, week of 20th May).
thanks,
Kevin.
> Thanks,
> Rahul
>
>>
>>> Signed-off-by: Rahul Lakkireddy <rahul.lakkireddy@chelsio.com>
>>> ---
>>> drivers/net/cxgbe/base/t4_pci_id_tbl.h | 52 +++++++++++++++++++++++++-
>>> 1 file changed, 51 insertions(+), 1 deletion(-)
>>>
>>> diff --git a/drivers/net/cxgbe/base/t4_pci_id_tbl.h b/drivers/net/cxgbe/base/t4_pci_id_tbl.h
>>> index 5f5cbe048..f5f027a2e 100644
>>> --- a/drivers/net/cxgbe/base/t4_pci_id_tbl.h
>>> +++ b/drivers/net/cxgbe/base/t4_pci_id_tbl.h
>>> @@ -1,5 +1,5 @@
>>> /* SPDX-License-Identifier: BSD-3-Clause
>>> - * Copyright(c) 2014-2018 Chelsio Communications.
>>> + * Copyright(c) 2014-2019 Chelsio Communications.
>>> * All rights reserved.
>>> */
>>>
>>> @@ -103,6 +103,12 @@ CH_PCI_DEVICE_ID_TABLE_DEFINE_BEGIN
>>> CH_PCI_ID_TABLE_FENTRY(0x5013), /* T580-chr */
>>> CH_PCI_ID_TABLE_FENTRY(0x5014), /* T580-so */
>>> CH_PCI_ID_TABLE_FENTRY(0x5015), /* T502-bt */
>>> + CH_PCI_ID_TABLE_FENTRY(0x5016), /* T580-OCP-SO */
>>> + CH_PCI_ID_TABLE_FENTRY(0x5017), /* T520-OCP-SO */
>>> + CH_PCI_ID_TABLE_FENTRY(0x5018), /* T540-BT */
>>> + CH_PCI_ID_TABLE_FENTRY(0x5019), /* T540-LP-BT */
>>> + CH_PCI_ID_TABLE_FENTRY(0x501a), /* T540-SO-BT */
>>> + CH_PCI_ID_TABLE_FENTRY(0x501b), /* T540-SO-CR */
>>> CH_PCI_ID_TABLE_FENTRY(0x5080), /* Custom T540-cr */
>>> CH_PCI_ID_TABLE_FENTRY(0x5081), /* Custom T540-LL-cr */
>>> CH_PCI_ID_TABLE_FENTRY(0x5082), /* Custom T504-cr */
>>> @@ -116,19 +122,63 @@ CH_PCI_DEVICE_ID_TABLE_DEFINE_BEGIN
>>> CH_PCI_ID_TABLE_FENTRY(0x5090), /* Custom T540-CR */
>>> CH_PCI_ID_TABLE_FENTRY(0x5091), /* Custom T522-CR */
>>> CH_PCI_ID_TABLE_FENTRY(0x5092), /* Custom T520-CR */
>>> + CH_PCI_ID_TABLE_FENTRY(0x5093), /* Custom T580-LP-CR */
>>> + CH_PCI_ID_TABLE_FENTRY(0x5094), /* Custom T540-CR */
>>> + CH_PCI_ID_TABLE_FENTRY(0x5095), /* Custom T540-CR-SO */
>>> + CH_PCI_ID_TABLE_FENTRY(0x5096), /* Custom T580-CR */
>>> + CH_PCI_ID_TABLE_FENTRY(0x5097), /* Custom T520-KR */
>>> + CH_PCI_ID_TABLE_FENTRY(0x5098), /* Custom 2x40G QSFP */
>>> + CH_PCI_ID_TABLE_FENTRY(0x5099), /* Custom 2x40G QSFP */
>>> + CH_PCI_ID_TABLE_FENTRY(0x509A), /* Custom T520-CR */
>>> + CH_PCI_ID_TABLE_FENTRY(0x509B), /* Custom T540-CR LOM */
>>> + CH_PCI_ID_TABLE_FENTRY(0x509c), /* Custom T520-CR SFP+ LOM */
>>> + CH_PCI_ID_TABLE_FENTRY(0x509d), /* Custom T540-CR SFP+ */
>>> + CH_PCI_ID_TABLE_FENTRY(0x509e), /* Custom T520-CR */
>>> + CH_PCI_ID_TABLE_FENTRY(0x509f), /* Custom T540-CR */
>>> + CH_PCI_ID_TABLE_FENTRY(0x50a0), /* Custom T540-CR */
>>> + CH_PCI_ID_TABLE_FENTRY(0x50a1), /* Custom T540-CR */
>>> + CH_PCI_ID_TABLE_FENTRY(0x50a2), /* Custom T580-KR4 */
>>> + CH_PCI_ID_TABLE_FENTRY(0x50a3), /* Custom T580-KR4 */
>>> + CH_PCI_ID_TABLE_FENTRY(0x50a4), /* Custom 2x T540-CR */
>>> + CH_PCI_ID_TABLE_FENTRY(0x50a5), /* Custom T522-BT */
>>> + CH_PCI_ID_TABLE_FENTRY(0x50a6), /* Custom T522-BT-SO */
>>> + CH_PCI_ID_TABLE_FENTRY(0x50a7), /* Custom T580-CR */
>>> + CH_PCI_ID_TABLE_FENTRY(0x50a8), /* Custom T580-KR */
>>> + CH_PCI_ID_TABLE_FENTRY(0x50a9), /* Custom T580-KR */
>>> + CH_PCI_ID_TABLE_FENTRY(0x50aa), /* Custom T580-CR */
>>> + CH_PCI_ID_TABLE_FENTRY(0x50ab), /* Custom T520-CR */
>>> + CH_PCI_ID_TABLE_FENTRY(0x50ac), /* Custom T540-BT */
>>> + CH_PCI_ID_TABLE_FENTRY(0x50ad), /* Custom T520-CR */
>>> + CH_PCI_ID_TABLE_FENTRY(0x50ae), /* Custom T540-XL-SO */
>>> + CH_PCI_ID_TABLE_FENTRY(0x50af), /* Custom T580-KR-SO */
>>> + CH_PCI_ID_TABLE_FENTRY(0x50b0), /* Custom T520-CR-LOM */
>>>
>>> /* T6 adapter */
>>> CH_PCI_ID_TABLE_FENTRY(0x6001), /* T6225-CR */
>>> CH_PCI_ID_TABLE_FENTRY(0x6002), /* T6225-SO-CR */
>>> CH_PCI_ID_TABLE_FENTRY(0x6003), /* T6425-CR */
>>> + CH_PCI_ID_TABLE_FENTRY(0x6004), /* T6425-SO-CR */
>>> CH_PCI_ID_TABLE_FENTRY(0x6005), /* T6225-OCP */
>>> + CH_PCI_ID_TABLE_FENTRY(0x6006), /* T62100-OCP-SO */
>>> CH_PCI_ID_TABLE_FENTRY(0x6007), /* T62100-LP-CR */
>>> CH_PCI_ID_TABLE_FENTRY(0x6008), /* T62100-SO-CR */
>>> + CH_PCI_ID_TABLE_FENTRY(0x6009), /* T6210-BT */
>>> CH_PCI_ID_TABLE_FENTRY(0x600d), /* T62100-CR */
>>> CH_PCI_ID_TABLE_FENTRY(0x6011), /* T6225-LL-CR */
>>> CH_PCI_ID_TABLE_FENTRY(0x6014), /* T61100-OCP-SO */
>>> + CH_PCI_ID_TABLE_FENTRY(0x6015), /* T6201-BT */
>>> CH_PCI_ID_TABLE_FENTRY(0x6080), /* Custom T6225-CR SFP28 */
>>> CH_PCI_ID_TABLE_FENTRY(0x6081), /* Custom T62100-CR */
>>> + CH_PCI_ID_TABLE_FENTRY(0x6082), /* Custom T6225-CR */
>>> + CH_PCI_ID_TABLE_FENTRY(0x6083), /* Custom T62100-CR */
>>> + CH_PCI_ID_TABLE_FENTRY(0x6084), /* Custom T64100-CR */
>>> + CH_PCI_ID_TABLE_FENTRY(0x6085), /* Custom T6240-SO */
>>> + CH_PCI_ID_TABLE_FENTRY(0x6086), /* Custom T6225-SO-CR */
>>> + CH_PCI_ID_TABLE_FENTRY(0x6087), /* Custom T6225-CR */
>>> + CH_PCI_ID_TABLE_FENTRY(0x6088), /* Custom T62100-CR */
>>> + CH_PCI_ID_TABLE_FENTRY(0x6089), /* Custom T62100-KR */
>>> + CH_PCI_ID_TABLE_FENTRY(0x608a), /* Custom T62100-CR */
>>> + CH_PCI_ID_TABLE_FENTRY(0x608b), /* Custom T6225-CR */
>>> CH_PCI_DEVICE_ID_TABLE_DEFINE_END;
>>>
>>> #endif /* CH_PCI_DEVICE_ID_TABLE_DEFINE_BEGIN */
>>>
>>
^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [dpdk-dev] [PATCH] net/cxgbe: update Chelsio T5/T6 NIC device ids
2019-05-09 10:38 ` Kevin Traynor
@ 2019-05-09 10:38 ` Kevin Traynor
2019-05-09 11:11 ` Rahul Lakkireddy
1 sibling, 0 replies; 24+ messages in thread
From: Kevin Traynor @ 2019-05-09 10:38 UTC (permalink / raw)
To: Rahul Lakkireddy
Cc: dev, Nirranjan Kirubaharan, Indranil Choudhury, stable,
Ferruh Yigit, Yongseok Koh, Luca Boccassi
On 09/05/2019 11:03, Rahul Lakkireddy wrote:
> On Thursday, May 05/09/19, 2019 at 14:29:05 +0530, Kevin Traynor wrote:
>> On 19/04/2019 10:22, Rahul Lakkireddy wrote:
>>> Fixes: 04868e5bfddd ("net/cxgbe: add support to run Chelsio T6 cards")
>>> Cc: stable@dpdk.org
>>>
>>
>> Hi Rahul,
>>
>> Which stable release(s) is this change requested for? Have you tested
>> these devices on those branches to ensure they work with the cxgbe pmd
>> on those branches?
>>
>> Kevin.
>
> Hi Kevin,
>
> Please target 18.11 and 19.02 branches. All these devices work fine
> in these branches.
>
Sorry, but this is not clear. There is no 19.02 branch and for 18.11
branch, I don't know if you tested some/all of the devices with that
branch or are making assumption based on the cxgbe pmd in 18.11.
I think these need to have some testing to be in 18.11.2. I will apply
it, but requesting you to do some testing to check they're working as
you expect when 18.11.2 RC is available (ETA for RC-1, week of 20th May).
thanks,
Kevin.
> Thanks,
> Rahul
>
>>
>>> Signed-off-by: Rahul Lakkireddy <rahul.lakkireddy@chelsio.com>
>>> ---
>>> drivers/net/cxgbe/base/t4_pci_id_tbl.h | 52 +++++++++++++++++++++++++-
>>> 1 file changed, 51 insertions(+), 1 deletion(-)
>>>
>>> diff --git a/drivers/net/cxgbe/base/t4_pci_id_tbl.h b/drivers/net/cxgbe/base/t4_pci_id_tbl.h
>>> index 5f5cbe048..f5f027a2e 100644
>>> --- a/drivers/net/cxgbe/base/t4_pci_id_tbl.h
>>> +++ b/drivers/net/cxgbe/base/t4_pci_id_tbl.h
>>> @@ -1,5 +1,5 @@
>>> /* SPDX-License-Identifier: BSD-3-Clause
>>> - * Copyright(c) 2014-2018 Chelsio Communications.
>>> + * Copyright(c) 2014-2019 Chelsio Communications.
>>> * All rights reserved.
>>> */
>>>
>>> @@ -103,6 +103,12 @@ CH_PCI_DEVICE_ID_TABLE_DEFINE_BEGIN
>>> CH_PCI_ID_TABLE_FENTRY(0x5013), /* T580-chr */
>>> CH_PCI_ID_TABLE_FENTRY(0x5014), /* T580-so */
>>> CH_PCI_ID_TABLE_FENTRY(0x5015), /* T502-bt */
>>> + CH_PCI_ID_TABLE_FENTRY(0x5016), /* T580-OCP-SO */
>>> + CH_PCI_ID_TABLE_FENTRY(0x5017), /* T520-OCP-SO */
>>> + CH_PCI_ID_TABLE_FENTRY(0x5018), /* T540-BT */
>>> + CH_PCI_ID_TABLE_FENTRY(0x5019), /* T540-LP-BT */
>>> + CH_PCI_ID_TABLE_FENTRY(0x501a), /* T540-SO-BT */
>>> + CH_PCI_ID_TABLE_FENTRY(0x501b), /* T540-SO-CR */
>>> CH_PCI_ID_TABLE_FENTRY(0x5080), /* Custom T540-cr */
>>> CH_PCI_ID_TABLE_FENTRY(0x5081), /* Custom T540-LL-cr */
>>> CH_PCI_ID_TABLE_FENTRY(0x5082), /* Custom T504-cr */
>>> @@ -116,19 +122,63 @@ CH_PCI_DEVICE_ID_TABLE_DEFINE_BEGIN
>>> CH_PCI_ID_TABLE_FENTRY(0x5090), /* Custom T540-CR */
>>> CH_PCI_ID_TABLE_FENTRY(0x5091), /* Custom T522-CR */
>>> CH_PCI_ID_TABLE_FENTRY(0x5092), /* Custom T520-CR */
>>> + CH_PCI_ID_TABLE_FENTRY(0x5093), /* Custom T580-LP-CR */
>>> + CH_PCI_ID_TABLE_FENTRY(0x5094), /* Custom T540-CR */
>>> + CH_PCI_ID_TABLE_FENTRY(0x5095), /* Custom T540-CR-SO */
>>> + CH_PCI_ID_TABLE_FENTRY(0x5096), /* Custom T580-CR */
>>> + CH_PCI_ID_TABLE_FENTRY(0x5097), /* Custom T520-KR */
>>> + CH_PCI_ID_TABLE_FENTRY(0x5098), /* Custom 2x40G QSFP */
>>> + CH_PCI_ID_TABLE_FENTRY(0x5099), /* Custom 2x40G QSFP */
>>> + CH_PCI_ID_TABLE_FENTRY(0x509A), /* Custom T520-CR */
>>> + CH_PCI_ID_TABLE_FENTRY(0x509B), /* Custom T540-CR LOM */
>>> + CH_PCI_ID_TABLE_FENTRY(0x509c), /* Custom T520-CR SFP+ LOM */
>>> + CH_PCI_ID_TABLE_FENTRY(0x509d), /* Custom T540-CR SFP+ */
>>> + CH_PCI_ID_TABLE_FENTRY(0x509e), /* Custom T520-CR */
>>> + CH_PCI_ID_TABLE_FENTRY(0x509f), /* Custom T540-CR */
>>> + CH_PCI_ID_TABLE_FENTRY(0x50a0), /* Custom T540-CR */
>>> + CH_PCI_ID_TABLE_FENTRY(0x50a1), /* Custom T540-CR */
>>> + CH_PCI_ID_TABLE_FENTRY(0x50a2), /* Custom T580-KR4 */
>>> + CH_PCI_ID_TABLE_FENTRY(0x50a3), /* Custom T580-KR4 */
>>> + CH_PCI_ID_TABLE_FENTRY(0x50a4), /* Custom 2x T540-CR */
>>> + CH_PCI_ID_TABLE_FENTRY(0x50a5), /* Custom T522-BT */
>>> + CH_PCI_ID_TABLE_FENTRY(0x50a6), /* Custom T522-BT-SO */
>>> + CH_PCI_ID_TABLE_FENTRY(0x50a7), /* Custom T580-CR */
>>> + CH_PCI_ID_TABLE_FENTRY(0x50a8), /* Custom T580-KR */
>>> + CH_PCI_ID_TABLE_FENTRY(0x50a9), /* Custom T580-KR */
>>> + CH_PCI_ID_TABLE_FENTRY(0x50aa), /* Custom T580-CR */
>>> + CH_PCI_ID_TABLE_FENTRY(0x50ab), /* Custom T520-CR */
>>> + CH_PCI_ID_TABLE_FENTRY(0x50ac), /* Custom T540-BT */
>>> + CH_PCI_ID_TABLE_FENTRY(0x50ad), /* Custom T520-CR */
>>> + CH_PCI_ID_TABLE_FENTRY(0x50ae), /* Custom T540-XL-SO */
>>> + CH_PCI_ID_TABLE_FENTRY(0x50af), /* Custom T580-KR-SO */
>>> + CH_PCI_ID_TABLE_FENTRY(0x50b0), /* Custom T520-CR-LOM */
>>>
>>> /* T6 adapter */
>>> CH_PCI_ID_TABLE_FENTRY(0x6001), /* T6225-CR */
>>> CH_PCI_ID_TABLE_FENTRY(0x6002), /* T6225-SO-CR */
>>> CH_PCI_ID_TABLE_FENTRY(0x6003), /* T6425-CR */
>>> + CH_PCI_ID_TABLE_FENTRY(0x6004), /* T6425-SO-CR */
>>> CH_PCI_ID_TABLE_FENTRY(0x6005), /* T6225-OCP */
>>> + CH_PCI_ID_TABLE_FENTRY(0x6006), /* T62100-OCP-SO */
>>> CH_PCI_ID_TABLE_FENTRY(0x6007), /* T62100-LP-CR */
>>> CH_PCI_ID_TABLE_FENTRY(0x6008), /* T62100-SO-CR */
>>> + CH_PCI_ID_TABLE_FENTRY(0x6009), /* T6210-BT */
>>> CH_PCI_ID_TABLE_FENTRY(0x600d), /* T62100-CR */
>>> CH_PCI_ID_TABLE_FENTRY(0x6011), /* T6225-LL-CR */
>>> CH_PCI_ID_TABLE_FENTRY(0x6014), /* T61100-OCP-SO */
>>> + CH_PCI_ID_TABLE_FENTRY(0x6015), /* T6201-BT */
>>> CH_PCI_ID_TABLE_FENTRY(0x6080), /* Custom T6225-CR SFP28 */
>>> CH_PCI_ID_TABLE_FENTRY(0x6081), /* Custom T62100-CR */
>>> + CH_PCI_ID_TABLE_FENTRY(0x6082), /* Custom T6225-CR */
>>> + CH_PCI_ID_TABLE_FENTRY(0x6083), /* Custom T62100-CR */
>>> + CH_PCI_ID_TABLE_FENTRY(0x6084), /* Custom T64100-CR */
>>> + CH_PCI_ID_TABLE_FENTRY(0x6085), /* Custom T6240-SO */
>>> + CH_PCI_ID_TABLE_FENTRY(0x6086), /* Custom T6225-SO-CR */
>>> + CH_PCI_ID_TABLE_FENTRY(0x6087), /* Custom T6225-CR */
>>> + CH_PCI_ID_TABLE_FENTRY(0x6088), /* Custom T62100-CR */
>>> + CH_PCI_ID_TABLE_FENTRY(0x6089), /* Custom T62100-KR */
>>> + CH_PCI_ID_TABLE_FENTRY(0x608a), /* Custom T62100-CR */
>>> + CH_PCI_ID_TABLE_FENTRY(0x608b), /* Custom T6225-CR */
>>> CH_PCI_DEVICE_ID_TABLE_DEFINE_END;
>>>
>>> #endif /* CH_PCI_DEVICE_ID_TABLE_DEFINE_BEGIN */
>>>
>>
^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [dpdk-dev] [PATCH] net/cxgbe: update Chelsio T5/T6 NIC device ids
2019-05-09 10:38 ` Kevin Traynor
2019-05-09 10:38 ` Kevin Traynor
@ 2019-05-09 11:11 ` Rahul Lakkireddy
2019-05-09 11:11 ` Rahul Lakkireddy
2019-05-09 11:55 ` Kevin Traynor
1 sibling, 2 replies; 24+ messages in thread
From: Rahul Lakkireddy @ 2019-05-09 11:11 UTC (permalink / raw)
To: Kevin Traynor
Cc: dev, Nirranjan Kirubaharan, Indranil Choudhury, stable,
Ferruh Yigit, Yongseok Koh, Luca Boccassi
On Thursday, May 05/09/19, 2019 at 16:08:05 +0530, Kevin Traynor wrote:
> On 09/05/2019 11:03, Rahul Lakkireddy wrote:
> > On Thursday, May 05/09/19, 2019 at 14:29:05 +0530, Kevin Traynor wrote:
> >> On 19/04/2019 10:22, Rahul Lakkireddy wrote:
> >>> Fixes: 04868e5bfddd ("net/cxgbe: add support to run Chelsio T6 cards")
> >>> Cc: stable@dpdk.org
> >>>
> >>
> >> Hi Rahul,
> >>
> >> Which stable release(s) is this change requested for? Have you tested
> >> these devices on those branches to ensure they work with the cxgbe pmd
> >> on those branches?
> >>
> >> Kevin.
> >
> > Hi Kevin,
> >
> > Please target 18.11 and 19.02 branches. All these devices work fine
> > in these branches.
> >
>
> Sorry, but this is not clear. There is no 19.02 branch and for 18.11
> branch, I don't know if you tested some/all of the devices with that
> branch or are making assumption based on the cxgbe pmd in 18.11.
>
Yes, these devices work fine by manually applying this patch on 18.11.1
and 19.02.
I had thought 19.02 branch is available by now. Looks like this is not
the case. Please apply the patch to 19.02 branch as well when it becomes
available.
> I think these need to have some testing to be in 18.11.2. I will apply
> it, but requesting you to do some testing to check they're working as
> you expect when 18.11.2 RC is available (ETA for RC-1, week of 20th May).
>
Sure, will test these devices on 18.11.2 RC as well.
Thanks,
Rahul
> thanks,
> Kevin.
>
> > Thanks,
> > Rahul
> >
> >>
> >>> Signed-off-by: Rahul Lakkireddy <rahul.lakkireddy@chelsio.com>
> >>> ---
> >>> drivers/net/cxgbe/base/t4_pci_id_tbl.h | 52 +++++++++++++++++++++++++-
> >>> 1 file changed, 51 insertions(+), 1 deletion(-)
> >>>
> >>> diff --git a/drivers/net/cxgbe/base/t4_pci_id_tbl.h b/drivers/net/cxgbe/base/t4_pci_id_tbl.h
> >>> index 5f5cbe048..f5f027a2e 100644
> >>> --- a/drivers/net/cxgbe/base/t4_pci_id_tbl.h
> >>> +++ b/drivers/net/cxgbe/base/t4_pci_id_tbl.h
> >>> @@ -1,5 +1,5 @@
> >>> /* SPDX-License-Identifier: BSD-3-Clause
> >>> - * Copyright(c) 2014-2018 Chelsio Communications.
> >>> + * Copyright(c) 2014-2019 Chelsio Communications.
> >>> * All rights reserved.
> >>> */
> >>>
> >>> @@ -103,6 +103,12 @@ CH_PCI_DEVICE_ID_TABLE_DEFINE_BEGIN
> >>> CH_PCI_ID_TABLE_FENTRY(0x5013), /* T580-chr */
> >>> CH_PCI_ID_TABLE_FENTRY(0x5014), /* T580-so */
> >>> CH_PCI_ID_TABLE_FENTRY(0x5015), /* T502-bt */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x5016), /* T580-OCP-SO */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x5017), /* T520-OCP-SO */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x5018), /* T540-BT */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x5019), /* T540-LP-BT */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x501a), /* T540-SO-BT */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x501b), /* T540-SO-CR */
> >>> CH_PCI_ID_TABLE_FENTRY(0x5080), /* Custom T540-cr */
> >>> CH_PCI_ID_TABLE_FENTRY(0x5081), /* Custom T540-LL-cr */
> >>> CH_PCI_ID_TABLE_FENTRY(0x5082), /* Custom T504-cr */
> >>> @@ -116,19 +122,63 @@ CH_PCI_DEVICE_ID_TABLE_DEFINE_BEGIN
> >>> CH_PCI_ID_TABLE_FENTRY(0x5090), /* Custom T540-CR */
> >>> CH_PCI_ID_TABLE_FENTRY(0x5091), /* Custom T522-CR */
> >>> CH_PCI_ID_TABLE_FENTRY(0x5092), /* Custom T520-CR */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x5093), /* Custom T580-LP-CR */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x5094), /* Custom T540-CR */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x5095), /* Custom T540-CR-SO */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x5096), /* Custom T580-CR */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x5097), /* Custom T520-KR */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x5098), /* Custom 2x40G QSFP */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x5099), /* Custom 2x40G QSFP */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x509A), /* Custom T520-CR */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x509B), /* Custom T540-CR LOM */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x509c), /* Custom T520-CR SFP+ LOM */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x509d), /* Custom T540-CR SFP+ */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x509e), /* Custom T520-CR */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x509f), /* Custom T540-CR */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x50a0), /* Custom T540-CR */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x50a1), /* Custom T540-CR */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x50a2), /* Custom T580-KR4 */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x50a3), /* Custom T580-KR4 */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x50a4), /* Custom 2x T540-CR */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x50a5), /* Custom T522-BT */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x50a6), /* Custom T522-BT-SO */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x50a7), /* Custom T580-CR */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x50a8), /* Custom T580-KR */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x50a9), /* Custom T580-KR */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x50aa), /* Custom T580-CR */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x50ab), /* Custom T520-CR */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x50ac), /* Custom T540-BT */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x50ad), /* Custom T520-CR */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x50ae), /* Custom T540-XL-SO */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x50af), /* Custom T580-KR-SO */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x50b0), /* Custom T520-CR-LOM */
> >>>
> >>> /* T6 adapter */
> >>> CH_PCI_ID_TABLE_FENTRY(0x6001), /* T6225-CR */
> >>> CH_PCI_ID_TABLE_FENTRY(0x6002), /* T6225-SO-CR */
> >>> CH_PCI_ID_TABLE_FENTRY(0x6003), /* T6425-CR */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x6004), /* T6425-SO-CR */
> >>> CH_PCI_ID_TABLE_FENTRY(0x6005), /* T6225-OCP */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x6006), /* T62100-OCP-SO */
> >>> CH_PCI_ID_TABLE_FENTRY(0x6007), /* T62100-LP-CR */
> >>> CH_PCI_ID_TABLE_FENTRY(0x6008), /* T62100-SO-CR */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x6009), /* T6210-BT */
> >>> CH_PCI_ID_TABLE_FENTRY(0x600d), /* T62100-CR */
> >>> CH_PCI_ID_TABLE_FENTRY(0x6011), /* T6225-LL-CR */
> >>> CH_PCI_ID_TABLE_FENTRY(0x6014), /* T61100-OCP-SO */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x6015), /* T6201-BT */
> >>> CH_PCI_ID_TABLE_FENTRY(0x6080), /* Custom T6225-CR SFP28 */
> >>> CH_PCI_ID_TABLE_FENTRY(0x6081), /* Custom T62100-CR */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x6082), /* Custom T6225-CR */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x6083), /* Custom T62100-CR */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x6084), /* Custom T64100-CR */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x6085), /* Custom T6240-SO */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x6086), /* Custom T6225-SO-CR */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x6087), /* Custom T6225-CR */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x6088), /* Custom T62100-CR */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x6089), /* Custom T62100-KR */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x608a), /* Custom T62100-CR */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x608b), /* Custom T6225-CR */
> >>> CH_PCI_DEVICE_ID_TABLE_DEFINE_END;
> >>>
> >>> #endif /* CH_PCI_DEVICE_ID_TABLE_DEFINE_BEGIN */
> >>>
> >>
>
^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [dpdk-dev] [PATCH] net/cxgbe: update Chelsio T5/T6 NIC device ids
2019-05-09 11:11 ` Rahul Lakkireddy
@ 2019-05-09 11:11 ` Rahul Lakkireddy
2019-05-09 11:55 ` Kevin Traynor
1 sibling, 0 replies; 24+ messages in thread
From: Rahul Lakkireddy @ 2019-05-09 11:11 UTC (permalink / raw)
To: Kevin Traynor
Cc: dev, Nirranjan Kirubaharan, Indranil Choudhury, stable,
Ferruh Yigit, Yongseok Koh, Luca Boccassi
On Thursday, May 05/09/19, 2019 at 16:08:05 +0530, Kevin Traynor wrote:
> On 09/05/2019 11:03, Rahul Lakkireddy wrote:
> > On Thursday, May 05/09/19, 2019 at 14:29:05 +0530, Kevin Traynor wrote:
> >> On 19/04/2019 10:22, Rahul Lakkireddy wrote:
> >>> Fixes: 04868e5bfddd ("net/cxgbe: add support to run Chelsio T6 cards")
> >>> Cc: stable@dpdk.org
> >>>
> >>
> >> Hi Rahul,
> >>
> >> Which stable release(s) is this change requested for? Have you tested
> >> these devices on those branches to ensure they work with the cxgbe pmd
> >> on those branches?
> >>
> >> Kevin.
> >
> > Hi Kevin,
> >
> > Please target 18.11 and 19.02 branches. All these devices work fine
> > in these branches.
> >
>
> Sorry, but this is not clear. There is no 19.02 branch and for 18.11
> branch, I don't know if you tested some/all of the devices with that
> branch or are making assumption based on the cxgbe pmd in 18.11.
>
Yes, these devices work fine by manually applying this patch on 18.11.1
and 19.02.
I had thought 19.02 branch is available by now. Looks like this is not
the case. Please apply the patch to 19.02 branch as well when it becomes
available.
> I think these need to have some testing to be in 18.11.2. I will apply
> it, but requesting you to do some testing to check they're working as
> you expect when 18.11.2 RC is available (ETA for RC-1, week of 20th May).
>
Sure, will test these devices on 18.11.2 RC as well.
Thanks,
Rahul
> thanks,
> Kevin.
>
> > Thanks,
> > Rahul
> >
> >>
> >>> Signed-off-by: Rahul Lakkireddy <rahul.lakkireddy@chelsio.com>
> >>> ---
> >>> drivers/net/cxgbe/base/t4_pci_id_tbl.h | 52 +++++++++++++++++++++++++-
> >>> 1 file changed, 51 insertions(+), 1 deletion(-)
> >>>
> >>> diff --git a/drivers/net/cxgbe/base/t4_pci_id_tbl.h b/drivers/net/cxgbe/base/t4_pci_id_tbl.h
> >>> index 5f5cbe048..f5f027a2e 100644
> >>> --- a/drivers/net/cxgbe/base/t4_pci_id_tbl.h
> >>> +++ b/drivers/net/cxgbe/base/t4_pci_id_tbl.h
> >>> @@ -1,5 +1,5 @@
> >>> /* SPDX-License-Identifier: BSD-3-Clause
> >>> - * Copyright(c) 2014-2018 Chelsio Communications.
> >>> + * Copyright(c) 2014-2019 Chelsio Communications.
> >>> * All rights reserved.
> >>> */
> >>>
> >>> @@ -103,6 +103,12 @@ CH_PCI_DEVICE_ID_TABLE_DEFINE_BEGIN
> >>> CH_PCI_ID_TABLE_FENTRY(0x5013), /* T580-chr */
> >>> CH_PCI_ID_TABLE_FENTRY(0x5014), /* T580-so */
> >>> CH_PCI_ID_TABLE_FENTRY(0x5015), /* T502-bt */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x5016), /* T580-OCP-SO */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x5017), /* T520-OCP-SO */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x5018), /* T540-BT */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x5019), /* T540-LP-BT */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x501a), /* T540-SO-BT */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x501b), /* T540-SO-CR */
> >>> CH_PCI_ID_TABLE_FENTRY(0x5080), /* Custom T540-cr */
> >>> CH_PCI_ID_TABLE_FENTRY(0x5081), /* Custom T540-LL-cr */
> >>> CH_PCI_ID_TABLE_FENTRY(0x5082), /* Custom T504-cr */
> >>> @@ -116,19 +122,63 @@ CH_PCI_DEVICE_ID_TABLE_DEFINE_BEGIN
> >>> CH_PCI_ID_TABLE_FENTRY(0x5090), /* Custom T540-CR */
> >>> CH_PCI_ID_TABLE_FENTRY(0x5091), /* Custom T522-CR */
> >>> CH_PCI_ID_TABLE_FENTRY(0x5092), /* Custom T520-CR */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x5093), /* Custom T580-LP-CR */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x5094), /* Custom T540-CR */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x5095), /* Custom T540-CR-SO */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x5096), /* Custom T580-CR */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x5097), /* Custom T520-KR */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x5098), /* Custom 2x40G QSFP */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x5099), /* Custom 2x40G QSFP */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x509A), /* Custom T520-CR */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x509B), /* Custom T540-CR LOM */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x509c), /* Custom T520-CR SFP+ LOM */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x509d), /* Custom T540-CR SFP+ */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x509e), /* Custom T520-CR */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x509f), /* Custom T540-CR */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x50a0), /* Custom T540-CR */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x50a1), /* Custom T540-CR */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x50a2), /* Custom T580-KR4 */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x50a3), /* Custom T580-KR4 */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x50a4), /* Custom 2x T540-CR */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x50a5), /* Custom T522-BT */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x50a6), /* Custom T522-BT-SO */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x50a7), /* Custom T580-CR */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x50a8), /* Custom T580-KR */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x50a9), /* Custom T580-KR */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x50aa), /* Custom T580-CR */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x50ab), /* Custom T520-CR */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x50ac), /* Custom T540-BT */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x50ad), /* Custom T520-CR */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x50ae), /* Custom T540-XL-SO */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x50af), /* Custom T580-KR-SO */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x50b0), /* Custom T520-CR-LOM */
> >>>
> >>> /* T6 adapter */
> >>> CH_PCI_ID_TABLE_FENTRY(0x6001), /* T6225-CR */
> >>> CH_PCI_ID_TABLE_FENTRY(0x6002), /* T6225-SO-CR */
> >>> CH_PCI_ID_TABLE_FENTRY(0x6003), /* T6425-CR */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x6004), /* T6425-SO-CR */
> >>> CH_PCI_ID_TABLE_FENTRY(0x6005), /* T6225-OCP */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x6006), /* T62100-OCP-SO */
> >>> CH_PCI_ID_TABLE_FENTRY(0x6007), /* T62100-LP-CR */
> >>> CH_PCI_ID_TABLE_FENTRY(0x6008), /* T62100-SO-CR */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x6009), /* T6210-BT */
> >>> CH_PCI_ID_TABLE_FENTRY(0x600d), /* T62100-CR */
> >>> CH_PCI_ID_TABLE_FENTRY(0x6011), /* T6225-LL-CR */
> >>> CH_PCI_ID_TABLE_FENTRY(0x6014), /* T61100-OCP-SO */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x6015), /* T6201-BT */
> >>> CH_PCI_ID_TABLE_FENTRY(0x6080), /* Custom T6225-CR SFP28 */
> >>> CH_PCI_ID_TABLE_FENTRY(0x6081), /* Custom T62100-CR */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x6082), /* Custom T6225-CR */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x6083), /* Custom T62100-CR */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x6084), /* Custom T64100-CR */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x6085), /* Custom T6240-SO */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x6086), /* Custom T6225-SO-CR */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x6087), /* Custom T6225-CR */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x6088), /* Custom T62100-CR */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x6089), /* Custom T62100-KR */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x608a), /* Custom T62100-CR */
> >>> + CH_PCI_ID_TABLE_FENTRY(0x608b), /* Custom T6225-CR */
> >>> CH_PCI_DEVICE_ID_TABLE_DEFINE_END;
> >>>
> >>> #endif /* CH_PCI_DEVICE_ID_TABLE_DEFINE_BEGIN */
> >>>
> >>
>
^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [dpdk-dev] [PATCH] net/cxgbe: update Chelsio T5/T6 NIC device ids
2019-05-09 11:11 ` Rahul Lakkireddy
2019-05-09 11:11 ` Rahul Lakkireddy
@ 2019-05-09 11:55 ` Kevin Traynor
2019-05-09 11:55 ` Kevin Traynor
2019-05-09 12:19 ` Rahul Lakkireddy
1 sibling, 2 replies; 24+ messages in thread
From: Kevin Traynor @ 2019-05-09 11:55 UTC (permalink / raw)
To: Rahul Lakkireddy
Cc: dev, Nirranjan Kirubaharan, Indranil Choudhury, stable,
Ferruh Yigit, Yongseok Koh, Luca Boccassi
On 09/05/2019 12:11, Rahul Lakkireddy wrote:
> On Thursday, May 05/09/19, 2019 at 16:08:05 +0530, Kevin Traynor wrote:
>> On 09/05/2019 11:03, Rahul Lakkireddy wrote:
>>> On Thursday, May 05/09/19, 2019 at 14:29:05 +0530, Kevin Traynor wrote:
>>>> On 19/04/2019 10:22, Rahul Lakkireddy wrote:
>>>>> Fixes: 04868e5bfddd ("net/cxgbe: add support to run Chelsio T6 cards")
>>>>> Cc: stable@dpdk.org
>>>>>
>>>>
>>>> Hi Rahul,
>>>>
>>>> Which stable release(s) is this change requested for? Have you tested
>>>> these devices on those branches to ensure they work with the cxgbe pmd
>>>> on those branches?
>>>>
>>>> Kevin.
>>>
>>> Hi Kevin,
>>>
>>> Please target 18.11 and 19.02 branches. All these devices work fine
>>> in these branches.
>>>
>>
>> Sorry, but this is not clear. There is no 19.02 branch and for 18.11
>> branch, I don't know if you tested some/all of the devices with that
>> branch or are making assumption based on the cxgbe pmd in 18.11.
>>
>
> Yes, these devices work fine by manually applying this patch on 18.11.1
> and 19.02.
>
Thanks for clarifying.
> I had thought 19.02 branch is available by now. Looks like this is not
> the case. Please apply the patch to 19.02 branch as well when it becomes
> available.
>
There won't be a 19.02 stable branch. tl;dr there wasn't a maintainer
and not enough resources in the community for validating non-LTS
stables. See policy: here
http://doc.dpdk.org/guides/contributing/stable.html#stable-releases
>> I think these need to have some testing to be in 18.11.2. I will apply
>> it, but requesting you to do some testing to check they're working as
>> you expect when 18.11.2 RC is available (ETA for RC-1, week of 20th May).
>>
>
> Sure, will test these devices on 18.11.2 RC as well.
>
ok, sounds good,
Kevin.
> Thanks,
> Rahul
>
>> thanks,
>> Kevin.
>>
>>> Thanks,
>>> Rahul
>>>
>>>>
>>>>> Signed-off-by: Rahul Lakkireddy <rahul.lakkireddy@chelsio.com>
>>>>> ---
>>>>> drivers/net/cxgbe/base/t4_pci_id_tbl.h | 52 +++++++++++++++++++++++++-
>>>>> 1 file changed, 51 insertions(+), 1 deletion(-)
>>>>>
>>>>> diff --git a/drivers/net/cxgbe/base/t4_pci_id_tbl.h b/drivers/net/cxgbe/base/t4_pci_id_tbl.h
>>>>> index 5f5cbe048..f5f027a2e 100644
>>>>> --- a/drivers/net/cxgbe/base/t4_pci_id_tbl.h
>>>>> +++ b/drivers/net/cxgbe/base/t4_pci_id_tbl.h
>>>>> @@ -1,5 +1,5 @@
>>>>> /* SPDX-License-Identifier: BSD-3-Clause
>>>>> - * Copyright(c) 2014-2018 Chelsio Communications.
>>>>> + * Copyright(c) 2014-2019 Chelsio Communications.
>>>>> * All rights reserved.
>>>>> */
>>>>>
>>>>> @@ -103,6 +103,12 @@ CH_PCI_DEVICE_ID_TABLE_DEFINE_BEGIN
>>>>> CH_PCI_ID_TABLE_FENTRY(0x5013), /* T580-chr */
>>>>> CH_PCI_ID_TABLE_FENTRY(0x5014), /* T580-so */
>>>>> CH_PCI_ID_TABLE_FENTRY(0x5015), /* T502-bt */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x5016), /* T580-OCP-SO */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x5017), /* T520-OCP-SO */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x5018), /* T540-BT */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x5019), /* T540-LP-BT */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x501a), /* T540-SO-BT */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x501b), /* T540-SO-CR */
>>>>> CH_PCI_ID_TABLE_FENTRY(0x5080), /* Custom T540-cr */
>>>>> CH_PCI_ID_TABLE_FENTRY(0x5081), /* Custom T540-LL-cr */
>>>>> CH_PCI_ID_TABLE_FENTRY(0x5082), /* Custom T504-cr */
>>>>> @@ -116,19 +122,63 @@ CH_PCI_DEVICE_ID_TABLE_DEFINE_BEGIN
>>>>> CH_PCI_ID_TABLE_FENTRY(0x5090), /* Custom T540-CR */
>>>>> CH_PCI_ID_TABLE_FENTRY(0x5091), /* Custom T522-CR */
>>>>> CH_PCI_ID_TABLE_FENTRY(0x5092), /* Custom T520-CR */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x5093), /* Custom T580-LP-CR */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x5094), /* Custom T540-CR */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x5095), /* Custom T540-CR-SO */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x5096), /* Custom T580-CR */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x5097), /* Custom T520-KR */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x5098), /* Custom 2x40G QSFP */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x5099), /* Custom 2x40G QSFP */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x509A), /* Custom T520-CR */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x509B), /* Custom T540-CR LOM */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x509c), /* Custom T520-CR SFP+ LOM */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x509d), /* Custom T540-CR SFP+ */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x509e), /* Custom T520-CR */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x509f), /* Custom T540-CR */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x50a0), /* Custom T540-CR */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x50a1), /* Custom T540-CR */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x50a2), /* Custom T580-KR4 */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x50a3), /* Custom T580-KR4 */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x50a4), /* Custom 2x T540-CR */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x50a5), /* Custom T522-BT */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x50a6), /* Custom T522-BT-SO */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x50a7), /* Custom T580-CR */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x50a8), /* Custom T580-KR */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x50a9), /* Custom T580-KR */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x50aa), /* Custom T580-CR */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x50ab), /* Custom T520-CR */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x50ac), /* Custom T540-BT */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x50ad), /* Custom T520-CR */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x50ae), /* Custom T540-XL-SO */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x50af), /* Custom T580-KR-SO */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x50b0), /* Custom T520-CR-LOM */
>>>>>
>>>>> /* T6 adapter */
>>>>> CH_PCI_ID_TABLE_FENTRY(0x6001), /* T6225-CR */
>>>>> CH_PCI_ID_TABLE_FENTRY(0x6002), /* T6225-SO-CR */
>>>>> CH_PCI_ID_TABLE_FENTRY(0x6003), /* T6425-CR */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x6004), /* T6425-SO-CR */
>>>>> CH_PCI_ID_TABLE_FENTRY(0x6005), /* T6225-OCP */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x6006), /* T62100-OCP-SO */
>>>>> CH_PCI_ID_TABLE_FENTRY(0x6007), /* T62100-LP-CR */
>>>>> CH_PCI_ID_TABLE_FENTRY(0x6008), /* T62100-SO-CR */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x6009), /* T6210-BT */
>>>>> CH_PCI_ID_TABLE_FENTRY(0x600d), /* T62100-CR */
>>>>> CH_PCI_ID_TABLE_FENTRY(0x6011), /* T6225-LL-CR */
>>>>> CH_PCI_ID_TABLE_FENTRY(0x6014), /* T61100-OCP-SO */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x6015), /* T6201-BT */
>>>>> CH_PCI_ID_TABLE_FENTRY(0x6080), /* Custom T6225-CR SFP28 */
>>>>> CH_PCI_ID_TABLE_FENTRY(0x6081), /* Custom T62100-CR */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x6082), /* Custom T6225-CR */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x6083), /* Custom T62100-CR */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x6084), /* Custom T64100-CR */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x6085), /* Custom T6240-SO */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x6086), /* Custom T6225-SO-CR */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x6087), /* Custom T6225-CR */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x6088), /* Custom T62100-CR */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x6089), /* Custom T62100-KR */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x608a), /* Custom T62100-CR */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x608b), /* Custom T6225-CR */
>>>>> CH_PCI_DEVICE_ID_TABLE_DEFINE_END;
>>>>>
>>>>> #endif /* CH_PCI_DEVICE_ID_TABLE_DEFINE_BEGIN */
>>>>>
>>>>
>>
^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [dpdk-dev] [PATCH] net/cxgbe: update Chelsio T5/T6 NIC device ids
2019-05-09 11:55 ` Kevin Traynor
@ 2019-05-09 11:55 ` Kevin Traynor
2019-05-09 12:19 ` Rahul Lakkireddy
1 sibling, 0 replies; 24+ messages in thread
From: Kevin Traynor @ 2019-05-09 11:55 UTC (permalink / raw)
To: Rahul Lakkireddy
Cc: dev, Nirranjan Kirubaharan, Indranil Choudhury, stable,
Ferruh Yigit, Yongseok Koh, Luca Boccassi
On 09/05/2019 12:11, Rahul Lakkireddy wrote:
> On Thursday, May 05/09/19, 2019 at 16:08:05 +0530, Kevin Traynor wrote:
>> On 09/05/2019 11:03, Rahul Lakkireddy wrote:
>>> On Thursday, May 05/09/19, 2019 at 14:29:05 +0530, Kevin Traynor wrote:
>>>> On 19/04/2019 10:22, Rahul Lakkireddy wrote:
>>>>> Fixes: 04868e5bfddd ("net/cxgbe: add support to run Chelsio T6 cards")
>>>>> Cc: stable@dpdk.org
>>>>>
>>>>
>>>> Hi Rahul,
>>>>
>>>> Which stable release(s) is this change requested for? Have you tested
>>>> these devices on those branches to ensure they work with the cxgbe pmd
>>>> on those branches?
>>>>
>>>> Kevin.
>>>
>>> Hi Kevin,
>>>
>>> Please target 18.11 and 19.02 branches. All these devices work fine
>>> in these branches.
>>>
>>
>> Sorry, but this is not clear. There is no 19.02 branch and for 18.11
>> branch, I don't know if you tested some/all of the devices with that
>> branch or are making assumption based on the cxgbe pmd in 18.11.
>>
>
> Yes, these devices work fine by manually applying this patch on 18.11.1
> and 19.02.
>
Thanks for clarifying.
> I had thought 19.02 branch is available by now. Looks like this is not
> the case. Please apply the patch to 19.02 branch as well when it becomes
> available.
>
There won't be a 19.02 stable branch. tl;dr there wasn't a maintainer
and not enough resources in the community for validating non-LTS
stables. See policy: here
http://doc.dpdk.org/guides/contributing/stable.html#stable-releases
>> I think these need to have some testing to be in 18.11.2. I will apply
>> it, but requesting you to do some testing to check they're working as
>> you expect when 18.11.2 RC is available (ETA for RC-1, week of 20th May).
>>
>
> Sure, will test these devices on 18.11.2 RC as well.
>
ok, sounds good,
Kevin.
> Thanks,
> Rahul
>
>> thanks,
>> Kevin.
>>
>>> Thanks,
>>> Rahul
>>>
>>>>
>>>>> Signed-off-by: Rahul Lakkireddy <rahul.lakkireddy@chelsio.com>
>>>>> ---
>>>>> drivers/net/cxgbe/base/t4_pci_id_tbl.h | 52 +++++++++++++++++++++++++-
>>>>> 1 file changed, 51 insertions(+), 1 deletion(-)
>>>>>
>>>>> diff --git a/drivers/net/cxgbe/base/t4_pci_id_tbl.h b/drivers/net/cxgbe/base/t4_pci_id_tbl.h
>>>>> index 5f5cbe048..f5f027a2e 100644
>>>>> --- a/drivers/net/cxgbe/base/t4_pci_id_tbl.h
>>>>> +++ b/drivers/net/cxgbe/base/t4_pci_id_tbl.h
>>>>> @@ -1,5 +1,5 @@
>>>>> /* SPDX-License-Identifier: BSD-3-Clause
>>>>> - * Copyright(c) 2014-2018 Chelsio Communications.
>>>>> + * Copyright(c) 2014-2019 Chelsio Communications.
>>>>> * All rights reserved.
>>>>> */
>>>>>
>>>>> @@ -103,6 +103,12 @@ CH_PCI_DEVICE_ID_TABLE_DEFINE_BEGIN
>>>>> CH_PCI_ID_TABLE_FENTRY(0x5013), /* T580-chr */
>>>>> CH_PCI_ID_TABLE_FENTRY(0x5014), /* T580-so */
>>>>> CH_PCI_ID_TABLE_FENTRY(0x5015), /* T502-bt */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x5016), /* T580-OCP-SO */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x5017), /* T520-OCP-SO */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x5018), /* T540-BT */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x5019), /* T540-LP-BT */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x501a), /* T540-SO-BT */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x501b), /* T540-SO-CR */
>>>>> CH_PCI_ID_TABLE_FENTRY(0x5080), /* Custom T540-cr */
>>>>> CH_PCI_ID_TABLE_FENTRY(0x5081), /* Custom T540-LL-cr */
>>>>> CH_PCI_ID_TABLE_FENTRY(0x5082), /* Custom T504-cr */
>>>>> @@ -116,19 +122,63 @@ CH_PCI_DEVICE_ID_TABLE_DEFINE_BEGIN
>>>>> CH_PCI_ID_TABLE_FENTRY(0x5090), /* Custom T540-CR */
>>>>> CH_PCI_ID_TABLE_FENTRY(0x5091), /* Custom T522-CR */
>>>>> CH_PCI_ID_TABLE_FENTRY(0x5092), /* Custom T520-CR */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x5093), /* Custom T580-LP-CR */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x5094), /* Custom T540-CR */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x5095), /* Custom T540-CR-SO */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x5096), /* Custom T580-CR */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x5097), /* Custom T520-KR */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x5098), /* Custom 2x40G QSFP */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x5099), /* Custom 2x40G QSFP */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x509A), /* Custom T520-CR */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x509B), /* Custom T540-CR LOM */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x509c), /* Custom T520-CR SFP+ LOM */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x509d), /* Custom T540-CR SFP+ */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x509e), /* Custom T520-CR */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x509f), /* Custom T540-CR */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x50a0), /* Custom T540-CR */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x50a1), /* Custom T540-CR */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x50a2), /* Custom T580-KR4 */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x50a3), /* Custom T580-KR4 */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x50a4), /* Custom 2x T540-CR */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x50a5), /* Custom T522-BT */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x50a6), /* Custom T522-BT-SO */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x50a7), /* Custom T580-CR */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x50a8), /* Custom T580-KR */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x50a9), /* Custom T580-KR */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x50aa), /* Custom T580-CR */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x50ab), /* Custom T520-CR */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x50ac), /* Custom T540-BT */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x50ad), /* Custom T520-CR */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x50ae), /* Custom T540-XL-SO */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x50af), /* Custom T580-KR-SO */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x50b0), /* Custom T520-CR-LOM */
>>>>>
>>>>> /* T6 adapter */
>>>>> CH_PCI_ID_TABLE_FENTRY(0x6001), /* T6225-CR */
>>>>> CH_PCI_ID_TABLE_FENTRY(0x6002), /* T6225-SO-CR */
>>>>> CH_PCI_ID_TABLE_FENTRY(0x6003), /* T6425-CR */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x6004), /* T6425-SO-CR */
>>>>> CH_PCI_ID_TABLE_FENTRY(0x6005), /* T6225-OCP */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x6006), /* T62100-OCP-SO */
>>>>> CH_PCI_ID_TABLE_FENTRY(0x6007), /* T62100-LP-CR */
>>>>> CH_PCI_ID_TABLE_FENTRY(0x6008), /* T62100-SO-CR */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x6009), /* T6210-BT */
>>>>> CH_PCI_ID_TABLE_FENTRY(0x600d), /* T62100-CR */
>>>>> CH_PCI_ID_TABLE_FENTRY(0x6011), /* T6225-LL-CR */
>>>>> CH_PCI_ID_TABLE_FENTRY(0x6014), /* T61100-OCP-SO */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x6015), /* T6201-BT */
>>>>> CH_PCI_ID_TABLE_FENTRY(0x6080), /* Custom T6225-CR SFP28 */
>>>>> CH_PCI_ID_TABLE_FENTRY(0x6081), /* Custom T62100-CR */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x6082), /* Custom T6225-CR */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x6083), /* Custom T62100-CR */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x6084), /* Custom T64100-CR */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x6085), /* Custom T6240-SO */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x6086), /* Custom T6225-SO-CR */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x6087), /* Custom T6225-CR */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x6088), /* Custom T62100-CR */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x6089), /* Custom T62100-KR */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x608a), /* Custom T62100-CR */
>>>>> + CH_PCI_ID_TABLE_FENTRY(0x608b), /* Custom T6225-CR */
>>>>> CH_PCI_DEVICE_ID_TABLE_DEFINE_END;
>>>>>
>>>>> #endif /* CH_PCI_DEVICE_ID_TABLE_DEFINE_BEGIN */
>>>>>
>>>>
>>
^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [dpdk-dev] [PATCH] net/cxgbe: update Chelsio T5/T6 NIC device ids
2019-05-09 11:55 ` Kevin Traynor
2019-05-09 11:55 ` Kevin Traynor
@ 2019-05-09 12:19 ` Rahul Lakkireddy
2019-05-09 12:19 ` Rahul Lakkireddy
1 sibling, 1 reply; 24+ messages in thread
From: Rahul Lakkireddy @ 2019-05-09 12:19 UTC (permalink / raw)
To: Kevin Traynor
Cc: dev, Nirranjan Kirubaharan, Indranil Choudhury, stable,
Ferruh Yigit, Yongseok Koh, Luca Boccassi
On Thursday, May 05/09/19, 2019 at 17:25:56 +0530, Kevin Traynor wrote:
> On 09/05/2019 12:11, Rahul Lakkireddy wrote:
> > On Thursday, May 05/09/19, 2019 at 16:08:05 +0530, Kevin Traynor wrote:
> >> On 09/05/2019 11:03, Rahul Lakkireddy wrote:
> >>> On Thursday, May 05/09/19, 2019 at 14:29:05 +0530, Kevin Traynor wrote:
> >>>> On 19/04/2019 10:22, Rahul Lakkireddy wrote:
> >>>>> Fixes: 04868e5bfddd ("net/cxgbe: add support to run Chelsio T6 cards")
> >>>>> Cc: stable@dpdk.org
> >>>>>
> >>>>
> >>>> Hi Rahul,
> >>>>
> >>>> Which stable release(s) is this change requested for? Have you tested
> >>>> these devices on those branches to ensure they work with the cxgbe pmd
> >>>> on those branches?
> >>>>
> >>>> Kevin.
> >>>
> >>> Hi Kevin,
> >>>
> >>> Please target 18.11 and 19.02 branches. All these devices work fine
> >>> in these branches.
> >>>
> >>
> >> Sorry, but this is not clear. There is no 19.02 branch and for 18.11
> >> branch, I don't know if you tested some/all of the devices with that
> >> branch or are making assumption based on the cxgbe pmd in 18.11.
> >>
> >
> > Yes, these devices work fine by manually applying this patch on 18.11.1
> > and 19.02.
> >
>
> Thanks for clarifying.
>
> > I had thought 19.02 branch is available by now. Looks like this is not
> > the case. Please apply the patch to 19.02 branch as well when it becomes
> > available.
> >
>
> There won't be a 19.02 stable branch. tl;dr there wasn't a maintainer
> and not enough resources in the community for validating non-LTS
> stables. See policy: here
> http://doc.dpdk.org/guides/contributing/stable.html#stable-releases
>
I see. I wasn't aware of this. Thanks for clarifying.
Thanks,
Rahul
> >> I think these need to have some testing to be in 18.11.2. I will apply
> >> it, but requesting you to do some testing to check they're working as
> >> you expect when 18.11.2 RC is available (ETA for RC-1, week of 20th May).
> >>
> >
> > Sure, will test these devices on 18.11.2 RC as well.
> >
>
> ok, sounds good,
>
> Kevin.
>
> > Thanks,
> > Rahul
> >
> >> thanks,
> >> Kevin.
> >>
> >>> Thanks,
> >>> Rahul
> >>>
> >>>>
> >>>>> Signed-off-by: Rahul Lakkireddy <rahul.lakkireddy@chelsio.com>
> >>>>> ---
> >>>>> drivers/net/cxgbe/base/t4_pci_id_tbl.h | 52 +++++++++++++++++++++++++-
> >>>>> 1 file changed, 51 insertions(+), 1 deletion(-)
> >>>>>
> >>>>> diff --git a/drivers/net/cxgbe/base/t4_pci_id_tbl.h b/drivers/net/cxgbe/base/t4_pci_id_tbl.h
> >>>>> index 5f5cbe048..f5f027a2e 100644
> >>>>> --- a/drivers/net/cxgbe/base/t4_pci_id_tbl.h
> >>>>> +++ b/drivers/net/cxgbe/base/t4_pci_id_tbl.h
> >>>>> @@ -1,5 +1,5 @@
> >>>>> /* SPDX-License-Identifier: BSD-3-Clause
> >>>>> - * Copyright(c) 2014-2018 Chelsio Communications.
> >>>>> + * Copyright(c) 2014-2019 Chelsio Communications.
> >>>>> * All rights reserved.
> >>>>> */
> >>>>>
> >>>>> @@ -103,6 +103,12 @@ CH_PCI_DEVICE_ID_TABLE_DEFINE_BEGIN
> >>>>> CH_PCI_ID_TABLE_FENTRY(0x5013), /* T580-chr */
> >>>>> CH_PCI_ID_TABLE_FENTRY(0x5014), /* T580-so */
> >>>>> CH_PCI_ID_TABLE_FENTRY(0x5015), /* T502-bt */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x5016), /* T580-OCP-SO */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x5017), /* T520-OCP-SO */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x5018), /* T540-BT */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x5019), /* T540-LP-BT */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x501a), /* T540-SO-BT */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x501b), /* T540-SO-CR */
> >>>>> CH_PCI_ID_TABLE_FENTRY(0x5080), /* Custom T540-cr */
> >>>>> CH_PCI_ID_TABLE_FENTRY(0x5081), /* Custom T540-LL-cr */
> >>>>> CH_PCI_ID_TABLE_FENTRY(0x5082), /* Custom T504-cr */
> >>>>> @@ -116,19 +122,63 @@ CH_PCI_DEVICE_ID_TABLE_DEFINE_BEGIN
> >>>>> CH_PCI_ID_TABLE_FENTRY(0x5090), /* Custom T540-CR */
> >>>>> CH_PCI_ID_TABLE_FENTRY(0x5091), /* Custom T522-CR */
> >>>>> CH_PCI_ID_TABLE_FENTRY(0x5092), /* Custom T520-CR */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x5093), /* Custom T580-LP-CR */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x5094), /* Custom T540-CR */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x5095), /* Custom T540-CR-SO */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x5096), /* Custom T580-CR */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x5097), /* Custom T520-KR */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x5098), /* Custom 2x40G QSFP */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x5099), /* Custom 2x40G QSFP */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x509A), /* Custom T520-CR */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x509B), /* Custom T540-CR LOM */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x509c), /* Custom T520-CR SFP+ LOM */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x509d), /* Custom T540-CR SFP+ */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x509e), /* Custom T520-CR */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x509f), /* Custom T540-CR */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x50a0), /* Custom T540-CR */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x50a1), /* Custom T540-CR */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x50a2), /* Custom T580-KR4 */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x50a3), /* Custom T580-KR4 */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x50a4), /* Custom 2x T540-CR */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x50a5), /* Custom T522-BT */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x50a6), /* Custom T522-BT-SO */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x50a7), /* Custom T580-CR */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x50a8), /* Custom T580-KR */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x50a9), /* Custom T580-KR */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x50aa), /* Custom T580-CR */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x50ab), /* Custom T520-CR */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x50ac), /* Custom T540-BT */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x50ad), /* Custom T520-CR */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x50ae), /* Custom T540-XL-SO */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x50af), /* Custom T580-KR-SO */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x50b0), /* Custom T520-CR-LOM */
> >>>>>
> >>>>> /* T6 adapter */
> >>>>> CH_PCI_ID_TABLE_FENTRY(0x6001), /* T6225-CR */
> >>>>> CH_PCI_ID_TABLE_FENTRY(0x6002), /* T6225-SO-CR */
> >>>>> CH_PCI_ID_TABLE_FENTRY(0x6003), /* T6425-CR */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x6004), /* T6425-SO-CR */
> >>>>> CH_PCI_ID_TABLE_FENTRY(0x6005), /* T6225-OCP */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x6006), /* T62100-OCP-SO */
> >>>>> CH_PCI_ID_TABLE_FENTRY(0x6007), /* T62100-LP-CR */
> >>>>> CH_PCI_ID_TABLE_FENTRY(0x6008), /* T62100-SO-CR */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x6009), /* T6210-BT */
> >>>>> CH_PCI_ID_TABLE_FENTRY(0x600d), /* T62100-CR */
> >>>>> CH_PCI_ID_TABLE_FENTRY(0x6011), /* T6225-LL-CR */
> >>>>> CH_PCI_ID_TABLE_FENTRY(0x6014), /* T61100-OCP-SO */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x6015), /* T6201-BT */
> >>>>> CH_PCI_ID_TABLE_FENTRY(0x6080), /* Custom T6225-CR SFP28 */
> >>>>> CH_PCI_ID_TABLE_FENTRY(0x6081), /* Custom T62100-CR */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x6082), /* Custom T6225-CR */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x6083), /* Custom T62100-CR */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x6084), /* Custom T64100-CR */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x6085), /* Custom T6240-SO */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x6086), /* Custom T6225-SO-CR */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x6087), /* Custom T6225-CR */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x6088), /* Custom T62100-CR */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x6089), /* Custom T62100-KR */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x608a), /* Custom T62100-CR */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x608b), /* Custom T6225-CR */
> >>>>> CH_PCI_DEVICE_ID_TABLE_DEFINE_END;
> >>>>>
> >>>>> #endif /* CH_PCI_DEVICE_ID_TABLE_DEFINE_BEGIN */
> >>>>>
> >>>>
> >>
>
^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [dpdk-dev] [PATCH] net/cxgbe: update Chelsio T5/T6 NIC device ids
2019-05-09 12:19 ` Rahul Lakkireddy
@ 2019-05-09 12:19 ` Rahul Lakkireddy
0 siblings, 0 replies; 24+ messages in thread
From: Rahul Lakkireddy @ 2019-05-09 12:19 UTC (permalink / raw)
To: Kevin Traynor
Cc: dev, Nirranjan Kirubaharan, Indranil Choudhury, stable,
Ferruh Yigit, Yongseok Koh, Luca Boccassi
On Thursday, May 05/09/19, 2019 at 17:25:56 +0530, Kevin Traynor wrote:
> On 09/05/2019 12:11, Rahul Lakkireddy wrote:
> > On Thursday, May 05/09/19, 2019 at 16:08:05 +0530, Kevin Traynor wrote:
> >> On 09/05/2019 11:03, Rahul Lakkireddy wrote:
> >>> On Thursday, May 05/09/19, 2019 at 14:29:05 +0530, Kevin Traynor wrote:
> >>>> On 19/04/2019 10:22, Rahul Lakkireddy wrote:
> >>>>> Fixes: 04868e5bfddd ("net/cxgbe: add support to run Chelsio T6 cards")
> >>>>> Cc: stable@dpdk.org
> >>>>>
> >>>>
> >>>> Hi Rahul,
> >>>>
> >>>> Which stable release(s) is this change requested for? Have you tested
> >>>> these devices on those branches to ensure they work with the cxgbe pmd
> >>>> on those branches?
> >>>>
> >>>> Kevin.
> >>>
> >>> Hi Kevin,
> >>>
> >>> Please target 18.11 and 19.02 branches. All these devices work fine
> >>> in these branches.
> >>>
> >>
> >> Sorry, but this is not clear. There is no 19.02 branch and for 18.11
> >> branch, I don't know if you tested some/all of the devices with that
> >> branch or are making assumption based on the cxgbe pmd in 18.11.
> >>
> >
> > Yes, these devices work fine by manually applying this patch on 18.11.1
> > and 19.02.
> >
>
> Thanks for clarifying.
>
> > I had thought 19.02 branch is available by now. Looks like this is not
> > the case. Please apply the patch to 19.02 branch as well when it becomes
> > available.
> >
>
> There won't be a 19.02 stable branch. tl;dr there wasn't a maintainer
> and not enough resources in the community for validating non-LTS
> stables. See policy: here
> http://doc.dpdk.org/guides/contributing/stable.html#stable-releases
>
I see. I wasn't aware of this. Thanks for clarifying.
Thanks,
Rahul
> >> I think these need to have some testing to be in 18.11.2. I will apply
> >> it, but requesting you to do some testing to check they're working as
> >> you expect when 18.11.2 RC is available (ETA for RC-1, week of 20th May).
> >>
> >
> > Sure, will test these devices on 18.11.2 RC as well.
> >
>
> ok, sounds good,
>
> Kevin.
>
> > Thanks,
> > Rahul
> >
> >> thanks,
> >> Kevin.
> >>
> >>> Thanks,
> >>> Rahul
> >>>
> >>>>
> >>>>> Signed-off-by: Rahul Lakkireddy <rahul.lakkireddy@chelsio.com>
> >>>>> ---
> >>>>> drivers/net/cxgbe/base/t4_pci_id_tbl.h | 52 +++++++++++++++++++++++++-
> >>>>> 1 file changed, 51 insertions(+), 1 deletion(-)
> >>>>>
> >>>>> diff --git a/drivers/net/cxgbe/base/t4_pci_id_tbl.h b/drivers/net/cxgbe/base/t4_pci_id_tbl.h
> >>>>> index 5f5cbe048..f5f027a2e 100644
> >>>>> --- a/drivers/net/cxgbe/base/t4_pci_id_tbl.h
> >>>>> +++ b/drivers/net/cxgbe/base/t4_pci_id_tbl.h
> >>>>> @@ -1,5 +1,5 @@
> >>>>> /* SPDX-License-Identifier: BSD-3-Clause
> >>>>> - * Copyright(c) 2014-2018 Chelsio Communications.
> >>>>> + * Copyright(c) 2014-2019 Chelsio Communications.
> >>>>> * All rights reserved.
> >>>>> */
> >>>>>
> >>>>> @@ -103,6 +103,12 @@ CH_PCI_DEVICE_ID_TABLE_DEFINE_BEGIN
> >>>>> CH_PCI_ID_TABLE_FENTRY(0x5013), /* T580-chr */
> >>>>> CH_PCI_ID_TABLE_FENTRY(0x5014), /* T580-so */
> >>>>> CH_PCI_ID_TABLE_FENTRY(0x5015), /* T502-bt */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x5016), /* T580-OCP-SO */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x5017), /* T520-OCP-SO */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x5018), /* T540-BT */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x5019), /* T540-LP-BT */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x501a), /* T540-SO-BT */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x501b), /* T540-SO-CR */
> >>>>> CH_PCI_ID_TABLE_FENTRY(0x5080), /* Custom T540-cr */
> >>>>> CH_PCI_ID_TABLE_FENTRY(0x5081), /* Custom T540-LL-cr */
> >>>>> CH_PCI_ID_TABLE_FENTRY(0x5082), /* Custom T504-cr */
> >>>>> @@ -116,19 +122,63 @@ CH_PCI_DEVICE_ID_TABLE_DEFINE_BEGIN
> >>>>> CH_PCI_ID_TABLE_FENTRY(0x5090), /* Custom T540-CR */
> >>>>> CH_PCI_ID_TABLE_FENTRY(0x5091), /* Custom T522-CR */
> >>>>> CH_PCI_ID_TABLE_FENTRY(0x5092), /* Custom T520-CR */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x5093), /* Custom T580-LP-CR */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x5094), /* Custom T540-CR */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x5095), /* Custom T540-CR-SO */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x5096), /* Custom T580-CR */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x5097), /* Custom T520-KR */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x5098), /* Custom 2x40G QSFP */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x5099), /* Custom 2x40G QSFP */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x509A), /* Custom T520-CR */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x509B), /* Custom T540-CR LOM */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x509c), /* Custom T520-CR SFP+ LOM */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x509d), /* Custom T540-CR SFP+ */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x509e), /* Custom T520-CR */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x509f), /* Custom T540-CR */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x50a0), /* Custom T540-CR */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x50a1), /* Custom T540-CR */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x50a2), /* Custom T580-KR4 */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x50a3), /* Custom T580-KR4 */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x50a4), /* Custom 2x T540-CR */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x50a5), /* Custom T522-BT */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x50a6), /* Custom T522-BT-SO */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x50a7), /* Custom T580-CR */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x50a8), /* Custom T580-KR */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x50a9), /* Custom T580-KR */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x50aa), /* Custom T580-CR */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x50ab), /* Custom T520-CR */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x50ac), /* Custom T540-BT */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x50ad), /* Custom T520-CR */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x50ae), /* Custom T540-XL-SO */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x50af), /* Custom T580-KR-SO */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x50b0), /* Custom T520-CR-LOM */
> >>>>>
> >>>>> /* T6 adapter */
> >>>>> CH_PCI_ID_TABLE_FENTRY(0x6001), /* T6225-CR */
> >>>>> CH_PCI_ID_TABLE_FENTRY(0x6002), /* T6225-SO-CR */
> >>>>> CH_PCI_ID_TABLE_FENTRY(0x6003), /* T6425-CR */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x6004), /* T6425-SO-CR */
> >>>>> CH_PCI_ID_TABLE_FENTRY(0x6005), /* T6225-OCP */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x6006), /* T62100-OCP-SO */
> >>>>> CH_PCI_ID_TABLE_FENTRY(0x6007), /* T62100-LP-CR */
> >>>>> CH_PCI_ID_TABLE_FENTRY(0x6008), /* T62100-SO-CR */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x6009), /* T6210-BT */
> >>>>> CH_PCI_ID_TABLE_FENTRY(0x600d), /* T62100-CR */
> >>>>> CH_PCI_ID_TABLE_FENTRY(0x6011), /* T6225-LL-CR */
> >>>>> CH_PCI_ID_TABLE_FENTRY(0x6014), /* T61100-OCP-SO */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x6015), /* T6201-BT */
> >>>>> CH_PCI_ID_TABLE_FENTRY(0x6080), /* Custom T6225-CR SFP28 */
> >>>>> CH_PCI_ID_TABLE_FENTRY(0x6081), /* Custom T62100-CR */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x6082), /* Custom T6225-CR */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x6083), /* Custom T62100-CR */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x6084), /* Custom T64100-CR */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x6085), /* Custom T6240-SO */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x6086), /* Custom T6225-SO-CR */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x6087), /* Custom T6225-CR */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x6088), /* Custom T62100-CR */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x6089), /* Custom T62100-KR */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x608a), /* Custom T62100-CR */
> >>>>> + CH_PCI_ID_TABLE_FENTRY(0x608b), /* Custom T6225-CR */
> >>>>> CH_PCI_DEVICE_ID_TABLE_DEFINE_END;
> >>>>>
> >>>>> #endif /* CH_PCI_DEVICE_ID_TABLE_DEFINE_BEGIN */
> >>>>>
> >>>>
> >>
>
^ permalink raw reply [flat|nested] 24+ messages in thread
end of thread, other threads:[~2019-05-09 12:24 UTC | newest]
Thread overview: 24+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-04-19 9:22 [dpdk-dev] [PATCH] net/cxgbe: update Chelsio T5/T6 NIC device ids Rahul Lakkireddy
2019-04-19 9:22 ` Rahul Lakkireddy
2019-04-19 9:30 ` Ferruh Yigit
2019-04-19 9:30 ` Ferruh Yigit
2019-04-19 9:49 ` Rahul Lakkireddy
2019-04-19 9:49 ` Rahul Lakkireddy
2019-04-19 11:18 ` Ferruh Yigit
2019-04-19 11:18 ` Ferruh Yigit
2019-04-19 12:10 ` Rahul Lakkireddy
2019-04-19 12:10 ` Rahul Lakkireddy
2019-04-19 12:24 ` Ferruh Yigit
2019-04-19 12:24 ` Ferruh Yigit
2019-05-09 8:59 ` Kevin Traynor
2019-05-09 8:59 ` Kevin Traynor
2019-05-09 10:03 ` Rahul Lakkireddy
2019-05-09 10:03 ` Rahul Lakkireddy
2019-05-09 10:38 ` Kevin Traynor
2019-05-09 10:38 ` Kevin Traynor
2019-05-09 11:11 ` Rahul Lakkireddy
2019-05-09 11:11 ` Rahul Lakkireddy
2019-05-09 11:55 ` Kevin Traynor
2019-05-09 11:55 ` Kevin Traynor
2019-05-09 12:19 ` Rahul Lakkireddy
2019-05-09 12:19 ` Rahul Lakkireddy
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).