usb: phy: nop: Handle RESET for the PHY
authorRoger Quadros <rogerq@ti.com>
Tue, 12 Mar 2013 11:24:23 +0000 (13:24 +0200)
committerFelipe Balbi <balbi@ti.com>
Mon, 18 Mar 2013 09:14:53 +0000 (11:14 +0200)
We expect the RESET line to be modeled as a regulator with supply
name "reset". The regulator should be modeled such that enabling
the regulator brings the PHY device out of RESET and disabling the
regulator holds the device in RESET.

They PHY will be held in RESET in .shutdown() and brought out of
RESET in .init().

Signed-off-by: Roger Quadros <rogerq@ti.com>
Signed-off-by: Felipe Balbi <balbi@ti.com>
drivers/usb/otg/nop-usb-xceiv.c

index fbdcfef7169b498b9832b0b7874eb481f5b2d837..6efc9b793333e117b108c80496d7ea9e6ac2cc09 100644 (file)
@@ -40,6 +40,7 @@ struct nop_usb_xceiv {
        struct device           *dev;
        struct clk              *clk;
        struct regulator        *vcc;
+       struct regulator        *reset;
 };
 
 static struct platform_device *pd;
@@ -80,6 +81,12 @@ static int nop_init(struct usb_phy *phy)
        if (!IS_ERR(nop->clk))
                clk_enable(nop->clk);
 
+       if (!IS_ERR(nop->reset)) {
+               /* De-assert RESET */
+               if (regulator_enable(nop->reset))
+                       dev_err(phy->dev, "Failed to de-assert reset\n");
+       }
+
        return 0;
 }
 
@@ -87,6 +94,12 @@ static void nop_shutdown(struct usb_phy *phy)
 {
        struct nop_usb_xceiv *nop = dev_get_drvdata(phy->dev);
 
+       if (!IS_ERR(nop->reset)) {
+               /* Assert RESET */
+               if (regulator_disable(nop->reset))
+                       dev_err(phy->dev, "Failed to assert reset\n");
+       }
+
        if (!IS_ERR(nop->clk))
                clk_disable(nop->clk);
 
@@ -172,6 +185,12 @@ static int nop_usb_xceiv_probe(struct platform_device *pdev)
                                        PTR_ERR(nop->vcc));
        }
 
+       nop->reset = devm_regulator_get(&pdev->dev, "reset");
+       if (IS_ERR(nop->reset)) {
+               dev_dbg(&pdev->dev, "Error getting reset regulator: %ld\n",
+                                       PTR_ERR(nop->reset));
+       }
+
        nop->dev                = &pdev->dev;
        nop->phy.dev            = nop->dev;
        nop->phy.label          = "nop-xceiv";