From: Wenzhuo Lu Date: Wed, 24 Jun 2015 03:26:22 +0000 (+0800) Subject: ixgbe/base: fix X550em flow control to be KR only X-Git-Tag: spdx-start~8940 X-Git-Url: http://git.droids-corp.org/?a=commitdiff_plain;h=54091e55205d74f486fd7d1fbe8d3b9b3373ab8d;p=dpdk.git ixgbe/base: fix X550em flow control to be KR only The code in the if statement is specific to the KR PHY. However, the conditional was only checking for backplane. If the driver was using the KX4 backplane PHY, this code would execute but would write to the wrong PHY. Signed-off-by: Wenzhuo Lu Acked-by: Helin Zhang --- diff --git a/drivers/net/ixgbe/base/ixgbe_x550.c b/drivers/net/ixgbe/base/ixgbe_x550.c index 3b6f517bad..9a3f48875c 100644 --- a/drivers/net/ixgbe/base/ixgbe_x550.c +++ b/drivers/net/ixgbe/base/ixgbe_x550.c @@ -2929,7 +2929,7 @@ s32 ixgbe_setup_fc_X550em(struct ixgbe_hw *hw) goto out; } - if (hw->phy.media_type == ixgbe_media_type_backplane) { + if (hw->device_id == IXGBE_DEV_ID_X550EM_X_KR) { ret_val = ixgbe_read_iosf_sb_reg_x550(hw, IXGBE_KRM_AN_CNTL_1(hw->bus.lan_id), IXGBE_SB_IOSF_TARGET_KR_PHY, ®_val); @@ -2945,9 +2945,8 @@ s32 ixgbe_setup_fc_X550em(struct ixgbe_hw *hw) IXGBE_KRM_AN_CNTL_1(hw->bus.lan_id), IXGBE_SB_IOSF_TARGET_KR_PHY, reg_val); - /* Not all devices fully support AN. */ - if (hw->device_id == IXGBE_DEV_ID_X550EM_X_KR) - hw->fc.disable_fc_autoneg = true; + /* This device does not fully support AN. */ + hw->fc.disable_fc_autoneg = true; } out: