aboutsummaryrefslogtreecommitdiffstats
path: root/drivers
diff options
context:
space:
mode:
authorzhong jiang <zhongjiang@huawei.com>2018-09-20 16:13:47 +0800
committerDavid S. Miller <davem@davemloft.net>2018-09-21 09:00:55 -0700
commit68c385079cb3530f03e22e563885e6ddaa11a586 (patch)
tree5b586c6d73c85bc282f0a61c81ff406774abe63f /drivers
parentnet: amd: remove redundant continue (diff)
downloadlinux-dev-68c385079cb3530f03e22e563885e6ddaa11a586.tar.xz
linux-dev-68c385079cb3530f03e22e563885e6ddaa11a586.zip
net: neterion: remove redundant continue
The continue will not truely skip any code. hence it is safe to remove it. Signed-off-by: zhong jiang <zhongjiang@huawei.com> Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers')
-rw-r--r--drivers/net/ethernet/neterion/s2io.c4
1 files changed, 1 insertions, 3 deletions
diff --git a/drivers/net/ethernet/neterion/s2io.c b/drivers/net/ethernet/neterion/s2io.c
index 626603266ca7..82be90075695 100644
--- a/drivers/net/ethernet/neterion/s2io.c
+++ b/drivers/net/ethernet/neterion/s2io.c
@@ -3680,11 +3680,9 @@ static void restore_xmsi_data(struct s2io_nic *nic)
writeq(nic->msix_info[i].data, &bar0->xmsi_data);
val64 = (s2BIT(7) | s2BIT(15) | vBIT(msix_index, 26, 6));
writeq(val64, &bar0->xmsi_access);
- if (wait_for_msix_trans(nic, msix_index)) {
+ if (wait_for_msix_trans(nic, msix_index))
DBG_PRINT(ERR_DBG, "%s: index: %d failed\n",
__func__, msix_index);
- continue;
- }
}
}