Message ID | 20190917000933.GA254663@dtor-ws |
---|---|
State | Deferred |
Delegated to: | David Miller |
Headers | show |
Series | [v3] net: mdio: switch to using gpiod_get_optional() | expand |
On Mon, Sep 16, 2019 at 05:09:33PM -0700, Dmitry Torokhov wrote: > The MDIO device reset line is optional and now that gpiod_get_optional() > returns proper value when GPIO support is compiled out, there is no > reason to use fwnode_get_named_gpiod() that I plan to hide away. > > Let's switch to using more standard gpiod_get_optional() and > gpiod_set_consumer_name() to keep the nice "PHY reset" label. > > Reviewed-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com> > Signed-off-by: Dmitry Torokhov <dmitry.torokhov@gmail.com> Hi Dmitry What path into mainline do you expect this to take? Via DaveM? net-next is closed now, so i guess you will need to repost in two weeks time. Reviewed-by: Andrew Lunn <andrew@lunn.ch> Andrew
diff --git a/drivers/net/phy/mdio_bus.c b/drivers/net/phy/mdio_bus.c index ce940871331e..88c6ef7c7b13 100644 --- a/drivers/net/phy/mdio_bus.c +++ b/drivers/net/phy/mdio_bus.c @@ -42,22 +42,16 @@ static int mdiobus_register_gpiod(struct mdio_device *mdiodev) { - struct gpio_desc *gpiod = NULL; + int error; /* Deassert the optional reset signal */ - if (mdiodev->dev.of_node) - gpiod = fwnode_get_named_gpiod(&mdiodev->dev.of_node->fwnode, - "reset-gpios", 0, GPIOD_OUT_LOW, - "PHY reset"); - if (IS_ERR(gpiod)) { - if (PTR_ERR(gpiod) == -ENOENT || PTR_ERR(gpiod) == -ENOSYS) - gpiod = NULL; - else - return PTR_ERR(gpiod); - } - - mdiodev->reset_gpio = gpiod; + mdiodev->reset_gpio = gpiod_get_optional(&mdiodev->dev, + "reset", GPIOD_OUT_LOW); + error = PTR_ERR_OR_ZERO(mdiodev->reset_gpio); + if (error) + return error; + gpiod_set_consumer_name(mdiodev->reset_gpio, "PHY reset"); return 0; }