On the PF prefer the link speed value provided by the NSP.
Refresh port table if needed.

Signed-off-by: Jakub Kicinski <jakub.kicin...@netronome.com>
Reviewed-by: Simon Horman <simon.hor...@netronome.com>
---
 drivers/net/ethernet/netronome/nfp/nfp_net_ethtool.c | 11 +++++++++++
 1 file changed, 11 insertions(+)

diff --git a/drivers/net/ethernet/netronome/nfp/nfp_net_ethtool.c 
b/drivers/net/ethernet/netronome/nfp/nfp_net_ethtool.c
index d3cec0d4a978..0fdc14e7b576 100644
--- a/drivers/net/ethernet/netronome/nfp/nfp_net_ethtool.c
+++ b/drivers/net/ethernet/netronome/nfp/nfp_net_ethtool.c
@@ -49,6 +49,7 @@
 #include <linux/ethtool.h>
 
 #include "nfpcore/nfp.h"
+#include "nfpcore/nfp_nsp_eth.h"
 #include "nfp_net_ctrl.h"
 #include "nfp_net.h"
 
@@ -205,6 +206,16 @@ nfp_net_get_link_ksettings(struct net_device *netdev,
        if (!netif_carrier_ok(netdev))
                return 0;
 
+       /* Use link speed from ETH table if available, otherwise try the BAR */
+       if (nn->eth_port && nfp_net_link_changed_read_clear(nn))
+               nfp_net_refresh_port_config(nn);
+       /* Separate if - on FW error the port could've disappeared from table */
+       if (nn->eth_port) {
+               cmd->base.speed = nn->eth_port->speed;
+               cmd->base.duplex = DUPLEX_FULL;
+               return 0;
+       }
+
        sts = nn_readl(nn, NFP_NET_CFG_STS);
 
        ls = FIELD_GET(NFP_NET_CFG_STS_LINK_RATE, sts);
-- 
2.11.0

Reply via email to