Dear All, > Andy, Kerver, Lin, > > could you give a try to this series on evb-rk3328 please ? This has > been blocked for a long time already.
Yes, agree. Could somebody (who has this SoC) test this patch set?
Thanks in advance,
Łukasz
>
> Thanks,
>
> JJ
>
> On 22/10/2018 18:24, Jean-Jacques Hiblot wrote:
> >
> >
> > On 22/10/2018 18:16, Lukasz Majewski wrote:
> >> Hi Jean-Jacques,
> >>
> >>> Hi Lukasz,
> >>>
> >>> gentle ping on the status on this series.
> >> Do we have tested by from somebody from RockChip community?
> > I haven't seen anything on the ML. I pinged Kever Yang earlier
> > today about it.
> >
> > JJ
> >>
> >>> JJ
> >>>
> >>>
> >>> On 09/10/2018 16:29, Loic Devulder wrote:
> >>>> 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(-)
> >>
> >>
> >>
> >> Best regards,
> >>
> >> Lukasz Majewski
> >>
> >> --
> >>
> >> DENX Software Engineering GmbH, Managing Director: Wolfgang
> >> Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell,
> >> Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email:
> >> w...@denx.de
> >
> > _______________________________________________
> > U-Boot mailing list
> > U-Boot@lists.denx.de
> > https://lists.denx.de/listinfo/u-boot
Best regards,
Lukasz Majewski
--
DENX Software Engineering GmbH, Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de
pgpCT3YerGpPJ.pgp
Description: OpenPGP digital signature
_______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
