Skip to content

Commit eeeddbb

Browse files
jacob-kellerJeff Kirsher
authored andcommitted
i40e: drop i40e_pf *pf from i40e_vc_disable_vf()
It's never used, and the vf structure could get back to the PF if necessary. Lets just drop the extra unneeded parameter. Signed-off-by: Jacob Keller <[email protected]> Tested-by: Andrew Bowers <[email protected]> Signed-off-by: Jeff Kirsher <[email protected]>
1 parent ba4e003 commit eeeddbb

File tree

1 file changed

+3
-4
lines changed

1 file changed

+3
-4
lines changed

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

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -154,12 +154,11 @@ void i40e_vc_notify_vf_reset(struct i40e_vf *vf)
154154

155155
/**
156156
* i40e_vc_disable_vf
157-
* @pf: pointer to the PF info
158157
* @vf: pointer to the VF info
159158
*
160159
* Disable the VF through a SW reset
161160
**/
162-
static inline void i40e_vc_disable_vf(struct i40e_pf *pf, struct i40e_vf *vf)
161+
static inline void i40e_vc_disable_vf(struct i40e_vf *vf)
163162
{
164163
i40e_vc_notify_vf_reset(vf);
165164
i40e_reset_vf(vf, false);
@@ -2918,7 +2917,7 @@ int i40e_ndo_set_vf_mac(struct net_device *netdev, int vf_id, u8 *mac)
29182917
}
29192918

29202919
/* Force the VF driver stop so it has to reload with new MAC address */
2921-
i40e_vc_disable_vf(pf, vf);
2920+
i40e_vc_disable_vf(vf);
29222921
dev_info(&pf->pdev->dev, "Reload the VF driver to make this change effective.\n");
29232922

29242923
error_param:
@@ -3013,7 +3012,7 @@ int i40e_ndo_set_vf_port_vlan(struct net_device *netdev, int vf_id,
30133012
* the right thing by reconfiguring his network correctly
30143013
* and then reloading the VF driver.
30153014
*/
3016-
i40e_vc_disable_vf(pf, vf);
3015+
i40e_vc_disable_vf(vf);
30173016
/* During reset the VF got a new VSI, so refresh the pointer. */
30183017
vsi = pf->vsi[vf->lan_vsi_idx];
30193018
}

0 commit comments

Comments
 (0)