net: mdio-uclass: rename arguments of dm_mdio_phy_connect for clarity
authorAlex Marginean <alexandru.marginean@nxp.com>
Mon, 25 Nov 2019 15:15:11 +0000 (17:15 +0200)
committerJoe Hershberger <joe.hershberger@ni.com>
Mon, 9 Dec 2019 15:47:43 +0000 (09:47 -0600)
Renamed dm_mdio_phy_connect arguments dev to mdiodev and addr to phyaddr
for a bit more clarity and consistency with the following patches.
Also use NULL instead of 0 on error return path.

Signed-off-by: Alex Marginean <alexandru.marginean@nxp.com>
Acked-by: Joe Hershberger <joe.hershberger@ni.com>
include/miiphy.h
net/mdio-uclass.c

index 9b97d09f186ba09100e9999a5553439f8fe8035f..94bf0da24a0ff70cf970a6e905708fee4bcfb6bc 100644 (file)
@@ -154,14 +154,14 @@ void dm_mdio_probe_devices(void);
 /**
  * dm_mdio_phy_connect - Wrapper over phy_connect for DM MDIO
  *
- * @dev: mdio dev
- * @addr: PHY address on MDIO bus
+ * @mdiodev: mdio device the PHY is accesible on
+ * @phyaddr: PHY address on MDIO bus
  * @ethdev: ethernet device to connect to the PHY
  * @interface: MAC-PHY protocol
  *
  * @return pointer to phy_device, or 0 on error
  */
-struct phy_device *dm_mdio_phy_connect(struct udevice *dev, int addr,
+struct phy_device *dm_mdio_phy_connect(struct udevice *mdiodev, int phyaddr,
                                       struct udevice *ethdev,
                                       phy_interface_t interface);
 
index 6f922e80b6fb2ff0d0411b6cb9b01961c4d46aca..7a5f1d6dcc83c7f9569a4e0912857014bafbb9e3 100644 (file)
@@ -104,16 +104,16 @@ static int dm_mdio_pre_remove(struct udevice *dev)
        return 0;
 }
 
-struct phy_device *dm_mdio_phy_connect(struct udevice *dev, int addr,
+struct phy_device *dm_mdio_phy_connect(struct udevice *mdiodev, int phyaddr,
                                       struct udevice *ethdev,
                                       phy_interface_t interface)
 {
-       struct mdio_perdev_priv *pdata = dev_get_uclass_priv(dev);
+       struct mdio_perdev_priv *pdata = dev_get_uclass_priv(mdiodev);
 
-       if (device_probe(dev))
-               return 0;
+       if (device_probe(mdiodev))
+               return NULL;
 
-       return phy_connect(pdata->mii_bus, addr, ethdev, interface);
+       return phy_connect(pdata->mii_bus, phyaddr, ethdev, interface);
 }
 
 UCLASS_DRIVER(mdio) = {