Commit 002dd3de authored by Andrew Lunn's avatar Andrew Lunn Committed by Jakub Kicinski

net: mdio: mdio-bitbang: Separate C22 and C45 transactions

The bitbbanging bus driver can perform both C22 and C45 transfers.
Create separate functions for each and register the C45 versions using
the new driver API calls.

The SH Ethernet driver places wrappers around these functions. In
order to not break boards which might be using C45, add similar
wrappers for C45 operations.
Reviewed-by: default avatarGeert Uytterhoeven <geert+renesas@glider.be>
Signed-off-by: default avatarAndrew Lunn <andrew@lunn.ch>
Signed-off-by: default avatarMichael Walle <michael@walle.cc>
Signed-off-by: default avatarJakub Kicinski <kuba@kernel.org>
parent ce30fa56
...@@ -3044,23 +3044,46 @@ static int sh_mdio_release(struct sh_eth_private *mdp) ...@@ -3044,23 +3044,46 @@ static int sh_mdio_release(struct sh_eth_private *mdp)
return 0; return 0;
} }
static int sh_mdiobb_read(struct mii_bus *bus, int phy, int reg) static int sh_mdiobb_read_c22(struct mii_bus *bus, int phy, int reg)
{ {
int res; int res;
pm_runtime_get_sync(bus->parent); pm_runtime_get_sync(bus->parent);
res = mdiobb_read(bus, phy, reg); res = mdiobb_read_c22(bus, phy, reg);
pm_runtime_put(bus->parent); pm_runtime_put(bus->parent);
return res; return res;
} }
static int sh_mdiobb_write(struct mii_bus *bus, int phy, int reg, u16 val) static int sh_mdiobb_write_c22(struct mii_bus *bus, int phy, int reg, u16 val)
{ {
int res; int res;
pm_runtime_get_sync(bus->parent); pm_runtime_get_sync(bus->parent);
res = mdiobb_write(bus, phy, reg, val); res = mdiobb_write_c22(bus, phy, reg, val);
pm_runtime_put(bus->parent);
return res;
}
static int sh_mdiobb_read_c45(struct mii_bus *bus, int phy, int devad, int reg)
{
int res;
pm_runtime_get_sync(bus->parent);
res = mdiobb_read_c45(bus, phy, devad, reg);
pm_runtime_put(bus->parent);
return res;
}
static int sh_mdiobb_write_c45(struct mii_bus *bus, int phy, int devad,
int reg, u16 val)
{
int res;
pm_runtime_get_sync(bus->parent);
res = mdiobb_write_c45(bus, phy, devad, reg, val);
pm_runtime_put(bus->parent); pm_runtime_put(bus->parent);
return res; return res;
...@@ -3091,8 +3114,10 @@ static int sh_mdio_init(struct sh_eth_private *mdp, ...@@ -3091,8 +3114,10 @@ static int sh_mdio_init(struct sh_eth_private *mdp,
return -ENOMEM; return -ENOMEM;
/* Wrap accessors with Runtime PM-aware ops */ /* Wrap accessors with Runtime PM-aware ops */
mdp->mii_bus->read = sh_mdiobb_read; mdp->mii_bus->read = sh_mdiobb_read_c22;
mdp->mii_bus->write = sh_mdiobb_write; mdp->mii_bus->write = sh_mdiobb_write_c22;
mdp->mii_bus->read_c45 = sh_mdiobb_read_c45;
mdp->mii_bus->write_c45 = sh_mdiobb_write_c45;
/* Hook up MII support for ethtool */ /* Hook up MII support for ethtool */
mdp->mii_bus->name = "sh_mii"; mdp->mii_bus->name = "sh_mii";
......
...@@ -225,7 +225,7 @@ static int davinci_get_mdio_data(struct mdiobb_ctrl *ctrl) ...@@ -225,7 +225,7 @@ static int davinci_get_mdio_data(struct mdiobb_ctrl *ctrl)
return test_bit(MDIO_PIN, &reg); return test_bit(MDIO_PIN, &reg);
} }
static int davinci_mdiobb_read(struct mii_bus *bus, int phy, int reg) static int davinci_mdiobb_read_c22(struct mii_bus *bus, int phy, int reg)
{ {
int ret; int ret;
...@@ -233,7 +233,7 @@ static int davinci_mdiobb_read(struct mii_bus *bus, int phy, int reg) ...@@ -233,7 +233,7 @@ static int davinci_mdiobb_read(struct mii_bus *bus, int phy, int reg)
if (ret < 0) if (ret < 0)
return ret; return ret;
ret = mdiobb_read(bus, phy, reg); ret = mdiobb_read_c22(bus, phy, reg);
pm_runtime_mark_last_busy(bus->parent); pm_runtime_mark_last_busy(bus->parent);
pm_runtime_put_autosuspend(bus->parent); pm_runtime_put_autosuspend(bus->parent);
...@@ -241,8 +241,8 @@ static int davinci_mdiobb_read(struct mii_bus *bus, int phy, int reg) ...@@ -241,8 +241,8 @@ static int davinci_mdiobb_read(struct mii_bus *bus, int phy, int reg)
return ret; return ret;
} }
static int davinci_mdiobb_write(struct mii_bus *bus, int phy, int reg, static int davinci_mdiobb_write_c22(struct mii_bus *bus, int phy, int reg,
u16 val) u16 val)
{ {
int ret; int ret;
...@@ -250,7 +250,41 @@ static int davinci_mdiobb_write(struct mii_bus *bus, int phy, int reg, ...@@ -250,7 +250,41 @@ static int davinci_mdiobb_write(struct mii_bus *bus, int phy, int reg,
if (ret < 0) if (ret < 0)
return ret; return ret;
ret = mdiobb_write(bus, phy, reg, val); ret = mdiobb_write_c22(bus, phy, reg, val);
pm_runtime_mark_last_busy(bus->parent);
pm_runtime_put_autosuspend(bus->parent);
return ret;
}
static int davinci_mdiobb_read_c45(struct mii_bus *bus, int phy, int devad,
int reg)
{
int ret;
ret = pm_runtime_resume_and_get(bus->parent);
if (ret < 0)
return ret;
ret = mdiobb_read_c45(bus, phy, devad, reg);
pm_runtime_mark_last_busy(bus->parent);
pm_runtime_put_autosuspend(bus->parent);
return ret;
}
static int davinci_mdiobb_write_c45(struct mii_bus *bus, int phy, int devad,
int reg, u16 val)
{
int ret;
ret = pm_runtime_resume_and_get(bus->parent);
if (ret < 0)
return ret;
ret = mdiobb_write_c45(bus, phy, devad, reg, val);
pm_runtime_mark_last_busy(bus->parent); pm_runtime_mark_last_busy(bus->parent);
pm_runtime_put_autosuspend(bus->parent); pm_runtime_put_autosuspend(bus->parent);
...@@ -573,8 +607,10 @@ static int davinci_mdio_probe(struct platform_device *pdev) ...@@ -573,8 +607,10 @@ static int davinci_mdio_probe(struct platform_device *pdev)
data->bus->name = dev_name(dev); data->bus->name = dev_name(dev);
if (data->manual_mode) { if (data->manual_mode) {
data->bus->read = davinci_mdiobb_read; data->bus->read = davinci_mdiobb_read_c22;
data->bus->write = davinci_mdiobb_write; data->bus->write = davinci_mdiobb_write_c22;
data->bus->read_c45 = davinci_mdiobb_read_c45;
data->bus->write_c45 = davinci_mdiobb_write_c45;
data->bus->reset = davinci_mdiobb_reset; data->bus->reset = davinci_mdiobb_reset;
dev_info(dev, "Configuring MDIO in manual mode\n"); dev_info(dev, "Configuring MDIO in manual mode\n");
......
...@@ -127,14 +127,12 @@ static void mdiobb_cmd(struct mdiobb_ctrl *ctrl, int op, u8 phy, u8 reg) ...@@ -127,14 +127,12 @@ static void mdiobb_cmd(struct mdiobb_ctrl *ctrl, int op, u8 phy, u8 reg)
/* In clause 45 mode all commands are prefixed by MDIO_ADDR to specify the /* In clause 45 mode all commands are prefixed by MDIO_ADDR to specify the
lower 16 bits of the 21 bit address. This transfer is done identically to a lower 16 bits of the 21 bit address. This transfer is done identically to a
MDIO_WRITE except for a different code. To enable clause 45 mode or MDIO_WRITE except for a different code. Theoretically clause 45 and normal
MII_ADDR_C45 into the address. Theoretically clause 45 and normal devices devices can exist on the same bus. Normal devices should ignore the MDIO_ADDR
can exist on the same bus. Normal devices should ignore the MDIO_ADDR
phase. */ phase. */
static int mdiobb_cmd_addr(struct mdiobb_ctrl *ctrl, int phy, u32 addr) static void mdiobb_cmd_addr(struct mdiobb_ctrl *ctrl, int phy, int dev_addr,
int reg)
{ {
unsigned int dev_addr = (addr >> 16) & 0x1F;
unsigned int reg = addr & 0xFFFF;
mdiobb_cmd(ctrl, MDIO_C45_ADDR, phy, dev_addr); mdiobb_cmd(ctrl, MDIO_C45_ADDR, phy, dev_addr);
/* send the turnaround (10) */ /* send the turnaround (10) */
...@@ -145,21 +143,13 @@ static int mdiobb_cmd_addr(struct mdiobb_ctrl *ctrl, int phy, u32 addr) ...@@ -145,21 +143,13 @@ static int mdiobb_cmd_addr(struct mdiobb_ctrl *ctrl, int phy, u32 addr)
ctrl->ops->set_mdio_dir(ctrl, 0); ctrl->ops->set_mdio_dir(ctrl, 0);
mdiobb_get_bit(ctrl); mdiobb_get_bit(ctrl);
return dev_addr;
} }
int mdiobb_read(struct mii_bus *bus, int phy, int reg) static int mdiobb_read_common(struct mii_bus *bus, int phy)
{ {
struct mdiobb_ctrl *ctrl = bus->priv; struct mdiobb_ctrl *ctrl = bus->priv;
int ret, i; int ret, i;
if (reg & MII_ADDR_C45) {
reg = mdiobb_cmd_addr(ctrl, phy, reg);
mdiobb_cmd(ctrl, MDIO_C45_READ, phy, reg);
} else
mdiobb_cmd(ctrl, ctrl->op_c22_read, phy, reg);
ctrl->ops->set_mdio_dir(ctrl, 0); ctrl->ops->set_mdio_dir(ctrl, 0);
/* check the turnaround bit: the PHY should be driving it to zero, if this /* check the turnaround bit: the PHY should be driving it to zero, if this
...@@ -180,17 +170,31 @@ int mdiobb_read(struct mii_bus *bus, int phy, int reg) ...@@ -180,17 +170,31 @@ int mdiobb_read(struct mii_bus *bus, int phy, int reg)
mdiobb_get_bit(ctrl); mdiobb_get_bit(ctrl);
return ret; return ret;
} }
EXPORT_SYMBOL(mdiobb_read);
int mdiobb_write(struct mii_bus *bus, int phy, int reg, u16 val) int mdiobb_read_c22(struct mii_bus *bus, int phy, int reg)
{ {
struct mdiobb_ctrl *ctrl = bus->priv; struct mdiobb_ctrl *ctrl = bus->priv;
if (reg & MII_ADDR_C45) { mdiobb_cmd(ctrl, ctrl->op_c22_read, phy, reg);
reg = mdiobb_cmd_addr(ctrl, phy, reg);
mdiobb_cmd(ctrl, MDIO_C45_WRITE, phy, reg); return mdiobb_read_common(bus, phy);
} else }
mdiobb_cmd(ctrl, ctrl->op_c22_write, phy, reg); EXPORT_SYMBOL(mdiobb_read_c22);
int mdiobb_read_c45(struct mii_bus *bus, int phy, int devad, int reg)
{
struct mdiobb_ctrl *ctrl = bus->priv;
mdiobb_cmd_addr(ctrl, phy, devad, reg);
mdiobb_cmd(ctrl, MDIO_C45_READ, phy, reg);
return mdiobb_read_common(bus, phy);
}
EXPORT_SYMBOL(mdiobb_read_c45);
static int mdiobb_write_common(struct mii_bus *bus, u16 val)
{
struct mdiobb_ctrl *ctrl = bus->priv;
/* send the turnaround (10) */ /* send the turnaround (10) */
mdiobb_send_bit(ctrl, 1); mdiobb_send_bit(ctrl, 1);
...@@ -202,7 +206,27 @@ int mdiobb_write(struct mii_bus *bus, int phy, int reg, u16 val) ...@@ -202,7 +206,27 @@ int mdiobb_write(struct mii_bus *bus, int phy, int reg, u16 val)
mdiobb_get_bit(ctrl); mdiobb_get_bit(ctrl);
return 0; return 0;
} }
EXPORT_SYMBOL(mdiobb_write);
int mdiobb_write_c22(struct mii_bus *bus, int phy, int reg, u16 val)
{
struct mdiobb_ctrl *ctrl = bus->priv;
mdiobb_cmd(ctrl, ctrl->op_c22_write, phy, reg);
return mdiobb_write_common(bus, val);
}
EXPORT_SYMBOL(mdiobb_write_c22);
int mdiobb_write_c45(struct mii_bus *bus, int phy, int devad, int reg, u16 val)
{
struct mdiobb_ctrl *ctrl = bus->priv;
mdiobb_cmd_addr(ctrl, phy, devad, reg);
mdiobb_cmd(ctrl, MDIO_C45_WRITE, phy, reg);
return mdiobb_write_common(bus, val);
}
EXPORT_SYMBOL(mdiobb_write_c45);
struct mii_bus *alloc_mdio_bitbang(struct mdiobb_ctrl *ctrl) struct mii_bus *alloc_mdio_bitbang(struct mdiobb_ctrl *ctrl)
{ {
...@@ -214,8 +238,11 @@ struct mii_bus *alloc_mdio_bitbang(struct mdiobb_ctrl *ctrl) ...@@ -214,8 +238,11 @@ struct mii_bus *alloc_mdio_bitbang(struct mdiobb_ctrl *ctrl)
__module_get(ctrl->ops->owner); __module_get(ctrl->ops->owner);
bus->read = mdiobb_read; bus->read = mdiobb_read_c22;
bus->write = mdiobb_write; bus->write = mdiobb_write_c22;
bus->read_c45 = mdiobb_read_c45;
bus->write_c45 = mdiobb_write_c45;
bus->priv = ctrl; bus->priv = ctrl;
if (!ctrl->override_op_c22) { if (!ctrl->override_op_c22) {
ctrl->op_c22_read = MDIO_READ; ctrl->op_c22_read = MDIO_READ;
......
...@@ -38,8 +38,10 @@ struct mdiobb_ctrl { ...@@ -38,8 +38,10 @@ struct mdiobb_ctrl {
u8 op_c22_write; u8 op_c22_write;
}; };
int mdiobb_read(struct mii_bus *bus, int phy, int reg); int mdiobb_read_c22(struct mii_bus *bus, int phy, int reg);
int mdiobb_write(struct mii_bus *bus, int phy, int reg, u16 val); int mdiobb_write_c22(struct mii_bus *bus, int phy, int reg, u16 val);
int mdiobb_read_c45(struct mii_bus *bus, int devad, int phy, int reg);
int mdiobb_write_c45(struct mii_bus *bus, int devad, int phy, int reg, u16 val);
/* The returned bus is not yet registered with the phy layer. */ /* The returned bus is not yet registered with the phy layer. */
struct mii_bus *alloc_mdio_bitbang(struct mdiobb_ctrl *ctrl); struct mii_bus *alloc_mdio_bitbang(struct mdiobb_ctrl *ctrl);
......
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