net: neterion: remove redundant continue
authorzhong jiang <zhongjiang@huawei.com>
Thu, 20 Sep 2018 08:13:47 +0000 (16:13 +0800)
committerDavid S. Miller <davem@davemloft.net>
Fri, 21 Sep 2018 16:00:55 +0000 (09:00 -0700)
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>
drivers/net/ethernet/neterion/s2io.c

index 626603266ca792eb6e7b85b696474a6a34ad7461..82be900756952d137e3325a17c77306bcc865c54 100644 (file)
@@ -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;
-               }
        }
 }