> -----Original Message----- > From: Intel-wired-lan <intel-wired-lan-boun...@osuosl.org> On Behalf Of Simon > Horman > Sent: Sunday, December 17, 2023 3:15 PM > To: Brandeburg, Jesse <jesse.brandeb...@intel.com>; Nguyen, Anthony L > <anthony.l.ngu...@intel.com> > Cc: net...@vger.kernel.org; l...@lists.linux.dev; Nick Desaulniers > <ndesaulni...@google.com>; Nathan Chancellor <nat...@kernel.org>; Eric > Dumazet <eduma...@google.com>; intel-wired-...@lists.osuosl.org; Bill > Wendling <mo...@google.com>; Justin Stitt <justinst...@google.com>; Jakub > Kicinski <k...@kernel.org>; Paolo Abeni <pab...@redhat.com>; David S. Miller > <da...@davemloft.net> > Subject: [Intel-wired-lan] [PATCH iwl-next] i40e: Avoid unnecessary use of > comma operator > > 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-17 -Wcomma > > No functional change intended. > Compile tested only. > > Signed-off-by: Simon Horman <ho...@kernel.org> > --- > drivers/net/ethernet/intel/i40e/i40e_ethtool.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) >
Tested-by: Pucha Himasekhar Reddy <himasekharx.reddy.pu...@intel.com> (A Contingent worker at Intel)