[PATCH v2 06/20] net/txgbe: fix link status check condition

[Zaiyu Wang]

The original code incorrectly used 'if (link_up)' instead of
'if (*link_up)', causing the condition to always evaluate to true
because the pointer itself is non-NULL. This led to incorrect speed
assignment.

Fixes: fb6eb170dfa2 ("net/txgbe: add basic link configuration for Amber-Lite")
Cc: sta...@dpdk.org

Signed-off-by: Zaiyu Wang <zaiyuw...@trustnetic.com>
---
 drivers/net/txgbe/base/txgbe_aml.c   | 2 +-
 drivers/net/txgbe/base/txgbe_aml40.c | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/net/txgbe/base/txgbe_aml.c 
b/drivers/net/txgbe/base/txgbe_aml.c
index b376eca5b5..de9a1b1c93 100644
--- a/drivers/net/txgbe/base/txgbe_aml.c
+++ b/drivers/net/txgbe/base/txgbe_aml.c
@@ -67,7 +67,7 @@ s32 txgbe_check_mac_link_aml(struct txgbe_hw *hw, u32 *speed,
                        *link_up = false;
        }
 
-       if (link_up) {
+       if (*link_up) {
                switch (links_reg & TXGBE_CFG_PORT_ST_AML_LINK_MASK) {
                case TXGBE_CFG_PORT_ST_AML_LINK_25G:
                        *speed = TXGBE_LINK_SPEED_25GB_FULL;
diff --git a/drivers/net/txgbe/base/txgbe_aml40.c 
b/drivers/net/txgbe/base/txgbe_aml40.c
index 733bbac13a..eefd7119fd 100644
--- a/drivers/net/txgbe/base/txgbe_aml40.c
+++ b/drivers/net/txgbe/base/txgbe_aml40.c
@@ -68,7 +68,7 @@ s32 txgbe_check_mac_link_aml40(struct txgbe_hw *hw, u32 
*speed,
                        *link_up = false;
        }
 
-       if (link_up) {
+       if (*link_up) {
                if ((links_reg & TXGBE_CFG_PORT_ST_AML_LINK_40G) ==
                        TXGBE_CFG_PORT_ST_AML_LINK_40G)
                        *speed = TXGBE_LINK_SPEED_40GB_FULL;
-- 
2.21.0.windows.1