Commit f17e70d2 authored by Antoine Tenart's avatar Antoine Tenart Committed by David S. Miller

net: mvpp2: only update the XLG configuration when needed

This patch improves the XLG configuration function, to only update the
XLG configuration register when a change is needed. This helps not
writing over and over the same XLG configuration each time phylink
request the MAC to be configured. This mimics the GMAC configuration
function.
Signed-off-by: default avatarAntoine Tenart <antoine.tenart@bootlin.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 6b10bfc5
...@@ -4505,10 +4505,11 @@ static void mvpp2_mac_an_restart(struct net_device *dev) ...@@ -4505,10 +4505,11 @@ static void mvpp2_mac_an_restart(struct net_device *dev)
static void mvpp2_xlg_config(struct mvpp2_port *port, unsigned int mode, static void mvpp2_xlg_config(struct mvpp2_port *port, unsigned int mode,
const struct phylink_link_state *state) const struct phylink_link_state *state)
{ {
u32 ctrl0, ctrl4; u32 old_ctrl0, ctrl0;
u32 old_ctrl4, ctrl4;
ctrl0 = readl(port->base + MVPP22_XLG_CTRL0_REG); old_ctrl0 = ctrl0 = readl(port->base + MVPP22_XLG_CTRL0_REG);
ctrl4 = readl(port->base + MVPP22_XLG_CTRL4_REG); old_ctrl4 = ctrl4 = readl(port->base + MVPP22_XLG_CTRL4_REG);
if (state->pause & MLO_PAUSE_TX) if (state->pause & MLO_PAUSE_TX)
ctrl0 |= MVPP22_XLG_CTRL0_TX_FLOW_CTRL_EN; ctrl0 |= MVPP22_XLG_CTRL0_TX_FLOW_CTRL_EN;
...@@ -4524,7 +4525,9 @@ static void mvpp2_xlg_config(struct mvpp2_port *port, unsigned int mode, ...@@ -4524,7 +4525,9 @@ static void mvpp2_xlg_config(struct mvpp2_port *port, unsigned int mode,
ctrl4 |= MVPP22_XLG_CTRL4_FWD_FC | MVPP22_XLG_CTRL4_FWD_PFC | ctrl4 |= MVPP22_XLG_CTRL4_FWD_FC | MVPP22_XLG_CTRL4_FWD_PFC |
MVPP22_XLG_CTRL4_EN_IDLE_CHECK; MVPP22_XLG_CTRL4_EN_IDLE_CHECK;
if (old_ctrl0 != ctrl0)
writel(ctrl0, port->base + MVPP22_XLG_CTRL0_REG); writel(ctrl0, port->base + MVPP22_XLG_CTRL0_REG);
if (old_ctrl4 != ctrl4)
writel(ctrl4, port->base + MVPP22_XLG_CTRL4_REG); writel(ctrl4, port->base + MVPP22_XLG_CTRL4_REG);
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment