patch 'net/bnxt: fix commas instead of semicolons' has been queued to stable release 24.11.5

luca.boccassi at gmail.com luca.boccassi at gmail.com
Thu Mar 19 23:02:06 CET 2026


Hi,

FYI, your patch has been queued to stable release 24.11.5

Note it hasn't been pushed to http://dpdk.org/browse/dpdk-stable yet.
It will be pushed if I get no objections before 03/21/26. So please
shout if anyone has objections.

Also note that after the patch there's a diff of the upstream commit vs the
patch applied to the branch. This will indicate if there was any rebasing
needed to apply to the stable branch. If there were code changes for rebasing
(ie: not only metadata diffs), please double check that the rebase was
correctly done.

Queued patches are on a temporary branch at:
https://github.com/bluca/dpdk-stable

This queued commit can be viewed at:
https://github.com/bluca/dpdk-stable/commit/166c58366d7417050ba5fa394ec374ecb8ad5884

Thanks.

Luca Boccassi

---
>From 166c58366d7417050ba5fa394ec374ecb8ad5884 Mon Sep 17 00:00:00 2001
From: Keegan Freyhof <keegan.freyhof at broadcom.com>
Date: Fri, 27 Feb 2026 13:00:19 -0700
Subject: [PATCH] net/bnxt: fix commas instead of semicolons

[ upstream commit e99612b26f6be4324ba0af372964d883789d9345 ]

Fix compile warnings seen when using clang with the -Wcomma flag
in the bnxt ULP driver code by replacing commas with semicolons.

Bugzilla ID: 1807
Fixes: dd0191d5e70d ("net/bnxt/tf_ulp: support Thor2 ULP layer")

Signed-off-by: Keegan Freyhof <keegan.freyhof at broadcom.com>
Signed-off-by: Mohammad Shuab Siddique <mohammad-shuab.siddique at broadcom.com>
Acked-by: Kishore Padmanabha <kishore.padmanabha at broadcom.com>
---
 .mailmap                              | 4 ++++
 drivers/net/bnxt/tf_ulp/bnxt_ulp_tf.c | 8 ++++----
 2 files changed, 8 insertions(+), 4 deletions(-)

diff --git a/.mailmap b/.mailmap
index c9c893660e..34e86111c0 100644
--- a/.mailmap
+++ b/.mailmap
@@ -806,6 +806,9 @@ Karol Kolacinski <karol.kolacinski at intel.com>
 Karra Satwik <kaara.satwik at chelsio.com>
 Kathleen Capella <kathleen.capella at arm.com>
 Kazatsker Kirill <kirill.kazatsker at intel.com>
+Ke Xu <ke1.xu at intel.com>
+Ke Zhang <ke1x.zhang at intel.com>
+Keegan Freyhof <keegan.freyhof at broadcom.com>
 Kefu Chai <tchaikov at gmail.com>
 Keiichi Watanabe <keiichiw at chromium.org>
 Keith Wiles <keith.wiles at intel.com> <keith.wiles at windriver.com>
@@ -1071,6 +1074,7 @@ Mohamad Noor Alim Hussin <mohamad.noor.alim.hussin at intel.com>
 Mohamed Feroz Abdul Majeeth <mabdulmajeet at marvell.com>
 Mohammad Abdul Awal <mohammad.abdul.awal at intel.com>
 Mohammad Iqbal Ahmad <mahmad at marvell.com>
+Mohammad Shuab Siddique <mohammad-shuab.siddique at broadcom.com>
 Mohammed Gamal <mgamal at redhat.com>
 Mohand Alrasheed <mrasheed at wirefilter.com>
 Mohsin Kazmi <mohsin.kazmi14 at gmail.com>
diff --git a/drivers/net/bnxt/tf_ulp/bnxt_ulp_tf.c b/drivers/net/bnxt/tf_ulp/bnxt_ulp_tf.c
index fa7e8fe000..12d5dc530c 100644
--- a/drivers/net/bnxt/tf_ulp/bnxt_ulp_tf.c
+++ b/drivers/net/bnxt/tf_ulp/bnxt_ulp_tf.c
@@ -1299,10 +1299,10 @@ ulp_tf_global_cfg_update(struct bnxt *bp,
 	struct tf *tfp;
 
 	/* Initialize the params */
-	parms.dir = dir,
-	parms.type = type,
-	parms.offset = offset,
-	parms.config = (uint8_t *)&global_cfg,
+	parms.dir = dir;
+	parms.type = type;
+	parms.offset = offset;
+	parms.config = (uint8_t *)&global_cfg;
 	parms.config_sz_in_bytes = sizeof(global_cfg);
 
 	tfp = bnxt_ulp_bp_tfp_get(bp, BNXT_ULP_SESSION_TYPE_DEFAULT);
-- 
2.47.3

---
  Diff of the applied patch vs upstream commit (please double-check if non-empty:
---
--- -	2026-03-19 22:00:48.341543608 +0000
+++ 0014-net-bnxt-fix-commas-instead-of-semicolons.patch	2026-03-19 22:00:47.782359370 +0000
@@ -1 +1 @@
-From e99612b26f6be4324ba0af372964d883789d9345 Mon Sep 17 00:00:00 2001
+From 166c58366d7417050ba5fa394ec374ecb8ad5884 Mon Sep 17 00:00:00 2001
@@ -5,0 +6,2 @@
+[ upstream commit e99612b26f6be4324ba0af372964d883789d9345 ]
+
@@ -11 +12,0 @@
-Cc: stable at dpdk.org
@@ -17 +18 @@
- .mailmap                              | 2 ++
+ .mailmap                              | 4 ++++
@@ -19 +20 @@
- 2 files changed, 6 insertions(+), 4 deletions(-)
+ 2 files changed, 8 insertions(+), 4 deletions(-)
@@ -22 +23 @@
-index 0a02a23fe4..accbb29827 100644
+index c9c893660e..34e86111c0 100644
@@ -25 +26,3 @@
-@@ -841,6 +841,7 @@ Kathleen Capella <kathleen.capella at arm.com>
+@@ -806,6 +806,9 @@ Karol Kolacinski <karol.kolacinski at intel.com>
+ Karra Satwik <kaara.satwik at chelsio.com>
+ Kathleen Capella <kathleen.capella at arm.com>
@@ -27,2 +30,2 @@
- Ke Xu <ke1.xu at intel.com>
- Ke Zhang <ke1x.zhang at intel.com>
++Ke Xu <ke1.xu at intel.com>
++Ke Zhang <ke1x.zhang at intel.com>
@@ -33 +36 @@
-@@ -1113,6 +1114,7 @@ Mohamad Noor Alim Hussin <mohamad.noor.alim.hussin at intel.com>
+@@ -1071,6 +1074,7 @@ Mohamad Noor Alim Hussin <mohamad.noor.alim.hussin at intel.com>
@@ -42 +45 @@
-index 38cb5c8b31..bc347de202 100644
+index fa7e8fe000..12d5dc530c 100644
@@ -45 +48 @@
-@@ -1296,10 +1296,10 @@ ulp_tf_global_cfg_update(struct bnxt *bp,
+@@ -1299,10 +1299,10 @@ ulp_tf_global_cfg_update(struct bnxt *bp,


More information about the stable mailing list