powerpc/mpc5200: Refactor FEC mdio read/write routines
Read & write functions now call a generic transfer function, so identical code in both routines could be eliminated. The result is easier to maintain and smaller in source and binary code. Also, fix some checkpatch warnings. Signed-off-by: Wolfram Sang <w.sang@pengutronix.de> Signed-off-by: Grant Likely <grant.likely@secretlab.ca>
This commit is contained in:
parent
9629095761
commit
51d9861e9e
|
@ -2,6 +2,7 @@
|
||||||
* Driver for the MPC5200 Fast Ethernet Controller - MDIO bus driver
|
* Driver for the MPC5200 Fast Ethernet Controller - MDIO bus driver
|
||||||
*
|
*
|
||||||
* Copyright (C) 2007 Domen Puncer, Telargo, Inc.
|
* Copyright (C) 2007 Domen Puncer, Telargo, Inc.
|
||||||
|
* Copyright (C) 2008 Wolfram Sang, Pengutronix
|
||||||
*
|
*
|
||||||
* This file is licensed under the terms of the GNU General Public License
|
* This file is licensed under the terms of the GNU General Public License
|
||||||
* version 2. This program is licensed "as is" without any warranty of any
|
* version 2. This program is licensed "as is" without any warranty of any
|
||||||
|
@ -21,58 +22,45 @@ struct mpc52xx_fec_mdio_priv {
|
||||||
struct mpc52xx_fec __iomem *regs;
|
struct mpc52xx_fec __iomem *regs;
|
||||||
};
|
};
|
||||||
|
|
||||||
static int mpc52xx_fec_mdio_read(struct mii_bus *bus, int phy_id, int reg)
|
static int mpc52xx_fec_mdio_transfer(struct mii_bus *bus, int phy_id,
|
||||||
|
int reg, u32 value)
|
||||||
{
|
{
|
||||||
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 = 100;
|
||||||
u32 request = FEC_MII_READ_FRAME;
|
|
||||||
|
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;
|
||||||
|
|
||||||
fec = priv->regs;
|
fec = priv->regs;
|
||||||
out_be32(&fec->ievent, FEC_IEVENT_MII);
|
out_be32(&fec->ievent, FEC_IEVENT_MII);
|
||||||
|
out_be32(&priv->regs->mii_data, value);
|
||||||
request |= (phy_id << FEC_MII_DATA_PA_SHIFT) & FEC_MII_DATA_PA_MSK;
|
|
||||||
request |= (reg << FEC_MII_DATA_RA_SHIFT) & FEC_MII_DATA_RA_MSK;
|
|
||||||
|
|
||||||
out_be32(&priv->regs->mii_data, request);
|
|
||||||
|
|
||||||
/* 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);
|
udelay(5);
|
||||||
|
|
||||||
if (tries == 0)
|
if (!tries)
|
||||||
return -ETIMEDOUT;
|
return -ETIMEDOUT;
|
||||||
|
|
||||||
return in_be32(&priv->regs->mii_data) & FEC_MII_DATA_DATAMSK;
|
return value & FEC_MII_DATA_OP_RD ?
|
||||||
|
in_be32(&priv->regs->mii_data) & FEC_MII_DATA_DATAMSK : 0;
|
||||||
}
|
}
|
||||||
|
|
||||||
static int mpc52xx_fec_mdio_write(struct mii_bus *bus, int phy_id, int reg, u16 data)
|
static int mpc52xx_fec_mdio_read(struct mii_bus *bus, int phy_id, int reg)
|
||||||
{
|
{
|
||||||
struct mpc52xx_fec_mdio_priv *priv = bus->priv;
|
return mpc52xx_fec_mdio_transfer(bus, phy_id, reg, FEC_MII_READ_FRAME);
|
||||||
struct mpc52xx_fec __iomem *fec;
|
|
||||||
u32 value = data;
|
|
||||||
int tries = 100;
|
|
||||||
|
|
||||||
fec = priv->regs;
|
|
||||||
out_be32(&fec->ievent, FEC_IEVENT_MII);
|
|
||||||
|
|
||||||
value |= FEC_MII_WRITE_FRAME;
|
|
||||||
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;
|
|
||||||
|
|
||||||
out_be32(&priv->regs->mii_data, value);
|
|
||||||
|
|
||||||
/* wait for request to finish */
|
|
||||||
while (!(in_be32(&fec->ievent) & FEC_IEVENT_MII) && --tries)
|
|
||||||
udelay(5);
|
|
||||||
|
|
||||||
if (tries == 0)
|
|
||||||
return -ETIMEDOUT;
|
|
||||||
|
|
||||||
return 0;
|
|
||||||
}
|
}
|
||||||
|
|
||||||
static int mpc52xx_fec_mdio_probe(struct of_device *of, const struct of_device_id *match)
|
static int mpc52xx_fec_mdio_write(struct mii_bus *bus, int phy_id, int reg,
|
||||||
|
u16 data)
|
||||||
|
{
|
||||||
|
return mpc52xx_fec_mdio_transfer(bus, phy_id, reg,
|
||||||
|
data | FEC_MII_WRITE_FRAME);
|
||||||
|
}
|
||||||
|
|
||||||
|
static int mpc52xx_fec_mdio_probe(struct of_device *of,
|
||||||
|
const struct of_device_id *match)
|
||||||
{
|
{
|
||||||
struct device *dev = &of->dev;
|
struct device *dev = &of->dev;
|
||||||
struct device_node *np = of->node;
|
struct device_node *np = of->node;
|
||||||
|
@ -131,7 +119,8 @@ static int mpc52xx_fec_mdio_probe(struct of_device *of, const struct of_device_i
|
||||||
dev_set_drvdata(dev, bus);
|
dev_set_drvdata(dev, bus);
|
||||||
|
|
||||||
/* set MII speed */
|
/* set MII speed */
|
||||||
out_be32(&priv->regs->mii_speed, ((mpc52xx_find_ipb_freq(of->node) >> 20) / 5) << 1);
|
out_be32(&priv->regs->mii_speed,
|
||||||
|
((mpc52xx_find_ipb_freq(of->node) >> 20) / 5) << 1);
|
||||||
|
|
||||||
/* enable MII interrupt */
|
/* enable MII interrupt */
|
||||||
out_be32(&priv->regs->imask, in_be32(&priv->regs->imask) | FEC_IMASK_MII);
|
out_be32(&priv->regs->imask, in_be32(&priv->regs->imask) | FEC_IMASK_MII);
|
||||||
|
|
Loading…
Reference in New Issue