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