usb: tegra: Use regulators instead of GPIOs for USB PHY VBUS

The tegra ehci driver has enabled USB vbus regulators directly using
GPIOs and the device tree attribute nvidia,vbus-gpio. This is ugly
and causes error messages on boot when both the regulator driver
and the ehci driver want access to the same GPIO.

After this patch, usb vbus regulators for tegra usb phy devices are specified
with the device tree attribute vbus-supply = <&x> where x is a regulator defined
in the device tree. The old nvidia,vbus-gpio property is no longer supported.

Signed-off-by: Mikko Perttunen <mperttunen@nvidia.com>
Reviewed-by: Stephen Warren <swarren@nvidia.com>
Tested-by: Stephen Warren <swarren@nvidia.com>
Signed-off-by: Felipe Balbi <balbi@ti.com>

diff --git a/drivers/usb/host/ehci-tegra.c b/drivers/usb/host/ehci-tegra.c
index 6ee7ef79b4f86a5c802e33245592d40ec27127a5..14c1f35a4b102e47525bad0d89608e4b1086bf1b 100644 (file)
--- a/drivers/usb/host/ehci-tegra.c
+++ b/drivers/usb/host/ehci-tegra.c
@@ -322,33 +322,6 @@ static void tegra_ehci_unmap_urb_for_dma(struct usb_hcd *hcd, struct urb *urb)
        free_dma_aligned_buffer(urb);
 }
 
-static int setup_vbus_gpio(struct platform_device *pdev,
-                          struct tegra_ehci_platform_data *pdata)
-{
-       int err = 0;
-       int gpio;
-
-       gpio = pdata->vbus_gpio;
-       if (!gpio_is_valid(gpio))
-               gpio = of_get_named_gpio(pdev->dev.of_node,
-                                        "nvidia,vbus-gpio", 0);
-       if (!gpio_is_valid(gpio))
-               return 0;
-
-       err = gpio_request(gpio, "vbus_gpio");
-       if (err) {
-               dev_err(&pdev->dev, "can't request vbus gpio %d", gpio);
-               return err;
-       }
-       err = gpio_direction_output(gpio, 1);
-       if (err) {
-               dev_err(&pdev->dev, "can't enable vbus\n");
-               return err;
-       }
-
-       return err;
-}
-
 static int tegra_ehci_probe(struct platform_device *pdev)
 {
        struct resource *res;
@@ -376,14 +349,11 @@ static int tegra_ehci_probe(struct platform_device *pdev)
        if (!pdev->dev.coherent_dma_mask)
                pdev->dev.coherent_dma_mask = DMA_BIT_MASK(32);
 
-       setup_vbus_gpio(pdev, pdata);
-
        hcd = usb_create_hcd(&tegra_ehci_hc_driver, &pdev->dev,
                                        dev_name(&pdev->dev));
        if (!hcd) {
                dev_err(&pdev->dev, "Unable to create HCD\n");
-               err = -ENOMEM;
-               goto cleanup_vbus_gpio;
+               return -ENOMEM;
        }
        platform_set_drvdata(pdev, hcd);
        ehci = hcd_to_ehci(hcd);
@@ -494,8 +464,6 @@ cleanup_clk_get:
        clk_put(tegra->clk);
 cleanup_hcd_create:
        usb_put_hcd(hcd);
-cleanup_vbus_gpio:
-       /* FIXME: Undo setup_vbus_gpio() here */
        return err;
 }
 

diff --git a/drivers/usb/phy/phy-tegra-usb.c b/drivers/usb/phy/phy-tegra-usb.c
index 3ecd49871b88507610eaddb5167d7a093c01734b..d92a63a5b0c9dd25b5858bc7015dd816034ab1ca 100644 (file)
--- a/drivers/usb/phy/phy-tegra-usb.c
+++ b/drivers/usb/phy/phy-tegra-usb.c
@@ -34,6 +34,7 @@
 #include <asm/mach-types.h>
 #include <linux/usb/ehci_def.h>
 #include <linux/usb/tegra_usb_phy.h>
+#include <linux/regulator/consumer.h>
 
 #define ULPI_VIEWPORT          0x170
 
@@ -613,6 +614,9 @@ static void tegra_usb_phy_close(struct usb_phy *x)
 {
        struct tegra_usb_phy *phy = container_of(x, struct tegra_usb_phy, u_phy);
 
+       if (!IS_ERR(phy->vbus))
+               regulator_disable(phy->vbus);
+
        clk_disable_unprepare(phy->pll_u);
 }
 
@@ -705,6 +709,16 @@ static int tegra_usb_phy_init(struct tegra_usb_phy *phy)
                goto fail;
        }
 
+       if (!IS_ERR(phy->vbus)) {
+               err = regulator_enable(phy->vbus);
+               if (err) {
+                       dev_err(phy->dev,
+                               "failed to enable usb vbus regulator: %d\n",
+                               err);
+                       goto fail;
+               }
+       }
+
        if (phy->is_ulpi_phy)
                err = ulpi_open(phy);
        else
@@ -896,6 +910,16 @@ static int tegra_usb_phy_probe(struct platform_device *pdev)
        } else
                tegra_phy->mode = TEGRA_USB_PHY_MODE_OTG;
 
+       /* On some boards, the VBUS regulator doesn't need to be controlled */
+       if (of_find_property(np, "vbus-supply", NULL)) {
+               tegra_phy->vbus = devm_regulator_get(&pdev->dev, "vbus");
+               if (IS_ERR(tegra_phy->vbus))
+                       return PTR_ERR(tegra_phy->vbus);
+       } else {
+               dev_notice(&pdev->dev, "no vbus regulator");
+               tegra_phy->vbus = ERR_PTR(-ENODEV);
+       }
+
        tegra_phy->dev = &pdev->dev;
        err = tegra_usb_phy_init(tegra_phy);
        if (err < 0)

diff --git a/include/linux/usb/tegra_usb_phy.h b/include/linux/usb/tegra_usb_phy.h
index d2ca919a5b738387b28293918ce9d2e47ccc053c..2b5fa947980a511ff5194fc8bd379d63a6854a56 100644 (file)
--- a/include/linux/usb/tegra_usb_phy.h
+++ b/include/linux/usb/tegra_usb_phy.h
@@ -55,6 +55,7 @@ struct tegra_usb_phy {
        struct clk *clk;
        struct clk *pll_u;
        struct clk *pad_clk;
+       struct regulator *vbus;
        enum tegra_usb_phy_mode mode;
        void *config;
        struct usb_phy *ulpi;