From: Ioana Ciornei <ioana.cior...@nxp.com>

The pause settings are not actually lossy since the MC firmware was does
is to transform back these pause/asym-pause flags into rx/tx pause
enablement status. We are not losing information by this transformation,
thus remove the comment.

Signed-off-by: Ioana Ciornei <ioana.cior...@nxp.com>
---
 drivers/net/ethernet/freescale/dpaa2/dpaa2-mac.c | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/drivers/net/ethernet/freescale/dpaa2/dpaa2-mac.c 
b/drivers/net/ethernet/freescale/dpaa2/dpaa2-mac.c
index 40aa81eb3c4b..f76adbe8c32b 100644
--- a/drivers/net/ethernet/freescale/dpaa2/dpaa2-mac.c
+++ b/drivers/net/ethernet/freescale/dpaa2/dpaa2-mac.c
@@ -181,9 +181,6 @@ static void dpaa2_mac_link_up(struct phylink_config *config,
        else if (duplex == DUPLEX_FULL)
                dpmac_state->options &= ~DPMAC_LINK_OPT_HALF_DUPLEX;
 
-       /* This is lossy; the firmware really should take the pause
-        * enablement status rather than pause/asym pause status.
-        */
        if (rx_pause)
                dpmac_state->options |= DPMAC_LINK_OPT_PAUSE;
        else
-- 
2.29.2

Reply via email to