On 06/13/2017 02:10 PM, Vignesh R wrote:
> From: Mugunthan V N <mugunthan...@ti.com>
> 
> Add a misc driver for DWC3 wrapper, so that based on dr_mode the
> USB devices can bind to USB host or USB device drivers.
> 
> Based on dr_mode property, the device node needs to bind to either
> "host" driver or "peripheral" driver. This wrapper will on bind
> read dr_mode property and appropriately associate USB DT node with DWC3
> peripheral driver or host driver.
> 
> This is similar to what exists today for MUSB and is instantiated by
> arch_misc_init() function for am33xx and am43xx.
> 
> Signed-off-by: Mugunthan V N <mugunthan...@ti.com>
> Signed-off-by: Vignesh R <vigne...@ti.com>
> ---
>  drivers/usb/dwc3/dwc3-omap.c | 49 
> ++++++++++++++++++++++++++++++++++++++++++++
>  drivers/usb/dwc3/gadget.c    |  2 +-
>  2 files changed, 50 insertions(+), 1 deletion(-)
> 
> diff --git a/drivers/usb/dwc3/dwc3-omap.c b/drivers/usb/dwc3/dwc3-omap.c
> index b4dde726c6f3..b7daf499515d 100644
> --- a/drivers/usb/dwc3/dwc3-omap.c
> +++ b/drivers/usb/dwc3/dwc3-omap.c
> @@ -628,4 +628,53 @@ U_BOOT_DRIVER(dwc3_omap_peripheral) = {
>       .priv_auto_alloc_size = sizeof(struct omap_dwc3_priv),
>       .flags  = DM_FLAG_ALLOC_PRIV_DMA,
>  };
> +
> +static int ti_dwc3_wrapper_bind(struct udevice *parent)
> +{
> +     ofnode node;
> +     int ret;
> +
> +     dev_for_each_subnode(node, parent) {
> +             const char *name = ofnode_get_name(node);
> +             enum usb_dr_mode dr_mode;
> +             struct udevice *dev;
> +
> +             if (strncmp(name, "usb@", 4))
> +                     continue;
> +
> +             dr_mode = usb_get_dr_mode(ofnode_to_offset(node));
> +             switch (dr_mode) {
> +             case USB_DR_MODE_PERIPHERAL:
> +             case USB_DR_MODE_OTG:
> +                     /* Bind MUSB device */
> +                     ret = device_bind_driver_to_node(parent,
> +                                                      "dwc3-omap-peripheral",
> +                                                      name, node, &dev);
> +                     if (ret) {
> +                             error("dwc3 - not able to bind usb device 
> node\n");
> +                             return ret;
> +                     }
> +                     break;
> +             case USB_DR_MODE_HOST:
> +                     /* Bind MUSB host */
> +                     break;
> +             default:
> +                     break;
> +             };
> +     }
> +     return 0;
> +}
> +
> +static const struct udevice_id ti_dwc3_ids[] = {
> +     { .compatible = "ti,am437x-dwc3" },
> +     { }
> +};
> +
> +U_BOOT_DRIVER(ti_dwc3_wrapper) = {
> +     .name   = "ti-dwc3-wrapper",
> +     .id     = UCLASS_MISC,
> +     .of_match = ti_dwc3_ids,
> +     .bind = ti_dwc3_wrapper_bind,
> +};
> +
>  #endif /* CONFIG_DM_USB */
> diff --git a/drivers/usb/dwc3/gadget.c b/drivers/usb/dwc3/gadget.c
> index e065c5aeb38d..18bbd5318e48 100644
> --- a/drivers/usb/dwc3/gadget.c
> +++ b/drivers/usb/dwc3/gadget.c
> @@ -2610,7 +2610,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);

This is probably unintended ?

>       if (ret) {
>               dev_err(dwc->dev, "failed to register udc\n");
>               goto err4;
> 


-- 
Best regards,
Marek Vasut
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Reply via email to