Convert Xilinx ZynqMP to DM_USB with generic DWC3 glue logic driver

From: Mugunthan V N <[email protected]>

The patch is preparing dwc3 core for enabling DM_USB with peripheral driver with using driver model support.
The driver will be bound by the DWC3 wrapper driver based on the dr_mode device tree entry.

Signed-off-by Mugunthan V N <[email protected]>

(Remove dwc3-omap changes)

Signed-off-by Michal Simek <[email protected]>

---

Changes in v2:
- Retype dwc->dev in gadget.c to avoid compilation warning
- Guard udevice with __UBOOT__
- Move ifdef in core.c to avoid compilation issues for some platforms (for example edison)

Origin series here: https://patchwork.ozlabs.org/patch/775108/

- Fix labels as was asked in previous review.

---
  drivers/usb/dwc3/core.c   | 55 +++++++++++++++++++++++++++++++++++++++
  drivers/usb/dwc3/core.h   |  6 +++++
  drivers/usb/dwc3/gadget.c |  2 +-
  3 files changed, 62 insertions(+), 1 deletion(-)

drivers/usb/dwc3/core.c +0 -0

index dbdad22d1134..1ab5cee60969 100644

drivers/usb/dwc3/core.c +53 -0

+
+int dwc3_init(struct dwc3 *dwc)
+{
+	int ret;
+
+	dwc3_cache_hwparams(dwc);
+
+	ret = dwc3_alloc_event_buffers(dwc, DWC3_EVENT_BUFFERS_SIZE);
+	if (ret) {
+		dev_err(dwc->dev, "failed to allocate event buffers\n");
+		return -ENOMEM;
+	}
+
+	ret = dwc3_core_init(dwc);
+	if (ret) {
+		dev_err(dev, "failed to initialize core\n");
+		goto core_fail;
+	}
+
+	ret = dwc3_event_buffers_setup(dwc);
+	if (ret) {
+		dev_err(dwc->dev, "failed to setup event buffers\n");
+		goto event_fail;
+	}
+
+	ret = dwc3_core_init_mode(dwc);
+	if (ret)
+		goto mode_fail;
+
+	return 0;
+
+mode_fail:
+	dwc3_event_buffers_cleanup(dwc);
+
+event_fail:
+	dwc3_core_exit(dwc);
+
+core_fail:
+	dwc3_free_event_buffers(dwc);
+
+	return ret;
+}
+
+void dwc3_remove(struct dwc3 *dwc)
+{
+	dwc3_core_exit_mode(dwc);
+	dwc3_event_buffers_cleanup(dwc);
+	dwc3_free_event_buffers(dwc);
+	dwc3_core_exit(dwc);
+	kfree(dwc->mem);
+}
+
+#endif

drivers/usb/dwc3/core.h +6 -0

index cbe9850a0bda..58fe91dc5131 100644
@@ -712,7 +712,11 @@ struct dwc3 {
  	/* device lock */
  	spinlock_t		lock;
 
+#if defined(__UBOOT__) && defined(CONFIG_DM_USB)
+	struct udevice		*dev;
+#else
  	struct device		*dev;
+#endif
 
  	struct platform_device	*xhci;
  	struct resource		xhci_resources[DWC3_XHCI_RESOURCES_NUM];

@@ -987,6 +991,8 @@ struct dwc3_gadget_ep_cmd_params {
 
  /* prototypes */
  int dwc3_gadget_resize_tx_fifos(struct dwc3 *dwc);
+int dwc3_init(struct dwc3 *dwc);
+void dwc3_remove(struct dwc3 *dwc);
 
  #ifdef CONFIG_USB_DWC3_HOST
  int dwc3_host_init(struct dwc3 *dwc);

drivers/usb/dwc3/gadget.c +1 -1

index d45fae044c4a..e340cb268fcd 100644
@@ -2609,7 +2609,7 @@ int dwc3_gadget_init(struct dwc3 *dwc)
  	if (ret)
  		goto err4;
 
-	ret = usb_add_gadget_udc(dwc->dev, &dwc->gadget);
+	ret = usb_add_gadget_udc((struct device *)dwc->dev, &dwc->gadget);
  	if (ret) {
  		dev_err(dwc->dev, "failed to register udc\n");
  		goto err4;

By enabling BLK by default this is the next driver which needs to get support for DM_USB. Adding generic DWC3 glue logic which only parse nodes and read device mode. Based on it probe proper host/peripheral DWC3 drivers for it.

Signed-off-by Michal Simek <[email protected]>

---

Changes in v2:
- Change style to avoid correct but not nice indentation by using char *driver variable (suggested by Marek)

  drivers/usb/dwc3/Kconfig        |   6 ++
  drivers/usb/dwc3/Makefile       |   1 +
  drivers/usb/dwc3/dwc3-generic.c | 157 ++++++++++++++++++++++++++++++++
  3 files changed, 164 insertions(+)

create mode 100644 drivers/usb/dwc3/dwc3-generic.c

drivers/usb/dwc3/Kconfig +6 -0

index ae7fc1c6304d..943b7630eba4 100644
@@ -37,6 +37,12 @@ config USB_DWC3_OMAP
 
  	  Say 'Y' here if you have one such device
 
+config USB_DWC3_GENERIC
+	bool "Xilinx ZynqMP and similar Platforms"
+	depends on DM_USB && USB_DWC3
+	help
+	  Some platforms can reuse this DWC3 generic implementation.
+
  config USB_DWC3_UNIPHIER
  	bool "DesignWare USB3 Host Support on UniPhier Platforms"
  	depends on ARCH_UNIPHIER && USB_XHCI_DWC3

drivers/usb/dwc3/Makefile +1 -0

index cd18b8d9ec02..60b5515a67da 100644
@@ -7,6 +7,7 @@ dwc3-y					:= core.o
  obj-$(CONFIG_USB_DWC3_GADGET)		+= gadget.o ep0.o
 
  obj-$(CONFIG_USB_DWC3_OMAP)		+= dwc3-omap.o
+obj-$(CONFIG_USB_DWC3_GENERIC)		+= dwc3-generic.o
  obj-$(CONFIG_USB_DWC3_UNIPHIER)		+= dwc3-uniphier.o
  obj-$(CONFIG_USB_DWC3_PHY_OMAP)		+= ti_usb_phy.o
  obj-$(CONFIG_USB_DWC3_PHY_SAMSUNG)	+= samsung_usb_phy.o

drivers/usb/dwc3/dwc3-generic.c new file +76 -0

new file mode 100644
index 000000000000..ca63eac3d98e
@@ -0,0 +1,157 @@
+// SPDX-License-Identifier:     GPL-2.0
+/*
+ * Generic DWC3 Glue layer
+ *
+ * Copyright (C) 2016 - 2018 Xilinx, Inc.
+ *
+ * Based on dwc3-omap.c.
+ */
+
+#include <common.h>
+#include <dm.h>
+#include <dm/device-internal.h>
+#include <dm/lists.h>
+#include <linux/usb/otg.h>
+#include <linux/compat.h>
+#include <linux/usb/ch9.h>
+#include <linux/usb/gadget.h>
+#include <malloc.h>
+#include <usb.h>
+#include "core.h"
+#include "gadget.h"
+#include "linux-compat.h"
+
+DECLARE_GLOBAL_DATA_PTR;
+
+int usb_gadget_handle_interrupts(int index)
+{
+	struct dwc3 *priv;
+	struct udevice *dev;
+	int ret;
+
+	ret = uclass_first_device(UCLASS_USB_DEV_GENERIC, &dev);
+	if (!dev || ret) {
+		pr_err("No USB device found\n");
+		return -ENODEV;
+	}
+
+	priv = dev_get_priv(dev);
+
+	dwc3_gadget_uboot_handle_interrupt(priv);
+
+	return 0;
+}
+
+static int dwc3_generic_peripheral_probe(struct udevice *dev)
+{
+	struct dwc3 *priv = dev_get_priv(dev);
+
+	return dwc3_init(priv);
+}
+
+static int dwc3_generic_peripheral_remove(struct udevice *dev)
+{
+	struct dwc3 *priv = dev_get_priv(dev);
+
+	dwc3_remove(priv);
+
+	return 0;
+}
+
+static int dwc3_generic_peripheral_ofdata_to_platdata(struct udevice *dev)
+{
+	struct dwc3 *priv = dev_get_priv(dev);
+	int node = dev_of_offset(dev);
+
+	priv->regs = (void *)devfdt_get_addr(dev);
+	priv->regs += DWC3_GLOBALS_REGS_START;
+
+	priv->maximum_speed = usb_get_maximum_speed(node);
+	if (priv->maximum_speed == USB_SPEED_UNKNOWN) {
+		pr_err("Invalid usb maximum speed\n");
+		return -ENODEV;
+	}
+
+	priv->dr_mode = usb_get_dr_mode(node);
+	if (priv->dr_mode == USB_DR_MODE_UNKNOWN) {

drivers/usb/dwc3/dwc3-generic.c new file +71 -0

+}
+
+U_BOOT_DRIVER(dwc3_generic_peripheral) = {
+	.name	= "dwc3-generic-peripheral",
+	.id	= UCLASS_USB_DEV_GENERIC,
+	.ofdata_to_platdata = dwc3_generic_peripheral_ofdata_to_platdata,
+	.probe = dwc3_generic_peripheral_probe,
+	.remove = dwc3_generic_peripheral_remove,
+	.bind = dwc3_generic_peripheral_bind,
+	.platdata_auto_alloc_size = sizeof(struct usb_platdata),
+	.priv_auto_alloc_size = sizeof(struct dwc3),
+	.flags	= DM_FLAG_ALLOC_PRIV_DMA,
+};
+
+static int dwc3_generic_bind(struct udevice *parent)
+{
+	const void *fdt = gd->fdt_blob;
+	int node;
+	int ret;
+
+	for (node = fdt_first_subnode(fdt, dev_of_offset(parent)); node > 0;
+	     node = fdt_next_subnode(fdt, node)) {
+		const char *name = fdt_get_name(fdt, node, NULL);
+		enum usb_dr_mode dr_mode;
+		struct udevice *dev;
+		const char *driver;
+
+		debug("%s: subnode name: %s\n", __func__, name);
+		if (strncmp(name, "dwc3@", 4))
+			continue;
+
+		dr_mode = usb_get_dr_mode(node);
+
+		switch (dr_mode) {
+		case USB_DR_MODE_PERIPHERAL:
+		case USB_DR_MODE_OTG:
+			debug("%s: dr_mode: OTG or Peripheral\n", __func__);
+			driver = "dwc3-generic-peripheral";
+			break;
+		case USB_DR_MODE_HOST:
+			debug("%s: dr_mode: HOST\n", __func__);
+			driver = "dwc3-generic-host";
+			break;
+		default:
+			debug("%s: unsupported dr_mode\n", __func__);
+			return -ENODEV;
+		};
+
+		ret = device_bind_driver_to_node(parent, driver, name,
+						 offset_to_ofnode(node), &dev);
+		if (ret) {
+			debug("%s: not able to bind usb device mode\n",
+			      __func__);
+			return ret;
+		}
+	}
+
+	return 0;
+}
+
+static const struct udevice_id dwc3_generic_ids[] = {
+	{ .compatible = "xlnx,zynqmp-dwc3" },
+	{ }
+};
+
+U_BOOT_DRIVER(dwc3_generic_wrapper) = {
+	.name	= "dwc3-generic-wrapper",
+	.id	= UCLASS_MISC,
+	.of_match = dwc3_generic_ids,
+	.bind = dwc3_generic_bind,
+};

Quoted 134 lines · Michal Simek

By enabling BLK by default this is the next driver which needs to get support for DM_USB. Adding generic DWC3 glue logic which only parse nodes and read device mode. Based on it probe proper host/peripheral DWC3 drivers for it.

Signed-off-by Michal Simek <[email protected]>

---

Changes in v2:
- Change style to avoid correct but not nice indentation by using char *driver variable (suggested by Marek)

  drivers/usb/dwc3/Kconfig        |   6 ++ drivers/usb/dwc3/Makefile       |   1 + drivers/usb/dwc3/dwc3-generic.c | 157 ++++++++++++++++++++++++++++++++

  3 files changed, 164 insertions(+)

  create mode 100644 drivers/usb/dwc3/dwc3-generic.c

drivers/usb/dwc3/Kconfig +0 -0

index ae7fc1c6304d..943b7630eba4 100644
@@ -37,6 +37,12 @@ config USB_DWC3_OMAP

          Say 'Y' here if you have one such device +config USB_DWC3_GENERIC
+    bool "Xilinx ZynqMP and similar Platforms"
+    depends on DM_USB && USB_DWC3
+    help
+      Some platforms can reuse this DWC3 generic implementation. +
  config USB_DWC3_UNIPHIER
      bool "DesignWare USB3 Host Support on UniPhier Platforms" depends on ARCH_UNIPHIER && USB_XHCI_DWC3

drivers/usb/dwc3/Makefile +0 -0

index cd18b8d9ec02..60b5515a67da 100644
@@ -7,6 +7,7 @@ dwc3-y                    := core.o

  obj-$(CONFIG_USB_DWC3_GADGET)        += gadget.o ep0.o obj-$(CONFIG_USB_DWC3_OMAP)        += dwc3-omap.o +obj-$(CONFIG_USB_DWC3_GENERIC)        += dwc3-generic.o obj-$(CONFIG_USB_DWC3_UNIPHIER)        += dwc3-uniphier.o obj-$(CONFIG_USB_DWC3_PHY_OMAP)        += ti_usb_phy.o obj-$(CONFIG_USB_DWC3_PHY_SAMSUNG)    += samsung_usb_phy.o

drivers/usb/dwc3/dwc3-generic.c new file +61 -0

new file mode 100644
index 000000000000..ca63eac3d98e
@@ -0,0 +1,157 @@
+// SPDX-License-Identifier:     GPL-2.0
+/*
+ * Generic DWC3 Glue layer
+ *
+ * Copyright (C) 2016 - 2018 Xilinx, Inc.
+ *
+ * Based on dwc3-omap.c.
+ */
+
+#include <common.h>
+#include <dm.h>
+#include <dm/device-internal.h>
+#include <dm/lists.h>
+#include <linux/usb/otg.h>
+#include <linux/compat.h>
+#include <linux/usb/ch9.h>
+#include <linux/usb/gadget.h>
+#include <malloc.h>
+#include <usb.h>
+#include "core.h"
+#include "gadget.h"
+#include "linux-compat.h"
+
+DECLARE_GLOBAL_DATA_PTR;
+
+int usb_gadget_handle_interrupts(int index)
+{
+    struct dwc3 *priv;
+    struct udevice *dev;
+    int ret;
+
+    ret = uclass_first_device(UCLASS_USB_DEV_GENERIC, &dev);
+    if (!dev || ret) {
+        pr_err("No USB device found\n");
+        return -ENODEV;
+    }
+
+    priv = dev_get_priv(dev);
+
+    dwc3_gadget_uboot_handle_interrupt(priv);
+
+    return 0;
+}
+
+static int dwc3_generic_peripheral_probe(struct udevice *dev)
+{
+    struct dwc3 *priv = dev_get_priv(dev);
+
+    return dwc3_init(priv);
+}
+
+static int dwc3_generic_peripheral_remove(struct udevice *dev)
+{
+    struct dwc3 *priv = dev_get_priv(dev);
+
+    dwc3_remove(priv);
+
+    return 0;
+}
+
+static int dwc3_generic_peripheral_ofdata_to_platdata(struct udevice 

*dev)

drivers/usb/dwc3/dwc3-generic.c new file +21 -0

+{
+    struct dwc3 *priv = dev_get_priv(dev);
+    int node = dev_of_offset(dev);
+
+    priv->regs = (void *)devfdt_get_addr(dev);
+    priv->regs += DWC3_GLOBALS_REGS_START;
+
+    priv->maximum_speed = usb_get_maximum_speed(node);
+    if (priv->maximum_speed == USB_SPEED_UNKNOWN) {
+        pr_err("Invalid usb maximum speed\n");
+        return -ENODEV;
+    }
+
+    priv->dr_mode = usb_get_dr_mode(node);
+    if (priv->dr_mode == USB_DR_MODE_UNKNOWN) {
+        pr_err("Invalid usb mode setup\n");
+        return -ENODEV;
+    }
+
+    return 0;
+}

drivers/usb/dwc3/dwc3-generic.c line 86

Quoted 4 lines

drivers/usb/dwc3/dwc3-generic.c new file +4 -0

+
+static int dwc3_generic_peripheral_bind(struct udevice *dev)
+{
+    return device_probe(dev);

This is wrong place to probe the device.
What must be done is to probe the device when it is first used. I have a patch in progress for this. When it's cleaned up I'll share a repo so that you can have look and maybe pick it up.

Quoted 62 lines

drivers/usb/dwc3/dwc3-generic.c new file +21 -0

+}
+
+U_BOOT_DRIVER(dwc3_generic_peripheral) = {
+    .name    = "dwc3-generic-peripheral",
+    .id    = UCLASS_USB_DEV_GENERIC,
+    .ofdata_to_platdata = dwc3_generic_peripheral_ofdata_to_platdata,
+    .probe = dwc3_generic_peripheral_probe,
+    .remove = dwc3_generic_peripheral_remove,
+    .bind = dwc3_generic_peripheral_bind,
+    .platdata_auto_alloc_size = sizeof(struct usb_platdata),
+    .priv_auto_alloc_size = sizeof(struct dwc3),
+    .flags    = DM_FLAG_ALLOC_PRIV_DMA,
+};
+
+static int dwc3_generic_bind(struct udevice *parent)
+{
+    const void *fdt = gd->fdt_blob;
+    int node;
+    int ret;
+
+    for (node = fdt_first_subnode(fdt, dev_of_offset(parent)); node 

Quoted 1 line

0;

+         node = fdt_next_subnode(fdt, node)) {
+        const char *name = fdt_get_name(fdt, node, NULL); +        enum usb_dr_mode dr_mode;
+        struct udevice *dev;
+        const char *driver;

drivers/usb/dwc3/dwc3-generic.c new file +35 -0

+
+        debug("%s: subnode name: %s\n", __func__, name);
+        if (strncmp(name, "dwc3@", 4))
+            continue;
+
+        dr_mode = usb_get_dr_mode(node);
+
+        switch (dr_mode) {
+        case USB_DR_MODE_PERIPHERAL:
+        case USB_DR_MODE_OTG:
+            debug("%s: dr_mode: OTG or Peripheral\n", __func__);
+            driver = "dwc3-generic-peripheral";
+            break;
+        case USB_DR_MODE_HOST:
+            debug("%s: dr_mode: HOST\n", __func__);
+            driver = "dwc3-generic-host";
+            break;
+        default:
+            debug("%s: unsupported dr_mode\n", __func__);
+            return -ENODEV;
+        };
+
+        ret = device_bind_driver_to_node(parent, driver, name,
+                         offset_to_ofnode(node), &dev);
+        if (ret) {
+            debug("%s: not able to bind usb device mode\n",
+                  __func__);
+            return ret;
+        }
+    }
+
+    return 0;
+}
+
+static const struct udevice_id dwc3_generic_ids[] = {

drivers/usb/dwc3/dwc3-generic.c line 157

Quoted 10 lines

drivers/usb/dwc3/dwc3-generic.c new file +10 -0

+    { .compatible = "xlnx,zynqmp-dwc3" },
+    { }
+};
+
+U_BOOT_DRIVER(dwc3_generic_wrapper) = {
+    .name    = "dwc3-generic-wrapper",
+    .id    = UCLASS_MISC,
+    .of_match = dwc3_generic_ids,
+    .bind = dwc3_generic_bind,
+};

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

Quoted 137 lines · Jean-Jacques Hiblot

Quoted 134 lines · Michal Simek

By enabling BLK by default this is the next driver which needs to get support for DM_USB. Adding generic DWC3 glue logic which only parse nodes and read device mode. Based on it probe proper host/peripheral DWC3 drivers for it.

Signed-off-by Michal Simek <[email protected]>

---

Changes in v2:
- Change style to avoid correct but not nice indentation by using char *driver variable (suggested by Marek)

  drivers/usb/dwc3/Kconfig        |   6 ++ drivers/usb/dwc3/Makefile       |   1 + drivers/usb/dwc3/dwc3-generic.c | 157 ++++++++++++++++++++++++++++++++

  3 files changed, 164 insertions(+)

  create mode 100644 drivers/usb/dwc3/dwc3-generic.c

drivers/usb/dwc3/Kconfig +0 -0

index ae7fc1c6304d..943b7630eba4 100644
@@ -37,6 +37,12 @@ config USB_DWC3_OMAP

          Say 'Y' here if you have one such device +config USB_DWC3_GENERIC
+    bool "Xilinx ZynqMP and similar Platforms"
+    depends on DM_USB && USB_DWC3
+    help
+      Some platforms can reuse this DWC3 generic implementation. +
  config USB_DWC3_UNIPHIER
      bool "DesignWare USB3 Host Support on UniPhier Platforms" depends on ARCH_UNIPHIER && USB_XHCI_DWC3

drivers/usb/dwc3/Makefile +0 -0

index cd18b8d9ec02..60b5515a67da 100644
@@ -7,6 +7,7 @@ dwc3-y                    := core.o

  obj-$(CONFIG_USB_DWC3_GADGET)        += gadget.o ep0.o obj-$(CONFIG_USB_DWC3_OMAP)        += dwc3-omap.o +obj-$(CONFIG_USB_DWC3_GENERIC)        += dwc3-generic.o obj-$(CONFIG_USB_DWC3_UNIPHIER)        += dwc3-uniphier.o obj-$(CONFIG_USB_DWC3_PHY_OMAP)        += ti_usb_phy.o obj-$(CONFIG_USB_DWC3_PHY_SAMSUNG)    += samsung_usb_phy.o

drivers/usb/dwc3/dwc3-generic.c new file +61 -0

new file mode 100644
index 000000000000..ca63eac3d98e
@@ -0,0 +1,157 @@
+// SPDX-License-Identifier:     GPL-2.0
+/*
+ * Generic DWC3 Glue layer
+ *
+ * Copyright (C) 2016 - 2018 Xilinx, Inc.
+ *
+ * Based on dwc3-omap.c.
+ */
+
+#include <common.h>
+#include <dm.h>
+#include <dm/device-internal.h>
+#include <dm/lists.h>
+#include <linux/usb/otg.h>
+#include <linux/compat.h>
+#include <linux/usb/ch9.h>
+#include <linux/usb/gadget.h>
+#include <malloc.h>
+#include <usb.h>
+#include "core.h"
+#include "gadget.h"
+#include "linux-compat.h"
+
+DECLARE_GLOBAL_DATA_PTR;
+
+int usb_gadget_handle_interrupts(int index)
+{
+    struct dwc3 *priv;
+    struct udevice *dev;
+    int ret;
+
+    ret = uclass_first_device(UCLASS_USB_DEV_GENERIC, &dev);
+    if (!dev || ret) {
+        pr_err("No USB device found\n");
+        return -ENODEV;
+    }
+
+    priv = dev_get_priv(dev);
+
+    dwc3_gadget_uboot_handle_interrupt(priv);
+
+    return 0;
+}
+
+static int dwc3_generic_peripheral_probe(struct udevice *dev)
+{
+    struct dwc3 *priv = dev_get_priv(dev);
+
+    return dwc3_init(priv);
+}
+
+static int dwc3_generic_peripheral_remove(struct udevice *dev)
+{
+    struct dwc3 *priv = dev_get_priv(dev);
+
+    dwc3_remove(priv);
+
+    return 0;
+}
+
+static int dwc3_generic_peripheral_ofdata_to_platdata(struct udevice

*dev)

drivers/usb/dwc3/dwc3-generic.c new file +21 -0

+{
+    struct dwc3 *priv = dev_get_priv(dev);
+    int node = dev_of_offset(dev);
+
+    priv->regs = (void *)devfdt_get_addr(dev);
+    priv->regs += DWC3_GLOBALS_REGS_START;
+
+    priv->maximum_speed = usb_get_maximum_speed(node);
+    if (priv->maximum_speed == USB_SPEED_UNKNOWN) {
+        pr_err("Invalid usb maximum speed\n");
+        return -ENODEV;
+    }
+
+    priv->dr_mode = usb_get_dr_mode(node);
+    if (priv->dr_mode == USB_DR_MODE_UNKNOWN) {
+        pr_err("Invalid usb mode setup\n");
+        return -ENODEV;
+    }
+
+    return 0;
+}

Quoted 8 lines

drivers/usb/dwc3/dwc3-generic.c line 86

Quoted 4 lines

drivers/usb/dwc3/dwc3-generic.c new file +4 -0

+
+static int dwc3_generic_peripheral_bind(struct udevice *dev)
+{
+    return device_probe(dev);

This is wrong place to probe the device.
What must be done is to probe the device when it is first used. I have a patch in progress for this. When it's cleaned up I'll share a repo so that you can have look and maybe pick it up.

Well the series is already applied. I'll send the patch to ML in this case.

Hi,

this patchset is based on unfinished series send to ML 06/13/2017 by Vignesh
"[U-Boot] [PATCH v2 00/13] driver model bring-up of dwc3 usb peripheral" We have taken some part of this to Xilinx tree and use it for a while but it is time to review it and upstream it.

The patchset has 2 patches created by Mugunthan which are a little bit fixed.
dwc3-generic driver was inspired by omap version.
And the last 3 patches just doing conversion and enable DM_USB for our platform.