Commit ad739d08 authored by Tom Rix's avatar Tom Rix Committed by Tony Nguyen
Browse files

i40e: little endian only valid checksums



The calculation of the checksum can fail.
So move converting the checksum to little endian
to inside the return status check.

Signed-off-by: default avatarTom Rix <trix@redhat.com>
Tested-by: Gurucharan G <gurucharanx.g@intel.com> (A Contingent worker at Intel)
Signed-off-by: default avatarTony Nguyen <anthony.l.nguyen@intel.com>
parent 1abea24a
Loading
Loading
Loading
Loading
+3 −2
Original line number Diff line number Diff line
@@ -682,10 +682,11 @@ i40e_status i40e_update_nvm_checksum(struct i40e_hw *hw)
	__le16 le_sum;

	ret_code = i40e_calc_nvm_checksum(hw, &checksum);
	if (!ret_code) {
		le_sum = cpu_to_le16(checksum);
	if (!ret_code)
		ret_code = i40e_write_nvm_aq(hw, 0x00, I40E_SR_SW_CHECKSUM_WORD,
					     1, &le_sum, true);
	}

	return ret_code;
}