diff options
author | Colin Ian King <colin.king@canonical.com> | 2018-07-04 08:13:01 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2018-07-04 09:39:11 -0400 |
commit | 2e6dde5c618f847fdb811373ad520cdb3a4425ab (patch) | |
tree | 86cd4449bb1d80fd494579266797cbffac9b3904 | |
parent | 9b0bb10a84ffbbbb9f589122e34ceee0194ea5ca (diff) |
sfc: remove redundant variable old_vlan
Variable old_vlan is being assigned but is never used hence it is
and can be removed.
Cleans up clang warning:
warning: variable 'old_vlan' set but not used [-Wunused-but-set-variable]
Signed-off-by: Colin Ian King <colin.king@canonical.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
-rw-r--r-- | drivers/net/ethernet/sfc/ef10_sriov.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/net/ethernet/sfc/ef10_sriov.c b/drivers/net/ethernet/sfc/ef10_sriov.c index 8820be83ce85..3d76fd1504c2 100644 --- a/drivers/net/ethernet/sfc/ef10_sriov.c +++ b/drivers/net/ethernet/sfc/ef10_sriov.c | |||
@@ -564,7 +564,7 @@ int efx_ef10_sriov_set_vf_vlan(struct efx_nic *efx, int vf_i, u16 vlan, | |||
564 | { | 564 | { |
565 | struct efx_ef10_nic_data *nic_data = efx->nic_data; | 565 | struct efx_ef10_nic_data *nic_data = efx->nic_data; |
566 | struct ef10_vf *vf; | 566 | struct ef10_vf *vf; |
567 | u16 old_vlan, new_vlan; | 567 | u16 new_vlan; |
568 | int rc = 0, rc2 = 0; | 568 | int rc = 0, rc2 = 0; |
569 | 569 | ||
570 | if (vf_i >= efx->vf_count) | 570 | if (vf_i >= efx->vf_count) |
@@ -619,7 +619,6 @@ int efx_ef10_sriov_set_vf_vlan(struct efx_nic *efx, int vf_i, u16 vlan, | |||
619 | } | 619 | } |
620 | 620 | ||
621 | /* Do the actual vlan change */ | 621 | /* Do the actual vlan change */ |
622 | old_vlan = vf->vlan; | ||
623 | vf->vlan = new_vlan; | 622 | vf->vlan = new_vlan; |
624 | 623 | ||
625 | /* Restore everything in reverse order */ | 624 | /* Restore everything in reverse order */ |