ixgbe: fix return value checks

The value of status was incorrectly tested. Also whitespace cleanup. Signed-off-by: Emil Tantilov <[email protected]> Tested-by: Evan Swanson <[email protected]> Signed-off-by: Jeff Kirsher <[email protected]>
maraz · Apr 14, 2011 · 2ea5ea5 · 2ea5ea5
1 parent c8ca76e
commit 2ea5ea5
Showing 1 changed file with 2 additions and 3 deletions.
diff --git a/drivers/net/ixgbe/ixgbe_x540.c b/drivers/net/ixgbe/ixgbe_x540.c
@@ -452,7 +452,7 @@ static s32 ixgbe_update_flash_X540(struct ixgbe_hw *hw)
 	IXGBE_WRITE_REG(hw, IXGBE_EEC, flup);
 
 	status = ixgbe_poll_flash_update_done_X540(hw);
-	if (status)
+	if (status == 0)
 		hw_dbg(hw, "Flash update complete\n");
 	else
 		hw_dbg(hw, "Flash update time out\n");
@@ -466,11 +466,10 @@ static s32 ixgbe_update_flash_X540(struct ixgbe_hw *hw)
 		}
 
 		status = ixgbe_poll_flash_update_done_X540(hw);
-		if (status)
+		if (status == 0)
 			hw_dbg(hw, "Flash update complete\n");
 		else
 			hw_dbg(hw, "Flash update time out\n");
-
 	}
 out:
 	return status;