On 9/25/18 11:03 AM, Jean-Jacques Hiblot wrote: > > This series aims at bringing improvements to the dwc3_generic driver so > that it can be used by most of the platforms using the dwc3 controller. > > I tested this on with DRA7 and AM57x platforms for both Peripheral and Host > operations. The code to enable DM USB host & dev support for those > platforms will be submitted in a separate series. > > Michal Simek has tested this series: > " I have tested it on zcu100 with usb stick, usb to ethernet converter and > also dfu. > Tested-by: Michal Simek <michal.si...@xilinx.com>" >
Tested-by: Loic Devulder <ldevul...@suse.de> I tested this series of patches on Khadas VIM as well as Libretech-cc board and it resolved this issue ("USB0: "Synchronous Abort" handler, esr 0x96000210") I had on both cards (I think on all s905x Amlogic SBC)! > Enhancements: > - use separate Kconfig option for DM USB Periphal and DM USB Host. This > allow platforms to keep their non-DM USB peripheral code and use the DM > USB host. > - fixes the bind/probe confusion in dwc3_generic. The probe is done when > the USB device is first needed. > - handles PHYs when in the peripheral mode. The code to handle the PHYs is > shared with the host side > - handles clock and reset > - bind host controller to the more generic driver 'xhci-dwc3' > > > Changes in v3: > - fixes bug dwc3_setup_phy(): the phy arrays wasn't returned. This was > visible only when the device is removed. > - Stub the DWC3 PHY operations if CONFIG_IS_ENABLED(PHY) is false. > This fixes all build issues but one (evb-rk3328). > - Fix build issue with evb-rk3328 by enabling CONFIG_USB_DWC3. This has > little impact on the footprint and should not break the runtime as the > xhci-rockchip driver has its own probe function. > Nevertheless this was !!! NOT TESTED !!! by lack of hw > > Changes in v2: > - Updated commit log > - Fixed typo in thordown.c > - select DM_USB_DEV by default for zynqmp platforms > > Jean-Jacques Hiblot (8): > usb: gadget: Do not call board_usb_xxx() directly in USB gadget > drivers > usb: introduce a separate config option for DM USB device > usb: udc: implement DM versions of > usb_gadget_initialize()/_release()/_handle_interrupt() > dwc3_generic: do not probe the USB device driver when it's bound > dwc3: move phy operation to core.c > configs: evb-rk3328: Enable CONFIG_USB_DWC3 > dwc3-generic: Handle the PHYs, the clocks and the reset lines > dwc3-generic: Add select_dr_mode operation > > arch/arm/Kconfig | 2 + > cmd/fastboot.c | 4 +- > cmd/rockusb.c | 4 +- > cmd/thordown.c | 4 +- > cmd/usb_gadget_sdp.c | 4 +- > cmd/usb_mass_storage.c | 4 +- > common/dfu.c | 6 +- > configs/evb-rk3328_defconfig | 1 + > drivers/usb/Kconfig | 6 ++ > drivers/usb/dwc3/Kconfig | 7 +- > drivers/usb/dwc3/core.c | 89 +++++++++++++++- > drivers/usb/dwc3/dwc3-generic.c | 207 > +++++++++++++++++++++++++++++--------- > drivers/usb/dwc3/ep0.c | 1 - > drivers/usb/gadget/ether.c | 38 +------ > drivers/usb/gadget/udc/udc-core.c | 44 +++++++- > drivers/usb/host/xhci-dwc3.c | 95 ++--------------- > include/dwc3-uboot.h | 19 ++++ > include/linux/usb/gadget.h | 18 ++++ > 18 files changed, 367 insertions(+), 186 deletions(-) > -- Loic Devulder <ldevul...@suse.com> | ldevulder@irc 0x175A963893C85F55 | D220 DEF5 56A3 DE00 9DAA 78BA 175A 9638 93C8 5F55 Senior QA Engineer | Container & Storage Solutions Quality Assurance team (qa-css) SUSE LINUX GmbH, Maxfeldstr. 5, 90409 Nuernberg, Germany GF: F. Imendörffer, J. Smithard, J. Guild, D. Upmanyu, G. Norton HRB, 21284 (AG Nuernberg)
signature.asc
Description: OpenPGP digital signature
_______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot