Commit ca816d98 authored by Grant Likely's avatar Grant Likely Committed by David S. Miller

net: Rework mpc5200 fec driver to use of_mdio infrastructure.

The patch reworks the MPC5200 Fast Ethernet Controller (FEC) driver to
use the of_mdio infrastructure for registering PHY devices from data out
openfirmware device tree, and eliminates the assumption that the PHY
for the FEC is always attached to the FEC's own MDIO bus.  With this
patch, the FEC can use a PHY attached to any MDIO bus if it is described
in the device tree.

Tested on Freescale Lite5200b eval board
Signed-off-by: default avatarGrant Likely <grant.likely@secretlab.ca>
Acked-by: default avatarAndy Fleming <afleming@freescale.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 8bc487d1
...@@ -25,6 +25,7 @@ ...@@ -25,6 +25,7 @@
#include <linux/hardirq.h> #include <linux/hardirq.h>
#include <linux/delay.h> #include <linux/delay.h>
#include <linux/of_device.h> #include <linux/of_device.h>
#include <linux/of_mdio.h>
#include <linux/of_platform.h> #include <linux/of_platform.h>
#include <linux/netdevice.h> #include <linux/netdevice.h>
...@@ -43,11 +44,9 @@ ...@@ -43,11 +44,9 @@
#define DRIVER_NAME "mpc52xx-fec" #define DRIVER_NAME "mpc52xx-fec"
#define FEC5200_PHYADDR_NONE (-1)
#define FEC5200_PHYADDR_7WIRE (-2)
/* Private driver data structure */ /* Private driver data structure */
struct mpc52xx_fec_priv { struct mpc52xx_fec_priv {
struct net_device *ndev;
int duplex; int duplex;
int speed; int speed;
int r_irq; int r_irq;
...@@ -59,10 +58,11 @@ struct mpc52xx_fec_priv { ...@@ -59,10 +58,11 @@ struct mpc52xx_fec_priv {
int msg_enable; int msg_enable;
/* MDIO link details */ /* MDIO link details */
int phy_addr; unsigned int mdio_speed;
unsigned int phy_speed; struct device_node *phy_node;
struct phy_device *phydev; struct phy_device *phydev;
enum phy_state link; enum phy_state link;
int seven_wire_mode;
}; };
...@@ -211,85 +211,25 @@ static void mpc52xx_fec_adjust_link(struct net_device *dev) ...@@ -211,85 +211,25 @@ static void mpc52xx_fec_adjust_link(struct net_device *dev)
phy_print_status(phydev); phy_print_status(phydev);
} }
static int mpc52xx_fec_init_phy(struct net_device *dev) static int mpc52xx_fec_open(struct net_device *dev)
{
struct mpc52xx_fec_priv *priv = netdev_priv(dev);
struct phy_device *phydev;
char phy_id[BUS_ID_SIZE];
snprintf(phy_id, sizeof(phy_id), "%x:%02x",
(unsigned int)dev->base_addr, priv->phy_addr);
priv->link = PHY_DOWN;
priv->speed = 0;
priv->duplex = -1;
phydev = phy_connect(dev, phy_id, &mpc52xx_fec_adjust_link, 0, PHY_INTERFACE_MODE_MII);
if (IS_ERR(phydev)) {
dev_err(&dev->dev, "phy_connect failed\n");
return PTR_ERR(phydev);
}
dev_info(&dev->dev, "attached phy %i to driver %s\n",
phydev->addr, phydev->drv->name);
priv->phydev = phydev;
return 0;
}
static int mpc52xx_fec_phy_start(struct net_device *dev)
{ {
struct mpc52xx_fec_priv *priv = netdev_priv(dev); struct mpc52xx_fec_priv *priv = netdev_priv(dev);
int err; int err = -EBUSY;
if (priv->phy_addr < 0)
return 0;
err = mpc52xx_fec_init_phy(dev); if (priv->phy_node) {
if (err) { priv->phydev = of_phy_connect(priv->ndev, priv->phy_node,
dev_err(&dev->dev, "mpc52xx_fec_init_phy failed\n"); mpc52xx_fec_adjust_link, 0, 0);
return err; if (!priv->phydev) {
dev_err(&dev->dev, "of_phy_connect failed\n");
return -ENODEV;
} }
/* reset phy - this also wakes it from PDOWN */
phy_write(priv->phydev, MII_BMCR, BMCR_RESET);
phy_start(priv->phydev); phy_start(priv->phydev);
}
return 0;
}
static void mpc52xx_fec_phy_stop(struct net_device *dev)
{
struct mpc52xx_fec_priv *priv = netdev_priv(dev);
if (!priv->phydev)
return;
phy_disconnect(priv->phydev);
/* power down phy */
phy_stop(priv->phydev);
phy_write(priv->phydev, MII_BMCR, BMCR_PDOWN);
}
static void mpc52xx_fec_phy_hw_init(struct mpc52xx_fec_priv *priv)
{
struct mpc52xx_fec __iomem *fec = priv->fec;
if (priv->phydev)
return;
out_be32(&fec->mii_speed, priv->phy_speed);
}
static int mpc52xx_fec_open(struct net_device *dev)
{
struct mpc52xx_fec_priv *priv = netdev_priv(dev);
int err = -EBUSY;
if (request_irq(dev->irq, &mpc52xx_fec_interrupt, IRQF_SHARED, if (request_irq(dev->irq, &mpc52xx_fec_interrupt, IRQF_SHARED,
DRIVER_NAME "_ctrl", dev)) { DRIVER_NAME "_ctrl", dev)) {
dev_err(&dev->dev, "ctrl interrupt request failed\n"); dev_err(&dev->dev, "ctrl interrupt request failed\n");
goto out; goto free_phy;
} }
if (request_irq(priv->r_irq, &mpc52xx_fec_rx_interrupt, 0, if (request_irq(priv->r_irq, &mpc52xx_fec_rx_interrupt, 0,
DRIVER_NAME "_rx", dev)) { DRIVER_NAME "_rx", dev)) {
...@@ -311,10 +251,6 @@ static int mpc52xx_fec_open(struct net_device *dev) ...@@ -311,10 +251,6 @@ static int mpc52xx_fec_open(struct net_device *dev)
goto free_irqs; goto free_irqs;
} }
err = mpc52xx_fec_phy_start(dev);
if (err)
goto free_skbs;
bcom_enable(priv->rx_dmatsk); bcom_enable(priv->rx_dmatsk);
bcom_enable(priv->tx_dmatsk); bcom_enable(priv->tx_dmatsk);
...@@ -324,16 +260,18 @@ static int mpc52xx_fec_open(struct net_device *dev) ...@@ -324,16 +260,18 @@ static int mpc52xx_fec_open(struct net_device *dev)
return 0; return 0;
free_skbs:
mpc52xx_fec_free_rx_buffers(dev, priv->rx_dmatsk);
free_irqs: free_irqs:
free_irq(priv->t_irq, dev); free_irq(priv->t_irq, dev);
free_2irqs: free_2irqs:
free_irq(priv->r_irq, dev); free_irq(priv->r_irq, dev);
free_ctrl_irq: free_ctrl_irq:
free_irq(dev->irq, dev); free_irq(dev->irq, dev);
out: free_phy:
if (priv->phydev) {
phy_stop(priv->phydev);
phy_disconnect(priv->phydev);
priv->phydev = NULL;
}
return err; return err;
} }
...@@ -352,7 +290,12 @@ static int mpc52xx_fec_close(struct net_device *dev) ...@@ -352,7 +290,12 @@ static int mpc52xx_fec_close(struct net_device *dev)
free_irq(priv->r_irq, dev); free_irq(priv->r_irq, dev);
free_irq(priv->t_irq, dev); free_irq(priv->t_irq, dev);
mpc52xx_fec_phy_stop(dev); if (priv->phydev) {
/* power down phy */
phy_stop(priv->phydev);
phy_disconnect(priv->phydev);
priv->phydev = NULL;
}
return 0; return 0;
} }
...@@ -696,7 +639,7 @@ static void mpc52xx_fec_hw_init(struct net_device *dev) ...@@ -696,7 +639,7 @@ static void mpc52xx_fec_hw_init(struct net_device *dev)
/* set phy speed. /* set phy speed.
* this can't be done in phy driver, since it needs to be called * this can't be done in phy driver, since it needs to be called
* before fec stuff (even on resume) */ * before fec stuff (even on resume) */
mpc52xx_fec_phy_hw_init(priv); out_be32(&fec->mii_speed, priv->mdio_speed);
} }
/** /**
...@@ -732,7 +675,7 @@ static void mpc52xx_fec_start(struct net_device *dev) ...@@ -732,7 +675,7 @@ static void mpc52xx_fec_start(struct net_device *dev)
rcntrl = FEC_RX_BUFFER_SIZE << 16; /* max frame length */ rcntrl = FEC_RX_BUFFER_SIZE << 16; /* max frame length */
rcntrl |= FEC_RCNTRL_FCE; rcntrl |= FEC_RCNTRL_FCE;
if (priv->phy_addr != FEC5200_PHYADDR_7WIRE) if (!priv->seven_wire_mode)
rcntrl |= FEC_RCNTRL_MII_MODE; rcntrl |= FEC_RCNTRL_MII_MODE;
if (priv->duplex == DUPLEX_FULL) if (priv->duplex == DUPLEX_FULL)
...@@ -798,8 +741,6 @@ static void mpc52xx_fec_stop(struct net_device *dev) ...@@ -798,8 +741,6 @@ static void mpc52xx_fec_stop(struct net_device *dev)
/* Stop FEC */ /* Stop FEC */
out_be32(&fec->ecntrl, in_be32(&fec->ecntrl) & ~FEC_ECNTRL_ETHER_EN); out_be32(&fec->ecntrl, in_be32(&fec->ecntrl) & ~FEC_ECNTRL_ETHER_EN);
return;
} }
/* reset fec and bestcomm tasks */ /* reset fec and bestcomm tasks */
...@@ -817,9 +758,11 @@ static void mpc52xx_fec_reset(struct net_device *dev) ...@@ -817,9 +758,11 @@ static void mpc52xx_fec_reset(struct net_device *dev)
mpc52xx_fec_hw_init(dev); mpc52xx_fec_hw_init(dev);
if (priv->phydev) {
phy_stop(priv->phydev); phy_stop(priv->phydev);
phy_write(priv->phydev, MII_BMCR, BMCR_RESET); phy_write(priv->phydev, MII_BMCR, BMCR_RESET);
phy_start(priv->phydev); phy_start(priv->phydev);
}
bcom_fec_rx_reset(priv->rx_dmatsk); bcom_fec_rx_reset(priv->rx_dmatsk);
bcom_fec_tx_reset(priv->tx_dmatsk); bcom_fec_tx_reset(priv->tx_dmatsk);
...@@ -919,8 +862,6 @@ mpc52xx_fec_probe(struct of_device *op, const struct of_device_id *match) ...@@ -919,8 +862,6 @@ mpc52xx_fec_probe(struct of_device *op, const struct of_device_id *match)
struct net_device *ndev; struct net_device *ndev;
struct mpc52xx_fec_priv *priv = NULL; struct mpc52xx_fec_priv *priv = NULL;
struct resource mem; struct resource mem;
struct device_node *phy_node;
const phandle *phy_handle;
const u32 *prop; const u32 *prop;
int prop_size; int prop_size;
...@@ -933,6 +874,7 @@ mpc52xx_fec_probe(struct of_device *op, const struct of_device_id *match) ...@@ -933,6 +874,7 @@ mpc52xx_fec_probe(struct of_device *op, const struct of_device_id *match)
return -ENOMEM; return -ENOMEM;
priv = netdev_priv(ndev); priv = netdev_priv(ndev);
priv->ndev = ndev;
/* Reserve FEC control zone */ /* Reserve FEC control zone */
rv = of_address_to_resource(op->node, 0, &mem); rv = of_address_to_resource(op->node, 0, &mem);
...@@ -956,6 +898,7 @@ mpc52xx_fec_probe(struct of_device *op, const struct of_device_id *match) ...@@ -956,6 +898,7 @@ mpc52xx_fec_probe(struct of_device *op, const struct of_device_id *match)
ndev->ethtool_ops = &mpc52xx_fec_ethtool_ops; ndev->ethtool_ops = &mpc52xx_fec_ethtool_ops;
ndev->watchdog_timeo = FEC_WATCHDOG_TIMEOUT; ndev->watchdog_timeo = FEC_WATCHDOG_TIMEOUT;
ndev->base_addr = mem.start; ndev->base_addr = mem.start;
SET_NETDEV_DEV(ndev, &op->dev);
spin_lock_init(&priv->lock); spin_lock_init(&priv->lock);
...@@ -1003,14 +946,9 @@ mpc52xx_fec_probe(struct of_device *op, const struct of_device_id *match) ...@@ -1003,14 +946,9 @@ mpc52xx_fec_probe(struct of_device *op, const struct of_device_id *match)
*/ */
/* Start with safe defaults for link connection */ /* Start with safe defaults for link connection */
priv->phy_addr = FEC5200_PHYADDR_NONE;
priv->speed = 100; priv->speed = 100;
priv->duplex = DUPLEX_HALF; priv->duplex = DUPLEX_HALF;
priv->phy_speed = ((mpc52xx_find_ipb_freq(op->node) >> 20) / 5) << 1; priv->mdio_speed = ((mpc52xx_find_ipb_freq(op->node) >> 20) / 5) << 1;
/* the 7-wire property means don't use MII mode */
if (of_find_property(op->node, "fsl,7-wire-mode", NULL))
priv->phy_addr = FEC5200_PHYADDR_7WIRE;
/* The current speed preconfigures the speed of the MII link */ /* The current speed preconfigures the speed of the MII link */
prop = of_get_property(op->node, "current-speed", &prop_size); prop = of_get_property(op->node, "current-speed", &prop_size);
...@@ -1019,43 +957,23 @@ mpc52xx_fec_probe(struct of_device *op, const struct of_device_id *match) ...@@ -1019,43 +957,23 @@ mpc52xx_fec_probe(struct of_device *op, const struct of_device_id *match)
priv->duplex = prop[1] ? DUPLEX_FULL : DUPLEX_HALF; priv->duplex = prop[1] ? DUPLEX_FULL : DUPLEX_HALF;
} }
/* If there is a phy handle, setup link to that phy */ /* If there is a phy handle, then get the PHY node */
phy_handle = of_get_property(op->node, "phy-handle", &prop_size); priv->phy_node = of_parse_phandle(op->node, "phy-handle", 0);
if (phy_handle && (prop_size >= sizeof(phandle))) {
phy_node = of_find_node_by_phandle(*phy_handle); /* the 7-wire property means don't use MII mode */
prop = of_get_property(phy_node, "reg", &prop_size); if (of_find_property(op->node, "fsl,7-wire-mode", NULL)) {
if (prop && (prop_size >= sizeof(u32))) priv->seven_wire_mode = 1;
if ((*prop >= 0) && (*prop < PHY_MAX_ADDR)) dev_info(&ndev->dev, "using 7-wire PHY mode\n");
priv->phy_addr = *prop;
of_node_put(phy_node);
} }
/* Hardware init */ /* Hardware init */
mpc52xx_fec_hw_init(ndev); mpc52xx_fec_hw_init(ndev);
mpc52xx_fec_reset_stats(ndev); mpc52xx_fec_reset_stats(ndev);
SET_NETDEV_DEV(ndev, &op->dev);
/* Register the new network device */
rv = register_netdev(ndev); rv = register_netdev(ndev);
if (rv < 0) if (rv < 0)
goto probe_error; goto probe_error;
/* Now report the link setup */
switch (priv->phy_addr) {
case FEC5200_PHYADDR_NONE:
dev_info(&ndev->dev, "Fixed speed MII link: %i%cD\n",
priv->speed, priv->duplex ? 'F' : 'H');
break;
case FEC5200_PHYADDR_7WIRE:
dev_info(&ndev->dev, "using 7-wire PHY mode\n");
break;
default:
dev_info(&ndev->dev, "Using PHY at MDIO address %i\n",
priv->phy_addr);
}
/* We're done ! */ /* We're done ! */
dev_set_drvdata(&op->dev, ndev); dev_set_drvdata(&op->dev, ndev);
...@@ -1065,6 +983,10 @@ mpc52xx_fec_probe(struct of_device *op, const struct of_device_id *match) ...@@ -1065,6 +983,10 @@ mpc52xx_fec_probe(struct of_device *op, const struct of_device_id *match)
/* Error handling - free everything that might be allocated */ /* Error handling - free everything that might be allocated */
probe_error: probe_error:
if (priv->phy_node)
of_node_put(priv->phy_node);
priv->phy_node = NULL;
irq_dispose_mapping(ndev->irq); irq_dispose_mapping(ndev->irq);
if (priv->rx_dmatsk) if (priv->rx_dmatsk)
...@@ -1093,6 +1015,10 @@ mpc52xx_fec_remove(struct of_device *op) ...@@ -1093,6 +1015,10 @@ mpc52xx_fec_remove(struct of_device *op)
unregister_netdev(ndev); unregister_netdev(ndev);
if (priv->phy_node)
of_node_put(priv->phy_node);
priv->phy_node = NULL;
irq_dispose_mapping(ndev->irq); irq_dispose_mapping(ndev->irq);
bcom_fec_rx_release(priv->rx_dmatsk); bcom_fec_rx_release(priv->rx_dmatsk);
......
...@@ -14,12 +14,14 @@ ...@@ -14,12 +14,14 @@
#include <linux/netdevice.h> #include <linux/netdevice.h>
#include <linux/phy.h> #include <linux/phy.h>
#include <linux/of_platform.h> #include <linux/of_platform.h>
#include <linux/of_mdio.h>
#include <asm/io.h> #include <asm/io.h>
#include <asm/mpc52xx.h> #include <asm/mpc52xx.h>
#include "fec_mpc52xx.h" #include "fec_mpc52xx.h"
struct mpc52xx_fec_mdio_priv { struct mpc52xx_fec_mdio_priv {
struct mpc52xx_fec __iomem *regs; struct mpc52xx_fec __iomem *regs;
int mdio_irqs[PHY_MAX_ADDR];
}; };
static int mpc52xx_fec_mdio_transfer(struct mii_bus *bus, int phy_id, static int mpc52xx_fec_mdio_transfer(struct mii_bus *bus, int phy_id,
...@@ -27,7 +29,7 @@ static int mpc52xx_fec_mdio_transfer(struct mii_bus *bus, int phy_id, ...@@ -27,7 +29,7 @@ static int mpc52xx_fec_mdio_transfer(struct mii_bus *bus, int phy_id,
{ {
struct mpc52xx_fec_mdio_priv *priv = bus->priv; struct mpc52xx_fec_mdio_priv *priv = bus->priv;
struct mpc52xx_fec __iomem *fec; struct mpc52xx_fec __iomem *fec;
int tries = 100; int tries = 3;
value |= (phy_id << FEC_MII_DATA_PA_SHIFT) & FEC_MII_DATA_PA_MSK; value |= (phy_id << FEC_MII_DATA_PA_SHIFT) & FEC_MII_DATA_PA_MSK;
value |= (reg << FEC_MII_DATA_RA_SHIFT) & FEC_MII_DATA_RA_MSK; value |= (reg << FEC_MII_DATA_RA_SHIFT) & FEC_MII_DATA_RA_MSK;
...@@ -38,7 +40,7 @@ static int mpc52xx_fec_mdio_transfer(struct mii_bus *bus, int phy_id, ...@@ -38,7 +40,7 @@ static int mpc52xx_fec_mdio_transfer(struct mii_bus *bus, int phy_id,
/* wait for it to finish, this takes about 23 us on lite5200b */ /* wait for it to finish, this takes about 23 us on lite5200b */
while (!(in_be32(&fec->ievent) & FEC_IEVENT_MII) && --tries) while (!(in_be32(&fec->ievent) & FEC_IEVENT_MII) && --tries)
udelay(5); msleep(1);
if (!tries) if (!tries)
return -ETIMEDOUT; return -ETIMEDOUT;
...@@ -64,7 +66,6 @@ static int mpc52xx_fec_mdio_probe(struct of_device *of, ...@@ -64,7 +66,6 @@ static int mpc52xx_fec_mdio_probe(struct of_device *of,
{ {
struct device *dev = &of->dev; struct device *dev = &of->dev;
struct device_node *np = of->node; struct device_node *np = of->node;
struct device_node *child = NULL;
struct mii_bus *bus; struct mii_bus *bus;
struct mpc52xx_fec_mdio_priv *priv; struct mpc52xx_fec_mdio_priv *priv;
struct resource res = {}; struct resource res = {};
...@@ -85,22 +86,7 @@ static int mpc52xx_fec_mdio_probe(struct of_device *of, ...@@ -85,22 +86,7 @@ static int mpc52xx_fec_mdio_probe(struct of_device *of,
bus->write = mpc52xx_fec_mdio_write; bus->write = mpc52xx_fec_mdio_write;
/* setup irqs */ /* setup irqs */
bus->irq = kmalloc(sizeof(bus->irq[0]) * PHY_MAX_ADDR, GFP_KERNEL); bus->irq = priv->mdio_irqs;
if (bus->irq == NULL) {
err = -ENOMEM;
goto out_free;
}
for (i=0; i<PHY_MAX_ADDR; i++)
bus->irq[i] = PHY_POLL;
while ((child = of_get_next_child(np, child)) != NULL) {
int irq = irq_of_parse_and_map(child, 0);
if (irq != NO_IRQ) {
const u32 *id = of_get_property(child, "reg", NULL);
if (id)
bus->irq[*id] = irq;
}
}
/* setup registers */ /* setup registers */
err = of_address_to_resource(np, 0, &res); err = of_address_to_resource(np, 0, &res);
...@@ -122,7 +108,7 @@ static int mpc52xx_fec_mdio_probe(struct of_device *of, ...@@ -122,7 +108,7 @@ static int mpc52xx_fec_mdio_probe(struct of_device *of,
out_be32(&priv->regs->mii_speed, out_be32(&priv->regs->mii_speed,
((mpc52xx_find_ipb_freq(of->node) >> 20) / 5) << 1); ((mpc52xx_find_ipb_freq(of->node) >> 20) / 5) << 1);
err = mdiobus_register(bus); err = of_mdiobus_register(bus, np);
if (err) if (err)
goto out_unmap; goto out_unmap;
......
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