ixgbe: fix obvious return value bug.
authorDon Skidmore <donald.c.skidmore@intel.com>
Fri, 17 Feb 2012 07:38:13 +0000 (07:38 +0000)
committerJeff Kirsher <jeffrey.t.kirsher@intel.com>
Wed, 14 Mar 2012 07:41:26 +0000 (00:41 -0700)
This is clearly a typeo where we are not checking the return value from
get_link_capabilities but should.  This patch corrects that.

Signed-off-by: Don Skidmore <donald.c.skidmore@intel.com>
Tested-by: Stephen Ko <stephen.s.ko@intel.com>
Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
drivers/net/ethernet/intel/ixgbe/ixgbe_82599.c

index 4e59083a3de2404766b1357c7dcb6890d79988dd..e39df2203add7a417eae99a4a2af25bbca20483f 100644 (file)
@@ -779,7 +779,8 @@ static s32 ixgbe_setup_mac_link_82599(struct ixgbe_hw *hw,
        ixgbe_link_speed link_capabilities = IXGBE_LINK_SPEED_UNKNOWN;
 
        /* Check to see if speed passed in is supported. */
-       hw->mac.ops.get_link_capabilities(hw, &link_capabilities, &autoneg);
+       status = hw->mac.ops.get_link_capabilities(hw, &link_capabilities,
+                                                  &autoneg);
        if (status != 0)
                goto out;