Re: [PATCH v2 1/3] phy: phy-core: allow specifying supply at port level

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

 






On Thursday 02 April 2015 02:20 AM, Arun Ramamurthy wrote:
From: Dmitry Torokhov <dtor@xxxxxxxxxxxx>

Multi-port phy's may have per-port power supplies. Let's change phy core
to first attempt to look up the supply at the port level, and then, if
not found, check parent device.

This commit log is no longer valid.

-Kishon


Signed-off-by: Dmitry Torokhov <dtor@xxxxxxxxxxxx>
Signed-off-by: Arun Ramamurthy <arun.ramamurthy@xxxxxxxxxxxx>
Reviewed-by: Ray Jui <rjui@xxxxxxxxxxxx>
Reviewed-by: Scott Branden <sbranden@xxxxxxxxxxxx>
---
  drivers/phy/phy-core.c | 23 ++++++++++-------------
  1 file changed, 10 insertions(+), 13 deletions(-)

diff --git a/drivers/phy/phy-core.c b/drivers/phy/phy-core.c
index 3791838..c7aa297 100644
--- a/drivers/phy/phy-core.c
+++ b/drivers/phy/phy-core.c
@@ -651,16 +651,6 @@ struct phy *phy_create(struct device *dev, struct device_node *node,
  		goto free_phy;
  	}

-	/* phy-supply */
-	phy->pwr = regulator_get_optional(dev, "phy");
-	if (IS_ERR(phy->pwr)) {
-		if (PTR_ERR(phy->pwr) == -EPROBE_DEFER) {
-			ret = -EPROBE_DEFER;
-			goto free_ida;
-		}
-		phy->pwr = NULL;
-	}
-
  	device_initialize(&phy->dev);
  	mutex_init(&phy->mutex);

@@ -674,6 +664,16 @@ struct phy *phy_create(struct device *dev, struct device_node *node,
  	if (ret)
  		goto put_dev;

+	/* phy-supply */
+	phy->pwr = regulator_get_optional(&phy->dev, "phy");
+	if (IS_ERR(phy->pwr)) {
+		ret = PTR_ERR(phy->pwr);
+		if (ret == -EPROBE_DEFER)
+			goto put_dev;
+
+		phy->pwr = NULL;
+	}
+
  	ret = device_add(&phy->dev);
  	if (ret)
  		goto put_dev;
@@ -689,9 +689,6 @@ put_dev:
  	put_device(&phy->dev);  /* calls phy_release() which frees resources */
  	return ERR_PTR(ret);

-free_ida:
-	ida_simple_remove(&phy_ida, phy->id);
-
  free_phy:
  	kfree(phy);
  	return ERR_PTR(ret);

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




[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]
  Powered by Linux