Commit 889b8950 authored by Vladimir Oltean's avatar Vladimir Oltean Committed by David S. Miller

net: mscc: ocelot: separate the common implementation of ndo_open and ndo_stop

Allow these functions to be called from the .port_enable and
.port_disable callbacks of DSA.
Signed-off-by: default avatarVladimir Oltean <vladimir.oltean@nxp.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 31350d7f
...@@ -536,13 +536,9 @@ static void ocelot_port_adjust_link(struct net_device *dev) ...@@ -536,13 +536,9 @@ static void ocelot_port_adjust_link(struct net_device *dev)
ocelot_write(ocelot, ocelot_wm_enc(atop_wm), SYS_ATOP_TOT_CFG); ocelot_write(ocelot, ocelot_wm_enc(atop_wm), SYS_ATOP_TOT_CFG);
} }
static int ocelot_port_open(struct net_device *dev) static void ocelot_port_enable(struct ocelot *ocelot, int port,
struct phy_device *phy)
{ {
struct ocelot_port_private *priv = netdev_priv(dev);
struct ocelot *ocelot = priv->port.ocelot;
int port = priv->chip_port;
int err;
/* Enable receiving frames on the port, and activate auto-learning of /* Enable receiving frames on the port, and activate auto-learning of
* MAC addresses. * MAC addresses.
*/ */
...@@ -550,6 +546,14 @@ static int ocelot_port_open(struct net_device *dev) ...@@ -550,6 +546,14 @@ static int ocelot_port_open(struct net_device *dev)
ANA_PORT_PORT_CFG_RECV_ENA | ANA_PORT_PORT_CFG_RECV_ENA |
ANA_PORT_PORT_CFG_PORTID_VAL(port), ANA_PORT_PORT_CFG_PORTID_VAL(port),
ANA_PORT_PORT_CFG, port); ANA_PORT_PORT_CFG, port);
}
static int ocelot_port_open(struct net_device *dev)
{
struct ocelot_port_private *priv = netdev_priv(dev);
struct ocelot *ocelot = priv->port.ocelot;
int port = priv->chip_port;
int err;
if (priv->serdes) { if (priv->serdes) {
err = phy_set_mode_ext(priv->serdes, PHY_MODE_ETHERNET, err = phy_set_mode_ext(priv->serdes, PHY_MODE_ETHERNET,
...@@ -571,21 +575,33 @@ static int ocelot_port_open(struct net_device *dev) ...@@ -571,21 +575,33 @@ static int ocelot_port_open(struct net_device *dev)
phy_attached_info(priv->phy); phy_attached_info(priv->phy);
phy_start(priv->phy); phy_start(priv->phy);
ocelot_port_enable(ocelot, port, priv->phy);
return 0; return 0;
} }
static void ocelot_port_disable(struct ocelot *ocelot, int port)
{
struct ocelot_port *ocelot_port = ocelot->ports[port];
ocelot_port_writel(ocelot_port, 0, DEV_MAC_ENA_CFG);
ocelot_rmw_rix(ocelot, 0, QSYS_SWITCH_PORT_MODE_PORT_ENA,
QSYS_SWITCH_PORT_MODE, port);
}
static int ocelot_port_stop(struct net_device *dev) static int ocelot_port_stop(struct net_device *dev)
{ {
struct ocelot_port_private *priv = netdev_priv(dev); struct ocelot_port_private *priv = netdev_priv(dev);
struct ocelot_port *port = &priv->port; struct ocelot *ocelot = priv->port.ocelot;
int port = priv->chip_port;
phy_disconnect(priv->phy); phy_disconnect(priv->phy);
dev->phydev = NULL; dev->phydev = NULL;
ocelot_port_writel(port, 0, DEV_MAC_ENA_CFG); ocelot_port_disable(ocelot, port);
ocelot_rmw_rix(port->ocelot, 0, QSYS_SWITCH_PORT_MODE_PORT_ENA,
QSYS_SWITCH_PORT_MODE, priv->chip_port);
return 0; return 0;
} }
......
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