aboutsummaryrefslogtreecommitdiffstats
path: root/drivers/net/ixgbe
diff options
context:
space:
mode:
authorEmil Tantilov <emil.s.tantilov@intel.com>2011-03-12 03:56:38 -0500
committerJeff Kirsher <jeffrey.t.kirsher@intel.com>2011-04-13 22:20:15 -0400
commit2ea5ea5fc4f4f1daa74708c2a14e364d8474812d (patch)
tree2239644057f59379b5cbc315378f12b85961a509 /drivers/net/ixgbe
parentc8ca76ebc6e50752c5311b92bb9aef7edb324577 (diff)
ixgbe: fix return value checks
The value of status was incorrectly tested. Also whitespace cleanup. Signed-off-by: Emil Tantilov <emil.s.tantilov@intel.com> Tested-by: Evan Swanson <evan.swanson@intel.com> Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
Diffstat (limited to 'drivers/net/ixgbe')
-rw-r--r--drivers/net/ixgbe/ixgbe_x540.c5
1 files changed, 2 insertions, 3 deletions
diff --git a/drivers/net/ixgbe/ixgbe_x540.c b/drivers/net/ixgbe/ixgbe_x540.c
index d9323c08f5c7..7ce3f45cad71 100644
--- 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)
452 IXGBE_WRITE_REG(hw, IXGBE_EEC, flup); 452 IXGBE_WRITE_REG(hw, IXGBE_EEC, flup);
453 453
454 status = ixgbe_poll_flash_update_done_X540(hw); 454 status = ixgbe_poll_flash_update_done_X540(hw);
455 if (status) 455 if (status == 0)
456 hw_dbg(hw, "Flash update complete\n"); 456 hw_dbg(hw, "Flash update complete\n");
457 else 457 else
458 hw_dbg(hw, "Flash update time out\n"); 458 hw_dbg(hw, "Flash update time out\n");
@@ -466,11 +466,10 @@ static s32 ixgbe_update_flash_X540(struct ixgbe_hw *hw)
466 } 466 }
467 467
468 status = ixgbe_poll_flash_update_done_X540(hw); 468 status = ixgbe_poll_flash_update_done_X540(hw);
469 if (status) 469 if (status == 0)
470 hw_dbg(hw, "Flash update complete\n"); 470 hw_dbg(hw, "Flash update complete\n");
471 else 471 else
472 hw_dbg(hw, "Flash update time out\n"); 472 hw_dbg(hw, "Flash update time out\n");
473
474 } 473 }
475out: 474out:
476 return status; 475 return status;