From: sudarsana.kalluru@cavium.com Date: Thu, 4 May 2017 15:15:04 +0000 (-0700) Subject: qed: Fix overriding of supported autoneg value. X-Git-Tag: v4.1.12-111.0.20170918_2215~117 X-Git-Url: https://www.infradead.org/git/?a=commitdiff_plain;h=03068153365feaa3d2bf25ee4705a7794423f35f;p=users%2Fjedix%2Flinux-maple.git qed: Fix overriding of supported autoneg value. Orabug: 26783820 Driver currently uses advertised-autoneg value to populate the supported-autoneg field. When advertised field is updated, user gets the same value for supported field. Supported-autoneg value need to be populated from the link capabilities value returned by the MFW. Signed-off-by: Sudarsana Reddy Kalluru Signed-off-by: Yuval Mintz Signed-off-by: David S. Miller [ Upstream commit 34f9199ce7b7e5c641b96e928bd60e086bf7f278 ] Signed-off-by: Somasundaram Krishnasamy --- diff --git a/drivers/net/ethernet/qlogic/qed/qed_dev.c b/drivers/net/ethernet/qlogic/qed/qed_dev.c index 0861b19d5ab6b..de0d67ad54deb 100644 --- a/drivers/net/ethernet/qlogic/qed/qed_dev.c +++ b/drivers/net/ethernet/qlogic/qed/qed_dev.c @@ -2511,6 +2511,9 @@ static int qed_hw_get_nvm_info(struct qed_hwfn *p_hwfn, struct qed_ptt *p_ptt) DP_NOTICE(p_hwfn, "Unknown Speed in 0x%08x\n", link_temp); } + p_hwfn->mcp_info->link_capabilities.default_speed_autoneg = + link->speed.autoneg; + link_temp &= NVM_CFG1_PORT_DRV_FLOW_CONTROL_MASK; link_temp >>= NVM_CFG1_PORT_DRV_FLOW_CONTROL_OFFSET; link->pause.autoneg = !!(link_temp & diff --git a/drivers/net/ethernet/qlogic/qed/qed_main.c b/drivers/net/ethernet/qlogic/qed/qed_main.c index fc299c6d38e3b..d070c1bc8dc66 100644 --- a/drivers/net/ethernet/qlogic/qed/qed_main.c +++ b/drivers/net/ethernet/qlogic/qed/qed_main.c @@ -1370,7 +1370,7 @@ static void qed_fill_link(struct qed_hwfn *hwfn, /* TODO - at the moment assume supported and advertised speed equal */ if_link->supported_caps = SUPPORTED_FIBRE; - if (params.speed.autoneg) + if (link_caps.default_speed_autoneg) if_link->supported_caps |= SUPPORTED_Autoneg; if (params.pause.autoneg || (params.pause.forced_rx && params.pause.forced_tx)) @@ -1380,6 +1380,11 @@ static void qed_fill_link(struct qed_hwfn *hwfn, if_link->supported_caps |= SUPPORTED_Pause; if_link->advertised_caps = if_link->supported_caps; + if (params.speed.autoneg) + if_link->advertised_caps |= SUPPORTED_Autoneg; + else + if_link->advertised_caps &= ~SUPPORTED_Autoneg; + if (params.speed.advertised_speeds & NVM_CFG1_PORT_DRV_SPEED_CAPABILITY_MASK_1G) if_link->advertised_caps |= SUPPORTED_1000baseT_Half | diff --git a/drivers/net/ethernet/qlogic/qed/qed_mcp.h b/drivers/net/ethernet/qlogic/qed/qed_mcp.h index a4c8fa056cbe0..a2ba07792273c 100644 --- a/drivers/net/ethernet/qlogic/qed/qed_mcp.h +++ b/drivers/net/ethernet/qlogic/qed/qed_mcp.h @@ -61,6 +61,7 @@ struct qed_mcp_link_params { struct qed_mcp_link_capabilities { u32 speed_capabilities; + bool default_speed_autoneg; }; struct qed_mcp_link_state {