[net] net: stmmac: selftests: Flow Control test can also run with ASYM Pause

Commit Message

Jose Abreu Sept. 23, 2019, 7:30 a.m. UTC

From: Jose Abreu <[e-mail]>

The Flow Control selftest is also available with ASYM Pause. Lets add
this check to the test and fix eventual false positive failures.

Fixes: 091810dbded9 ("net: stmmac: Introduce selftests support")
Signed-off-by: Jose Abreu <[e-mail]>

---
Cc: Giuseppe Cavallaro <[e-mail]>
Cc: Alexandre Torgue <[e-mail]>
Cc: Jose Abreu <[e-mail]>
Cc: "David S. Miller" <[e-mail]>
Cc: Maxime Coquelin <[e-mail]>
Cc: [e-mail]
Cc: [e-mail]
Cc: [e-mail]
Cc: [e-mail]
---
 drivers/net/ethernet/stmicro/stmmac/stmmac_selftests.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Jose Abreu Sept. 23, 2019, 7:47 a.m. UTC | #1

From: Jose Abreu <[e-mail]>
Date: Sep/23/2019, 08:30:43 (UTC+00:00)

> From: Jose Abreu <[e-mail]>
> 
> The Flow Control selftest is also available with ASYM Pause. Lets add
> this check to the test and fix eventual false positive failures.
> 
> Fixes: 091810dbded9 ("net: stmmac: Introduce selftests support")
> Signed-off-by: Jose Abreu <[e-mail]>

Oops, "From:" and "Signed-off-by:" are still different. I'll amend this 
and re-submit. Please disregard this one.

---
Thanks,
Jose Miguel Abreu

Patch

diff --git a/drivers/net/ethernet/stmicro/stmmac/stmmac_selftests.c b/drivers/net/ethernet/stmicro/stmmac/stmmac_selftests.c
index 9c8d210b2d6a..5f66f6161629 100644
--- a/drivers/net/ethernet/stmicro/stmmac/stmmac_selftests.c
+++ b/drivers/net/ethernet/stmicro/stmmac/stmmac_selftests.c
@@ -670,7 +670,7 @@  static int stmmac_test_flowctrl(struct stmmac_priv *priv)
 	unsigned int pkt_count;
 	int i, ret = 0;
 
-	if (!phydev || !phydev->pause)
+	if (!phydev || (!phydev->pause && !phydev->asym_pause))
 		return -EOPNOTSUPP;
 
 	tpriv = kzalloc(sizeof(*tpriv), GFP_KERNEL);