Hi, Ok I will do these changes. However, the property "phy_regulator" was not documented in Documentation/devicetree/bindings/net/rockchip-dwmac.txt ... I can document it anyway, it has probably been forgotten. Thanks ! 2015-01-19 21:12 GMT+01:00 Heiko Stübner <heiko@xxxxxxxxx>: > Am Montag, 19. Januar 2015, 18:08:08 schrieb Romain Perier: >> Currently, dwmac-rk uses a custom propety "phy_regulator" to get the name of >> the right regulator to use to power on or power off the phy. This commit >> converts the driver to use phy-supply devicetree property and the >> corresponding API, it cleans the code a bit and make it simpler to >> maintain. >> >> Signed-off-by: Romain Perier <romain.perier@xxxxxxxxx> > > I don't see updated binding documentation in here. > > Secondly I think patch4 which changes the property in the evb-rk808 could be > in here. The change is simple enough and would keep it bisectable without > needing transistional patches. > > And there is the one warning down below. > > >> --- >> drivers/net/ethernet/stmicro/stmmac/dwmac-rk.c | 61 >> ++++++++------------------ 1 file changed, 19 insertions(+), 42 >> deletions(-) >> >> diff --git a/drivers/net/ethernet/stmicro/stmmac/dwmac-rk.c >> b/drivers/net/ethernet/stmicro/stmmac/dwmac-rk.c index 06e1637..8a8091c >> 100644 >> --- a/drivers/net/ethernet/stmicro/stmmac/dwmac-rk.c >> +++ b/drivers/net/ethernet/stmicro/stmmac/dwmac-rk.c >> @@ -32,7 +32,7 @@ >> struct rk_priv_data { >> struct platform_device *pdev; >> int phy_iface; >> - char regulator[32]; >> + struct regulator *regulator; >> >> bool clk_enabled; >> bool clock_input; >> @@ -287,46 +287,25 @@ static int gmac_clk_enable(struct rk_priv_data >> *bsp_priv, bool enable) >> >> static int phy_power_on(struct rk_priv_data *bsp_priv, bool enable) >> { >> - struct regulator *ldo; >> - char *ldostr = bsp_priv->regulator; >> + struct regulator *ldo = bsp_priv->regulator; >> int ret; >> struct device *dev = &bsp_priv->pdev->dev; >> >> - if (!ldostr) { >> - dev_err(dev, "%s: no ldo found\n", __func__); >> + if (!ldo) { >> + dev_err(dev, "%s: no regulator found\n", __func__); >> return -1; >> } >> >> - ldo = regulator_get(NULL, ldostr); >> - if (!ldo) { >> - dev_err(dev, "\n%s get ldo %s failed\n", __func__, ldostr); >> + if (enable) { >> + ret = regulator_enable(ldo); >> + if (ret) >> + dev_err(dev, "%s: fail to enable phy-supply\n", >> + __func__); >> } else { >> - if (enable) { >> - if (!regulator_is_enabled(ldo)) { >> - ret = regulator_enable(ldo); >> - if (ret != 0) >> - dev_err(dev, "%s: fail to enable %s\n", >> - __func__, ldostr); >> - else >> - dev_info(dev, "turn on ldo done.\n"); >> - } else { >> - dev_warn(dev, "%s is enabled before enable", >> - ldostr); >> - } >> - } else { >> - if (regulator_is_enabled(ldo)) { >> - ret = regulator_disable(ldo); >> - if (ret != 0) >> - dev_err(dev, "%s: fail to disable %s\n", >> - __func__, ldostr); >> - else >> - dev_info(dev, "turn off ldo done.\n"); >> - } else { >> - dev_warn(dev, "%s is disabled before disable", >> - ldostr); >> - } >> - } >> - regulator_put(ldo); >> + ret = regulator_disable(ldo); >> + if (ret) >> + dev_err(dev, "%s: fail to disable phy-supply\n", >> + __func__); >> } >> >> return 0; >> @@ -346,14 +325,12 @@ static void *rk_gmac_setup(struct platform_device >> *pdev) >> >> bsp_priv->phy_iface = of_get_phy_mode(dev->of_node); >> >> - ret = of_property_read_string(dev->of_node, "phy_regulator", &strings); >> - if (ret) { >> - dev_warn(dev, "%s: Can not read property: phy_regulator.\n", >> - __func__); >> - } else { >> - dev_info(dev, "%s: PHY power controlled by regulator(%s).\n", >> - __func__, strings); >> - strcpy(bsp_priv->regulator, strings); >> + bsp_priv->regulator = devm_regulator_get_optional(dev, "phy"); >> + if (IS_ERR(bsp_priv->regulator)) { >> + if (PTR_ERR(bsp_priv->regulator) == -EPROBE_DEFER) >> + return -EPROBE_DEFER; > > this should be return ERR_PTR(-EPROBE_DEFER) and produces a warning in its > current state > > >> + dev_err(dev, "no regulator found\n"); >> + bsp_priv->regulator = NULL; >> } >> >> ret = of_property_read_string(dev->of_node, "clock_in_out", &strings); > -- 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