i40e: Remove unused i40e_commit_partition_bw_setting
i40e_commit_partition_bw_setting() was added in 2017 by
commit 4fc8c6763957 ("i40e: genericize the partition bandwidth control")
but hasn't been used.
Remove it.
Signed-off-by: Dr. David Alan Gilbert <linux@treblig.org>
Reviewed-by: Kalesh AP <kalesh-anakkur.purayil@broadcom.com>
Reviewed-by: Tony Nguyen <anthony.l.nguyen@intel.com>
Link: https://patch.msgid.link/20250102173717.200359-8-linux@treblig.org
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
This commit is contained in:
parent
38dfb07d9a
commit
a324484ac8
@ -1311,7 +1311,6 @@ int i40e_update_adq_vsi_queues(struct i40e_vsi *vsi, int vsi_offset);
|
||||
int i40e_is_vsi_uplink_mode_veb(struct i40e_vsi *vsi);
|
||||
int i40e_get_partition_bw_setting(struct i40e_pf *pf);
|
||||
int i40e_set_partition_bw_setting(struct i40e_pf *pf);
|
||||
int i40e_commit_partition_bw_setting(struct i40e_pf *pf);
|
||||
void i40e_print_link_message(struct i40e_vsi *vsi, bool isup);
|
||||
|
||||
void i40e_set_fec_in_flags(u8 fec_cfg, unsigned long *flags);
|
||||
|
||||
@ -12576,89 +12576,6 @@ int i40e_set_partition_bw_setting(struct i40e_pf *pf)
|
||||
return status;
|
||||
}
|
||||
|
||||
/**
|
||||
* i40e_commit_partition_bw_setting - Commit BW settings for this PF partition
|
||||
* @pf: board private structure
|
||||
**/
|
||||
int i40e_commit_partition_bw_setting(struct i40e_pf *pf)
|
||||
{
|
||||
/* Commit temporary BW setting to permanent NVM image */
|
||||
enum i40e_admin_queue_err last_aq_status;
|
||||
u16 nvm_word;
|
||||
int ret;
|
||||
|
||||
if (pf->hw.partition_id != 1) {
|
||||
dev_info(&pf->pdev->dev,
|
||||
"Commit BW only works on partition 1! This is partition %d",
|
||||
pf->hw.partition_id);
|
||||
ret = -EOPNOTSUPP;
|
||||
goto bw_commit_out;
|
||||
}
|
||||
|
||||
/* Acquire NVM for read access */
|
||||
ret = i40e_acquire_nvm(&pf->hw, I40E_RESOURCE_READ);
|
||||
last_aq_status = pf->hw.aq.asq_last_status;
|
||||
if (ret) {
|
||||
dev_info(&pf->pdev->dev,
|
||||
"Cannot acquire NVM for read access, err %pe aq_err %s\n",
|
||||
ERR_PTR(ret),
|
||||
i40e_aq_str(&pf->hw, last_aq_status));
|
||||
goto bw_commit_out;
|
||||
}
|
||||
|
||||
/* Read word 0x10 of NVM - SW compatibility word 1 */
|
||||
ret = i40e_aq_read_nvm(&pf->hw,
|
||||
I40E_SR_NVM_CONTROL_WORD,
|
||||
0x10, sizeof(nvm_word), &nvm_word,
|
||||
false, NULL);
|
||||
/* Save off last admin queue command status before releasing
|
||||
* the NVM
|
||||
*/
|
||||
last_aq_status = pf->hw.aq.asq_last_status;
|
||||
i40e_release_nvm(&pf->hw);
|
||||
if (ret) {
|
||||
dev_info(&pf->pdev->dev, "NVM read error, err %pe aq_err %s\n",
|
||||
ERR_PTR(ret),
|
||||
i40e_aq_str(&pf->hw, last_aq_status));
|
||||
goto bw_commit_out;
|
||||
}
|
||||
|
||||
/* Wait a bit for NVM release to complete */
|
||||
msleep(50);
|
||||
|
||||
/* Acquire NVM for write access */
|
||||
ret = i40e_acquire_nvm(&pf->hw, I40E_RESOURCE_WRITE);
|
||||
last_aq_status = pf->hw.aq.asq_last_status;
|
||||
if (ret) {
|
||||
dev_info(&pf->pdev->dev,
|
||||
"Cannot acquire NVM for write access, err %pe aq_err %s\n",
|
||||
ERR_PTR(ret),
|
||||
i40e_aq_str(&pf->hw, last_aq_status));
|
||||
goto bw_commit_out;
|
||||
}
|
||||
/* Write it back out unchanged to initiate update NVM,
|
||||
* which will force a write of the shadow (alt) RAM to
|
||||
* the NVM - thus storing the bandwidth values permanently.
|
||||
*/
|
||||
ret = i40e_aq_update_nvm(&pf->hw,
|
||||
I40E_SR_NVM_CONTROL_WORD,
|
||||
0x10, sizeof(nvm_word),
|
||||
&nvm_word, true, 0, NULL);
|
||||
/* Save off last admin queue command status before releasing
|
||||
* the NVM
|
||||
*/
|
||||
last_aq_status = pf->hw.aq.asq_last_status;
|
||||
i40e_release_nvm(&pf->hw);
|
||||
if (ret)
|
||||
dev_info(&pf->pdev->dev,
|
||||
"BW settings NOT SAVED, err %pe aq_err %s\n",
|
||||
ERR_PTR(ret),
|
||||
i40e_aq_str(&pf->hw, last_aq_status));
|
||||
bw_commit_out:
|
||||
|
||||
return ret;
|
||||
}
|
||||
|
||||
/**
|
||||
* i40e_is_total_port_shutdown_enabled - read NVM and return value
|
||||
* if total port shutdown feature is enabled for this PF
|
||||
|
||||
Loading…
Reference in New Issue
Block a user