On Thu, Mar 12, 2026 at 09:43:44AM -0700, Stephen Hemminger wrote:
> Fix comma operator misuse warnings with clang compiler in idpf that
> was missed in previous change. Shows up when -Wcomma enabled.
>
> Fixes: 57560a92167a ("net/idpf: add AVX2 Tx path for split queue config")
>
> Signed-off-by: Stephen Hemminger <[email protected]>
> ---
Acked-by: Bruce Richardson <[email protected]>
Yep, this was missed because the previous commit [1] was on your tree but
not applied to main, so missing in mine. :-(
[1]
https://patches.dpdk.org/project/dpdk/patch/[email protected]/
> drivers/net/intel/idpf/idpf_common_rxtx_avx2.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/net/intel/idpf/idpf_common_rxtx_avx2.c
> b/drivers/net/intel/idpf/idpf_common_rxtx_avx2.c
> index 4e1062f22d..db7728afad 100644
> --- a/drivers/net/intel/idpf/idpf_common_rxtx_avx2.c
> +++ b/drivers/net/intel/idpf/idpf_common_rxtx_avx2.c
> @@ -842,7 +842,7 @@ idpf_splitq_vtx_avx2(struct idpf_flex_tx_sched_desc *txdp,
> /* align if needed */
> if (((uintptr_t)txdp & 0x1F) != 0 && nb_pkts != 0) {
> idpf_splitq_vtx1_avx2(txdp, *pkt, flags);
> - txdp++, pkt++, nb_pkts--;
> + txdp++; pkt++; nb_pkts--;
> }
>
> for (; nb_pkts >= IDPF_VPMD_DESCS_PER_LOOP; txdp +=
> IDPF_VPMD_DESCS_PER_LOOP,
> --
> 2.51.0
>
