Commit 55f96e8b authored by Simon Horman's avatar Simon Horman Committed by Tony Nguyen
Browse files

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: default avatarSimon Horman <horms@kernel.org>
Reviewed-by: default avatarNick Desaulniers <ndesaulniers@google.com>
Tested-by: Pucha Himasekhar Reddy <himasekharx.reddy.pucha@intel.com> (A Contingent worker at Intel)
Signed-off-by: default avatarTony Nguyen <anthony.l.nguyen@intel.com>
parent 31deb12e

drivers/net/ethernet/intel/i40e/i40e_ethtool.c

+1 −1
Original line number Diff line number Diff line
@@ -1911,7 +1911,7 @@ static int i40e_get_eeprom(struct net_device *netdev, 
			len = eeprom->len - (I40E_NVM_SECTOR_SIZE * i);

			last = true;

		}

		offset = eeprom->offset + (I40E_NVM_SECTOR_SIZE * i),

		offset = eeprom->offset + (I40E_NVM_SECTOR_SIZE * i);

		ret_val = i40e_aq_read_nvm(hw, 0x0, offset, len,

				(u8 *)eeprom_buff + (I40E_NVM_SECTOR_SIZE * i),

				last, NULL);