Commit 7a208e83 authored by Kamil Krawczyk's avatar Kamil Krawczyk Committed by Jeff Kirsher

i40e: do not take NVM ownership for SR read

We do not need to acquire NVM for Shadow RAM XSUM calculation, as we only
read from SR through SRCTL register for which having the ownership is not
required.

Change-ID: Ie238a8f09917d1d25f24cc7cec271951ac7b98f2
Signed-off-by: default avatarKamil Krawczyk <kamil.krawczyk@intel.com>
Signed-off-by: default avatarJeff Kirsher <jeffrey.t.kirsher@intel.com>
parent 9b8d90b9
...@@ -324,13 +324,9 @@ i40e_status i40e_validate_nvm_checksum(struct i40e_hw *hw, ...@@ -324,13 +324,9 @@ i40e_status i40e_validate_nvm_checksum(struct i40e_hw *hw,
u16 checksum_sr = 0; u16 checksum_sr = 0;
u16 checksum_local = 0; u16 checksum_local = 0;
ret_code = i40e_acquire_nvm(hw, I40E_RESOURCE_READ);
if (ret_code)
goto i40e_validate_nvm_checksum_exit;
ret_code = i40e_calc_nvm_checksum(hw, &checksum_local); ret_code = i40e_calc_nvm_checksum(hw, &checksum_local);
if (ret_code) if (ret_code)
goto i40e_validate_nvm_checksum_free; goto i40e_validate_nvm_checksum_exit;
/* Do not use i40e_read_nvm_word() because we do not want to take /* Do not use i40e_read_nvm_word() because we do not want to take
* the synchronization semaphores twice here. * the synchronization semaphores twice here.
...@@ -347,9 +343,6 @@ i40e_status i40e_validate_nvm_checksum(struct i40e_hw *hw, ...@@ -347,9 +343,6 @@ i40e_status i40e_validate_nvm_checksum(struct i40e_hw *hw,
if (checksum) if (checksum)
*checksum = checksum_local; *checksum = checksum_local;
i40e_validate_nvm_checksum_free:
i40e_release_nvm(hw);
i40e_validate_nvm_checksum_exit: i40e_validate_nvm_checksum_exit:
return ret_code; return ret_code;
} }
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment