Re: [PATCH 1/3] phy: core: Add devm_of_phy_get_by_index to phy-core

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 





On 15-03-25 03:03 PM, Kishon Vijay Abraham I wrote:
Hi,

On Saturday 21 March 2015 02:59 AM, Arun Ramamurthy wrote:


On 15-03-20 02:26 PM, Dmitry Torokhov wrote:
Hi Arun,

On Fri, Mar 20, 2015 at 02:07:08PM -0700, Arun Ramamurthy wrote:
Adding devm_of_phy_get_by_index to get phys by supplying an index
and not a phy name when multiple phys are declared

I think a bit more explanation on why get_by_index is needed here.
Thanks Kison. Can you be more specific? I am unsure of what more I can explain here.

Reviewed-by: Ray Jui <rjui@xxxxxxxxxxxx>
Reviewed-by: Scott Branden <sbranden@xxxxxxxxxxxx>
Signed-off-by: Arun Ramamurthy <arun.ramamurthy@xxxxxxxxxxxx>
---
   drivers/phy/phy-core.c  | 30 ++++++++++++++++++++++++++++++
   include/linux/phy/phy.h |  2 ++
   2 files changed, 32 insertions(+)

diff --git a/drivers/phy/phy-core.c b/drivers/phy/phy-core.c
index a12d353..0c03876 100644
--- a/drivers/phy/phy-core.c
+++ b/drivers/phy/phy-core.c
@@ -622,6 +622,36 @@ struct phy *devm_of_phy_get(struct device *dev, struct
device_node *np,
   EXPORT_SYMBOL_GPL(devm_of_phy_get);

   /**
+ * devm_of_phy_get_by_index() - lookup and obtain a reference to a phy by
index.
+ * @dev: device that requests this phy
+ * @np: node containing the phy
+ * @index: index of the phy
+ *
+ * Gets the phy using _of_phy_get(), and associates a device with it using
+ * devres. On driver detach, release function is invoked on the devres data,
+ * then, devres data is freed.
+ *
+ */
+struct phy *devm_of_phy_get_by_index(struct device *dev, struct device_node
*np,
+                     int index)
+{
+    struct phy **ptr, *phy;
+
+    ptr = devres_alloc(devm_phy_release, sizeof(*ptr), GFP_KERNEL);
+    if (!ptr)
+        return ERR_PTR(-ENOMEM);
+
+    phy = _of_phy_get(np, index);
+    if (!IS_ERR(phy)) {
+        *ptr = phy;
+        devres_add(dev, ptr);
+    } else {
+        devres_free(ptr);
+    }
+
+    return phy;
+}

You want EXPORT_SYMBOL_GPL(devm_of_phy_get_by_index); here.

Also update the Documentation/phy.txt.

Ok will do.
-Kishon

--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux