Re: [U-Boot] [RESEND PATCH v4] usb: dwc2: add support for external vbus supply

[Marek Vasut]

On 03/06/2017 01:54 PM, Kever Yang wrote:

Some board do not use the dwc2 internal VBUS_DRV signal, but
use a gpio pin to enable the 5.0V VBUS power, add interface to
enable the power in dwc2 driver.

Signed-off-by: Kever Yang <kever.y...@rock-chips.com>
Signed-off-by: Simon Glass <s...@chromium.org>
---

Changes in v4:
- Drop no use code comment by Marek.

Changes in v3:
- Drop use of static variable

 drivers/usb/host/dwc2.c | 37 ++++++++++++++++++++++++++++++++-----
 1 file changed, 32 insertions(+), 5 deletions(-)

diff --git a/drivers/usb/host/dwc2.c b/drivers/usb/host/dwc2.c
index d253b94..5ac602e 100644
--- a/drivers/usb/host/dwc2.c
+++ b/drivers/usb/host/dwc2.c
@@ -15,6 +15,7 @@
 #include <usbroothubdes.h>
 #include <wait_bit.h>
 #include <asm/io.h>
+#include <power/regulator.h>

 #include "dwc2.h"

@@ -159,6 +160,27 @@ static void dwc_otg_core_reset(struct dwc2_core_regs *regs)
        mdelay(100);
 }

Nit:

add the ifdef around the function, ie.

#ifdef (.....)
static foo()
{
 do stuff
}
#else
static foo() { return 0 }
#endif

The rest is great, thanks!

+static int dwc_vbus_supply_init(struct udevice *dev)
+{
+#if defined(CONFIG_DM_USB) && defined(CONFIG_DM_REGULATOR)
+       struct udevice *vbus_supply;
+       int ret;
+
+       ret = device_get_supply_regulator(dev, "vbus-supply", &vbus_supply);
+       if (ret) {
+               debug("%s: No vbus supply\n", dev->name);
+               return 0;
+       }
+
+       ret = regulator_set_enable(vbus_supply, true);
+       if (ret) {
+               error("Error enabling vbus supply\n");
+               return ret;
+       }
+#endif
+       return 0;
+}
+
 /*
  * This function initializes the DWC_otg controller registers for
  * host mode.
@@ -167,10 +189,12 @@ static void dwc_otg_core_reset(struct dwc2_core_regs 
*regs)
  * request queues. Host channels are reset to ensure that they are ready for
  * performing transfers.
  *
+ * @param dev USB Device (NULL if driver model is not being used)
  * @param regs Programming view of DWC_otg controller
  *
  */
-static void dwc_otg_core_host_init(struct dwc2_core_regs *regs)
+static void dwc_otg_core_host_init(struct udevice *dev,
+                                  struct dwc2_core_regs *regs)
 {
        uint32_t nptxfifosize = 0;
        uint32_t ptxfifosize = 0;
@@ -248,6 +272,9 @@ static void dwc_otg_core_host_init(struct dwc2_core_regs 
*regs)
                        writel(hprt0, &regs->hprt0);
                }
        }
+
+       if (dev)
+               dwc_vbus_supply_init(dev);
 }

 /*
@@ -1048,7 +1075,7 @@ int _submit_int_msg(struct dwc2_priv *priv, struct 
usb_device *dev,
        }
 }

-static int dwc2_init_common(struct dwc2_priv *priv)
+static int dwc2_init_common(struct udevice *dev, struct dwc2_priv *priv)
 {
        struct dwc2_core_regs *regs = priv->regs;
        uint32_t snpsid;
@@ -1070,7 +1097,7 @@ static int dwc2_init_common(struct dwc2_priv *priv)
 #endif

        dwc_otg_core_init(priv);
-       dwc_otg_core_host_init(regs);
+       dwc_otg_core_host_init(dev, regs);

        clrsetbits_le32(&regs->hprt0, DWC2_HPRT0_PRTENA |
                        DWC2_HPRT0_PRTCONNDET | DWC2_HPRT0_PRTENCHNG |
@@ -1143,7 +1170,7 @@ int usb_lowlevel_init(int index, enum usb_init_type init, 
void **controller)
        if (board_usb_init(index, USB_INIT_HOST))
                return -1;

-       return dwc2_init_common(priv);
+       return dwc2_init_common(NULL, priv);
 }

 int usb_lowlevel_stop(int index)
@@ -1214,7 +1241,7 @@ static int dwc2_usb_probe(struct udevice *dev)

        bus_priv->desc_before_addr = true;

-       return dwc2_init_common(priv);
+       return dwc2_init_common(dev, priv);
 }

 static int dwc2_usb_remove(struct udevice *dev)

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot