[U-Boot] [PATCH v3 0/8] Improvements for the dwc3_generic driver

Jean-Jacques Hiblot jjhiblot at ti.com
Mon Oct 22 16:24:02 UTC 2018



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.simek at xilinx.com>"
>>>>   
>>> Tested-by: Loic Devulder <ldevulder at 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: wd at denx.de



More information about the U-Boot mailing list