From: Qin Ke <qin...@corigine.com>
The link status of representor port is reported by the flower
firmware through control message and it already parsed and
stored in the 'link' field of representor port structure.
The original logic read link status from the control BAR again,
and use it rather then the 'link' field of the representor port
structure in the following logic wrongly.
Fix this by delete the read control BAR statement and use the
right link status value.
Fixes: c4de52eca76c ("net/nfp: remove redundancy for representor port")
Cc: chaoyong...@corigine.com
Cc: sta...@dpdk.org
Signed-off-by: Qin Ke <qin...@corigine.com>
Reviewed-by: Chaoyong He <chaoyong...@corigine.com>
Reviewed-by: Long Wu <long...@corigine.com>
Reviewed-by: Peng Zhang <peng.zh...@corigine.com>
---
drivers/net/nfp/flower/nfp_flower_representor.c | 7 +------
1 file changed, 1 insertion(+), 6 deletions(-)
diff --git a/drivers/net/nfp/flower/nfp_flower_representor.c
b/drivers/net/nfp/flower/nfp_flower_representor.c
index 054ea1a938..5db7d50618 100644
--- a/drivers/net/nfp/flower/nfp_flower_representor.c
+++ b/drivers/net/nfp/flower/nfp_flower_representor.c
@@ -29,18 +29,13 @@ nfp_flower_repr_link_update(struct rte_eth_dev *dev,
__rte_unused int wait_to_complete)
{
int ret;
- uint32_t nn_link_status;
- struct nfp_net_hw *pf_hw;
struct rte_eth_link *link;
struct nfp_flower_representor *repr;
repr = dev->data->dev_private;
link = &repr->link;
- pf_hw = repr->app_fw_flower->pf_hw;
- nn_link_status = nn_cfg_readw(&pf_hw->super, NFP_NET_CFG_STS);
-
- ret = nfp_net_link_update_common(dev, link, nn_link_status);
+ ret = nfp_net_link_update_common(dev, link, link->link_status);
return ret;
}
--
2.39.1
