of: add of_mdio_find_device() api
authorRussell King <rmk+kernel@armlinux.org.uk>
Wed, 23 Sep 2020 15:41:22 +0000 (18:41 +0300)
committerDavid S. Miller <davem@davemloft.net>
Fri, 25 Sep 2020 02:49:36 +0000 (19:49 -0700)
Add a helper function which finds the mdio_device structure given a
device tree node. This is helpful for finding the PCS device based on a
DTS node but managing it as a mdio_device instead of a phy_device.

Signed-off-by: Russell King <rmk+kernel@armlinux.org.uk>
Signed-off-by: Ioana Ciornei <ioana.ciornei@nxp.com>
Reviewed-by: Andrew Lunn <andrew@lunn.ch>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/of/of_mdio.c
include/linux/of_mdio.h

index cb32d7ef49382f05d3e8065f9c5f316c8735ad3c..4daf94bb56a53d222ecadbd9e2c7028d7073b4a5 100644 (file)
@@ -337,6 +337,29 @@ unregister:
 }
 EXPORT_SYMBOL(of_mdiobus_register);
 
+/**
+ * of_mdio_find_device - Given a device tree node, find the mdio_device
+ * @np: pointer to the mdio_device's device tree node
+ *
+ * If successful, returns a pointer to the mdio_device with the embedded
+ * struct device refcount incremented by one, or NULL on failure.
+ * The caller should call put_device() on the mdio_device after its use
+ */
+struct mdio_device *of_mdio_find_device(struct device_node *np)
+{
+       struct device *d;
+
+       if (!np)
+               return NULL;
+
+       d = bus_find_device_by_of_node(&mdio_bus_type, np);
+       if (!d)
+               return NULL;
+
+       return to_mdio_device(d);
+}
+EXPORT_SYMBOL(of_mdio_find_device);
+
 /**
  * of_phy_find_device - Give a PHY node, find the phy_device
  * @phy_np: Pointer to the phy's device tree node
@@ -346,19 +369,16 @@ EXPORT_SYMBOL(of_mdiobus_register);
  */
 struct phy_device *of_phy_find_device(struct device_node *phy_np)
 {
-       struct device *d;
        struct mdio_device *mdiodev;
 
-       if (!phy_np)
+       mdiodev = of_mdio_find_device(phy_np);
+       if (!mdiodev)
                return NULL;
 
-       d = bus_find_device_by_of_node(&mdio_bus_type, phy_np);
-       if (d) {
-               mdiodev = to_mdio_device(d);
-               if (mdiodev->flags & MDIO_DEVICE_FLAG_PHY)
-                       return to_phy_device(d);
-               put_device(d);
-       }
+       if (mdiodev->flags & MDIO_DEVICE_FLAG_PHY)
+               return to_phy_device(&mdiodev->dev);
+
+       put_device(&mdiodev->dev);
 
        return NULL;
 }
index 1efb88d9f8928405448e46a6f0aa789c568c310e..cfe8c607a628d04010859a1b56663167ef6540a7 100644 (file)
@@ -17,6 +17,7 @@ bool of_mdiobus_child_is_phy(struct device_node *child);
 int of_mdiobus_register(struct mii_bus *mdio, struct device_node *np);
 int devm_of_mdiobus_register(struct device *dev, struct mii_bus *mdio,
                             struct device_node *np);
+struct mdio_device *of_mdio_find_device(struct device_node *np);
 struct phy_device *of_phy_find_device(struct device_node *phy_np);
 struct phy_device *
 of_phy_connect(struct net_device *dev, struct device_node *phy_np,
@@ -74,6 +75,11 @@ static inline int of_mdiobus_register(struct mii_bus *mdio, struct device_node *
        return mdiobus_register(mdio);
 }
 
+static inline struct mdio_device *of_mdio_find_device(struct device_node *np)
+{
+       return NULL;
+}
+
 static inline struct phy_device *of_phy_find_device(struct device_node *phy_np)
 {
        return NULL;