On Tuesday, September 01, 2015 at 12:31:02 PM, Siva Durga Prasad Paladugu wrote: > Added USB XHCI driver support for zynqmp. > > Signed-off-by: Siva Durga Prasad Paladugu <siva...@xilinx.com>
Hi, looks almost good, a few minor nits though ... [...] > +unsigned long ctr_addr[] = {ZYNQMP_USB0_XHCI_BASEADDR, static const void __iomem *ctl_addr[] > + ZYNQMP_USB1_XHCI_BASEADDR}; I guess you can define something like CONFIG_ZYNQMP_XHCI_LIST { address ... } in your board config file and then use static const unsigned long ctl_addr[] = CONFIG_ZYNQMP... ; This will cover board which only use one controller. The ideal way would be to obtain these information from DT though. > +__weak int __board_usb_init(int index, enum usb_init_type init) > +{ > + return 0; > +} > + > +void usb_phy_reset(struct dwc3 *dwc3_reg) > +{ > + /* Assert USB3 PHY reset */ > + setbits_le32(&dwc3_reg->g_usb3pipectl[0], DWC3_GUSB3PIPECTL_PHYSOFTRST); > + > + /* Assert USB2 PHY reset */ > + setbits_le32(&dwc3_reg->g_usb2phycfg, DWC3_GUSB2PHYCFG_PHYSOFTRST); > + > + mdelay(200); That's some lazy crappy controller. Is this long delay needed ? > + /* Clear USB3 PHY reset */ > + clrbits_le32(&dwc3_reg->g_usb3pipectl[0], DWC3_GUSB3PIPECTL_PHYSOFTRST); > + > + /* Clear USB2 PHY reset */ > + clrbits_le32(&dwc3_reg->g_usb2phycfg, DWC3_GUSB2PHYCFG_PHYSOFTRST); > +} > + > +static int zynqmp_xhci_core_init(struct zynqmp_xhci *zynqmp_xhci) > +{ > + int ret = 0; > + > + ret = dwc3_core_init(zynqmp_xhci->dwc3_reg); > + if (ret) { > + debug("%s:failed to initialize core\n", __func__); > + return ret; > + } > + > + /* We are hard-coding DWC3 core to Host Mode */ > + dwc3_set_mode(zynqmp_xhci->dwc3_reg, DWC3_GCTL_PRTCAP_HOST); > + > + return ret; > +} > + > +int xhci_hcd_init(int index, struct xhci_hccr **hccr, struct xhci_hcor > **hcor) +{ > + struct zynqmp_xhci *ctx = &zynqmp_xhci; > + int ret = 0; > + > + ctx->hcd = (struct xhci_hccr *)ctr_addr[index]; > + ctx->dwc3_reg = (struct dwc3 *)((char *)(ctx->hcd) + DWC3_REG_OFFSET); > + > + ret = board_usb_init(index, USB_INIT_HOST); > + if (ret != 0) { > + puts("Failed to initialize board for USB\n"); > + return ret; > + } > + > + ret = zynqmp_xhci_core_init(ctx); > + if (ret < 0) { > + puts("Failed to initialize xhci\n"); > + return ret; > + } > + > + *hccr = (struct xhci_hccr *)ctx->hcd; > + *hcor = (struct xhci_hcor *)((uint32_t) *hccr > + + HC_LENGTH(xhci_readl(&(*hccr)->cr_capbase))); > + > + debug("zynqmp-xhci: init hccr %x and hcor %x hc_length %d\n", Use %p in the formating string to print pointers and drop the type casts. > + (uint32_t)*hccr, (uint32_t)*hcor, > + (uint32_t)HC_LENGTH(xhci_readl(&(*hccr)->cr_capbase))); > + > + return ret; > +} > + > +void xhci_hcd_stop(int index) > +{ > + /* > + * Currently zynqmp socs do not support PHY shutdown from > + * sw. But this support may be added in future socs. > + */ > + > + return 0; > +} _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot