net/mlx5: Avoid unnecessary use of comma operator
authorSimon Horman <horms@kernel.org>
Fri, 7 Mar 2025 12:39:33 +0000 (12:39 +0000)
committerJakub Kicinski <kuba@kernel.org>
Wed, 12 Mar 2025 20:22:48 +0000 (13:22 -0700)
Although it does not seem to have any untoward side-effects,
the use of ';' to separate to assignments seems more appropriate than ','.

Flagged by clang-19 -Wcomma

No functional change intended.
Compile tested only.

Signed-off-by: Simon Horman <horms@kernel.org>
Reviewed-by: Michal Swiatkowski <michal.swiatkowski@linux.intel.com>
Reviewed-by: Tariq Toukan <tariqt@nvidia.com>
Link: https://patch.msgid.link/20250307-mlx5-comma-v1-1-934deb6927bb@kernel.org
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
drivers/net/ethernet/mellanox/mlx5/core/lib/fs_chains.c

index c862dd28c46619f086671f8509b1c085fea88185..e8cc91a9bd827afa6d0537325dff35367ca7f5ec 100644 (file)
@@ -700,7 +700,7 @@ mlx5_chains_create_global_table(struct mlx5_fs_chains *chains)
                goto err_ignore;
        }
 
-       chain = mlx5_chains_get_chain_range(chains),
+       chain = mlx5_chains_get_chain_range(chains);
        prio = mlx5_chains_get_prio_range(chains);
        level = mlx5_chains_get_level_range(chains);