i40evf: remove unnecessary else
authorMitch Williams <mitch.a.williams@intel.com>
Tue, 11 Nov 2014 20:03:13 +0000 (20:03 +0000)
committerJeff Kirsher <jeffrey.t.kirsher@intel.com>
Thu, 20 Nov 2014 22:56:44 +0000 (14:56 -0800)
Since the if part of this statement contains a break, there's no reason
for the else. Clean up the code and make it more obvious that the delay
happens each time through the loop.

Change-ID: I9292eaf7dd687688bdc401b8bd8d1d14f6944460
Signed-off-by: Mitch Williams <mitch.a.williams@intel.com>
Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
drivers/net/ethernet/intel/i40evf/i40evf_main.c

index 233710c91c84ba78872b43fb26fff317dc3ba117..8e01009695da87cbdbc2f5950726377272550b65 100644 (file)
@@ -1513,8 +1513,7 @@ static void i40evf_reset_task(struct work_struct *work)
                if ((rstat_val != I40E_VFR_VFACTIVE) &&
                    (rstat_val != I40E_VFR_COMPLETED))
                        break;
-               else
-                       msleep(I40EVF_RESET_WAIT_MS);
+               msleep(I40EVF_RESET_WAIT_MS);
        }
        if (i == I40EVF_RESET_WAIT_COUNT) {
                adapter->flags &= ~I40EVF_FLAG_RESET_PENDING;
@@ -1528,8 +1527,7 @@ static void i40evf_reset_task(struct work_struct *work)
                if ((rstat_val == I40E_VFR_VFACTIVE) ||
                    (rstat_val == I40E_VFR_COMPLETED))
                        break;
-               else
-                       msleep(I40EVF_RESET_WAIT_MS);
+               msleep(I40EVF_RESET_WAIT_MS);
        }
        if (i == I40EVF_RESET_WAIT_COUNT) {
                struct i40evf_mac_filter *f, *ftmp;