ice: remove unnecessary VSI assignment

anguy11 · anguy11 · commit 70fa0a078099 · 2021-06-25T11:30:50.000-07:00
ice_get_vf_vsi() is being called twice for the same VSI. Remove the
unnecessary call/assignment.

Signed-off-by: Tony Nguyen &lt;anthony.l.nguyen@intel.com&gt;
Tested-by: Tony Brelinski &lt;tonyx.brelinski@intel.com&gt;
diff --git a/drivers/net/ethernet/intel/ice/ice_virtchnl_pf.c b/drivers/net/ethernet/intel/ice/ice_virtchnl_pf.c
@@ -1689,7 +1689,6 @@ bool ice_reset_vf(struct ice_vf *vf, bool is_vflr)
 		else
 			promisc_m = ICE_UCAST_PROMISC_BITS;
 
-		vsi = ice_get_vf_vsi(vf);
 		if (ice_vf_set_vsi_promisc(vf, vsi, promisc_m, true))
 			dev_err(dev, "disabling promiscuous mode failed\n");
 	}

Original file line number	Diff line number	Diff line change
`@@ -1689,7 +1689,6 @@ bool ice_reset_vf(struct ice_vf *vf, bool is_vflr)`
`1689`	`1689`	`else`
`1690`	`1690`	`promisc_m = ICE_UCAST_PROMISC_BITS;`
`1691`	`1691`
`1692`		`- vsi = ice_get_vf_vsi(vf);`
`1693`	`1692`	`if (ice_vf_set_vsi_promisc(vf, vsi, promisc_m, true))`
`1694`	`1693`	`dev_err(dev, "disabling promiscuous mode failed\n");`
`1695`	`1694`	`}`