[U-Boot] [PATCH v3 0/8] Improvements for the dwc3_generic driver
Kever Yang
kever.yang at rock-chips.com
Mon Oct 8 02:07:20 UTC 2018
Hi Philipp,
On 10/01/2018 09:26 PM, Philipp Tomsich wrote:
> Kever,
>
> Could you test this?
> I assume you have the board, as the evb-rk3328 has you listed as a
> maintainer...
Sure, I can do it.
Sorry for reply late, I just come back to work from Chinese National Day
holiday.
Thanks,
- Kever
>
> Thanks,
> Philipp.
>
>> On 01.10.2018, at 14:51, Lukasz Majewski <lukma at denx.de
>> <mailto:lukma at denx.de>> wrote:
>>
>> On Tue, 25 Sep 2018 13:28:14 +0200
>> Lukasz Majewski <lukma at denx.de <mailto:lukma at denx.de>> wrote:
>>
>>> Dear All,
>>>
>>>> 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
>>>> <mailto:michal.simek at xilinx.com>>"
>>>>
>>>> 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
>>>
>>> Can somebody test this patch set on rk3328 and provide Tested-by tag?
>>
>> Anyone could test this patch series?
>>
>> Thanks in advance :-)
>>
>>>
>>>>
>>>> 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 <mailto:wd at denx.de>
>>
>>
>>
>>
>> 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
>> <mailto:wd at denx.de>
>
More information about the U-Boot
mailing list