net: dsa: bcm_sf2: force pause link settings

commit 3bc9f64a90276413257f6b79e6c9a03e83313780 upstream.

The pause settings reported by the PHY should also be applied to the GMII port
status override otherwise the switch will not generate pause frames towards the
link partner despite the advertisement saying otherwise.

Fixes: fccc2cd9438e ("net: dsa: add Broadcom SF2 switch driver")
Signed-off-by: Doug Berger <opendmb@gmail.com>
Signed-off-by: Florian Fainelli <f.fainelli@gmail.com>
Link: https://lore.kernel.org/r/20220623030204.1966851-1-f.fainelli@gmail.com
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

diff --git a/drivers/net/dsa/bcm_sf2.c b/drivers/net/dsa/bcm_sf2.c
index 0ee1c0a7b165b4ee6917dc71db9ff680fa894f5a..282e9e2a50d9cdf17680f5c2652504b5d2772048 100644 (file)
--- a/drivers/net/dsa/bcm_sf2.c
+++ b/drivers/net/dsa/bcm_sf2.c
@@ -602,6 +602,11 @@ force_link:
                reg |= LINK_STS;
        if (state->duplex == DUPLEX_FULL)
                reg |= DUPLX_MODE;
+       if (state->pause & MLO_PAUSE_TXRX_MASK) {
+               if (state->pause & MLO_PAUSE_TX)
+                       reg |= TXFLOW_CNTL;
+               reg |= RXFLOW_CNTL;
+       }
 
        core_writel(priv, reg, offset);
 }