bnxt_en: Fix and clarify link_info->advertising.

Orabug: 25645429

The advertising field is closely related to the auto_link_speeds field.
The former is the user setting while the latter is the firmware setting.
Both should be u16.  We should use the advertising field in
bnxt_get_link_ksettings because the auto_link_speeds field may not
be updated with the latest from the firmware yet.

Signed-off-by: Michael Chan <michael.chan@broadcom.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
(cherry picked from commit 68515a186cf8a8f97956eaea5829277752399f58)
Signed-off-by: Brian Maly <brian.maly@oracle.com>

diff --git a/drivers/net/ethernet/broadcom/bnxt/bnxt.c b/drivers/net/ethernet/broadcom/bnxt/bnxt.c
index 9aec3b3a1b6296181a2a03942fd7528874abe8ef..c3f56b4060aad1eb9d371f89aa3beb7142f4a0f9 100644 (file)
--- a/drivers/net/ethernet/broadcom/bnxt/bnxt.c
+++ b/drivers/net/ethernet/broadcom/bnxt/bnxt.c
@@ -5382,7 +5382,7 @@ static void bnxt_hwrm_set_link_common(struct bnxt *bp,
 {
        u8 autoneg = bp->link_info.autoneg;
        u16 fw_link_speed = bp->link_info.req_link_speed;
-       u32 advertising = bp->link_info.advertising;
+       u16 advertising = bp->link_info.advertising;
 
        if (autoneg & BNXT_AUTONEG_SPEED) {
                req->auto_mode |=

diff --git a/drivers/net/ethernet/broadcom/bnxt/bnxt.h b/drivers/net/ethernet/broadcom/bnxt/bnxt.h
index fddc316170b20ce52735458ae87d0168438543aa..0eb64013fa5e92bee672de3ab24e5d470ade6c3c 100644 (file)
--- a/drivers/net/ethernet/broadcom/bnxt/bnxt.h
+++ b/drivers/net/ethernet/broadcom/bnxt/bnxt.h
@@ -828,7 +828,7 @@ struct bnxt_link_info {
 #define BNXT_LINK_SPEED_40GB   PORT_PHY_QCFG_RESP_LINK_SPEED_40GB
 #define BNXT_LINK_SPEED_50GB   PORT_PHY_QCFG_RESP_LINK_SPEED_50GB
        u16                     support_speeds;
-       u16                     auto_link_speeds;
+       u16                     auto_link_speeds;       /* fw adv setting */
 #define BNXT_LINK_SPEED_MSK_100MB PORT_PHY_QCFG_RESP_SUPPORT_SPEEDS_100MB
 #define BNXT_LINK_SPEED_MSK_1GB PORT_PHY_QCFG_RESP_SUPPORT_SPEEDS_1GB
 #define BNXT_LINK_SPEED_MSK_2GB PORT_PHY_QCFG_RESP_SUPPORT_SPEEDS_2GB
@@ -851,7 +851,7 @@ struct bnxt_link_info {
        u8                      req_duplex;
        u8                      req_flow_ctrl;
        u16                     req_link_speed;
-       u32                     advertising;
+       u16                     advertising;    /* user adv setting */
        bool                    force_link_chng;
 
        /* a copy of phy_qcfg output used to report link

diff --git a/drivers/net/ethernet/broadcom/bnxt/bnxt_ethtool.c b/drivers/net/ethernet/broadcom/bnxt/bnxt_ethtool.c
index 14f6d5cbb643194cd2c9b726b6de5ae2b7e7d828..1f7026c8a44b23b446072d95d67ccf24e585e939 100644 (file)
--- a/drivers/net/ethernet/broadcom/bnxt/bnxt_ethtool.c
+++ b/drivers/net/ethernet/broadcom/bnxt/bnxt_ethtool.c
@@ -921,7 +921,7 @@ u32 _bnxt_fw_to_ethtool_adv_spds(u16 fw_speeds, u8 fw_pause)
 static void bnxt_fw_to_ethtool_advertised_spds(struct bnxt_link_info *link_info,
                                struct ethtool_link_ksettings *lk_ksettings)
 {
-       u16 fw_speeds = link_info->auto_link_speeds;
+       u16 fw_speeds = link_info->advertising;
        u8 fw_pause = 0;
 
        if (link_info->autoneg & BNXT_AUTONEG_FLOW_CTRL)
@@ -1231,8 +1231,9 @@ static int bnxt_set_link_ksettings(struct net_device *dev,
        struct bnxt *bp = netdev_priv(dev);
        struct bnxt_link_info *link_info = &bp->link_info;
        const struct ethtool_link_settings *base = &lk_ksettings->base;
-       u32 speed, fw_advertising = 0;
        bool set_pause = false;
+       u16 fw_advertising = 0;
+       u32 speed;
        int rc = 0;
 
        if (!BNXT_SINGLE_PF(bp))