diff options
| author | Daniel Golle <daniel@makrotopia.org> | 2026-03-03 06:17:43 +0300 |
|---|---|---|
| committer | Jakub Kicinski <kuba@kernel.org> | 2026-03-05 03:41:19 +0300 |
| commit | aefa52a28a36cf4c9063c095a8191c951e07eb4e (patch) | |
| tree | 00cafe03e920f6c92d062825a0ea44ad220a156d | |
| parent | 8838bb185ef3c801ea4641a95bdace6210cd4b4e (diff) | |
| download | linux-aefa52a28a36cf4c9063c095a8191c951e07eb4e.tar.xz | |
net: dsa: mxl862xx: rename MDIO op arguments
The use of the 'port' argument name for functions implementing the MDIO
bus operations is misleading as the port address isn't equal to the
PHY address.
Rename the MDIO operation argument name to match the prototypes of
mdiobus_write, mdiobus_read, mdiobus_c45_read and mdiobus_c45_write.
Suggested-by: Vladimir Oltean <olteanv@gmail.com>
Signed-off-by: Daniel Golle <daniel@makrotopia.org>
Reviewed-by: Andrew Lunn <andrew@lunn.ch>
Link: https://patch.msgid.link/e1f4cb3bcffc7df9af0f2c9b673b14c7e1201c9a.1772507674.git.daniel@makrotopia.org
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
| -rw-r--r-- | drivers/net/dsa/mxl862xx/mxl862xx.c | 32 |
1 files changed, 16 insertions, 16 deletions
diff --git a/drivers/net/dsa/mxl862xx/mxl862xx.c b/drivers/net/dsa/mxl862xx/mxl862xx.c index b1e2094b5816..8281c749967a 100644 --- a/drivers/net/dsa/mxl862xx/mxl862xx.c +++ b/drivers/net/dsa/mxl862xx/mxl862xx.c @@ -44,13 +44,13 @@ static enum dsa_tag_protocol mxl862xx_get_tag_protocol(struct dsa_switch *ds, } /* PHY access via firmware relay */ -static int mxl862xx_phy_read_mmd(struct mxl862xx_priv *priv, int port, - int devadd, int reg) +static int mxl862xx_phy_read_mmd(struct mxl862xx_priv *priv, int addr, + int devadd, int regnum) { struct mdio_relay_data param = { - .phy = port, + .phy = addr, .mmd = devadd, - .reg = cpu_to_le16(reg), + .reg = cpu_to_le16(regnum), }; int ret; @@ -61,40 +61,40 @@ static int mxl862xx_phy_read_mmd(struct mxl862xx_priv *priv, int port, return le16_to_cpu(param.data); } -static int mxl862xx_phy_write_mmd(struct mxl862xx_priv *priv, int port, - int devadd, int reg, u16 data) +static int mxl862xx_phy_write_mmd(struct mxl862xx_priv *priv, int addr, + int devadd, int regnum, u16 data) { struct mdio_relay_data param = { - .phy = port, + .phy = addr, .mmd = devadd, - .reg = cpu_to_le16(reg), + .reg = cpu_to_le16(regnum), .data = cpu_to_le16(data), }; return MXL862XX_API_WRITE(priv, INT_GPHY_WRITE, param); } -static int mxl862xx_phy_read_mii_bus(struct mii_bus *bus, int port, int regnum) +static int mxl862xx_phy_read_mii_bus(struct mii_bus *bus, int addr, int regnum) { - return mxl862xx_phy_read_mmd(bus->priv, port, 0, regnum); + return mxl862xx_phy_read_mmd(bus->priv, addr, 0, regnum); } -static int mxl862xx_phy_write_mii_bus(struct mii_bus *bus, int port, +static int mxl862xx_phy_write_mii_bus(struct mii_bus *bus, int addr, int regnum, u16 val) { - return mxl862xx_phy_write_mmd(bus->priv, port, 0, regnum, val); + return mxl862xx_phy_write_mmd(bus->priv, addr, 0, regnum, val); } -static int mxl862xx_phy_read_c45_mii_bus(struct mii_bus *bus, int port, +static int mxl862xx_phy_read_c45_mii_bus(struct mii_bus *bus, int addr, int devadd, int regnum) { - return mxl862xx_phy_read_mmd(bus->priv, port, devadd, regnum); + return mxl862xx_phy_read_mmd(bus->priv, addr, devadd, regnum); } -static int mxl862xx_phy_write_c45_mii_bus(struct mii_bus *bus, int port, +static int mxl862xx_phy_write_c45_mii_bus(struct mii_bus *bus, int addr, int devadd, int regnum, u16 val) { - return mxl862xx_phy_write_mmd(bus->priv, port, devadd, regnum, val); + return mxl862xx_phy_write_mmd(bus->priv, addr, devadd, regnum, val); } static int mxl862xx_wait_ready(struct dsa_switch *ds) |