From: Lukasz Majewski <lukma@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v2 0/7] [RESEND] Improvements for the dwc3_generic driver
Date: Fri, 21 Sep 2018 10:50:41 +0200 [thread overview]
Message-ID: <20180921105041.392f1d07@jawa> (raw)
In-Reply-To: <ba2bfb94-4726-88c2-0a22-e36ea662821e@ti.com>
Hi Jean-Jacques,
> Hi Marek,
>
> I haven't heard of you on this series.
>
> It is already a resend and has been tested on 2 platforms (zcu100 and
> dra7/am57).
>
> Can you please consider it for inclusion?
This is not Marek to blame for the delay - I had some non open source
tasks to finish (and I do apologise for this situation).
Your patches are now under build testing on travis-CI:
https://travis-ci.org/lmajewski/u-boot-dfu/builds/431388980
I will send the PR after CI finish.
>
> Thanks,
>
> JJ
>
>
>
> On 04/09/2018 15:42, Jean-Jacques Hiblot wrote:
> > Resending this series after rebasing on top of latest u-boot.
> >
> > 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@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 v2:
> > - Updated commit log
> > - Fixed typo in thordown.c
> > - select DM_USB_DEV by default for zynqmp platforms
> >
> > Jean-Jacques Hiblot (7):
> > 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
> > 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 +-
> > drivers/usb/Kconfig | 6 ++
> > drivers/usb/dwc3/Kconfig | 7 +-
> > drivers/usb/dwc3/core.c | 86 +++++++++++++++-
> > 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 | 93 ++---------------
> > include/dwc3-uboot.h | 7 ++
> > include/linux/usb/gadget.h | 18 ++++ 17 files changed, 351
> > insertions(+), 184 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
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 488 bytes
Desc: OpenPGP digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20180921/5e9adc94/attachment.sig>
next prev parent reply other threads:[~2018-09-21 8:50 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-09-04 13:42 [U-Boot] [PATCH v2 0/7] [RESEND] Improvements for the dwc3_generic driver Jean-Jacques Hiblot
2018-09-04 13:42 ` [U-Boot] [PATCH v2 1/7] usb: gadget: Do not call board_usb_xxx() directly in USB gadget drivers Jean-Jacques Hiblot
2018-09-04 13:42 ` [U-Boot] [PATCH v2 2/7] usb: introduce a separate config option for DM USB device Jean-Jacques Hiblot
2018-09-04 13:42 ` [U-Boot] [PATCH v2 3/7] usb: udc: implement DM versions of usb_gadget_initialize()/_release()/_handle_interrupt() Jean-Jacques Hiblot
2018-09-04 13:42 ` [U-Boot] [PATCH v2 4/7] dwc3_generic: do not probe the USB device driver when it's bound Jean-Jacques Hiblot
2018-09-04 13:42 ` [U-Boot] [PATCH v2 5/7] dwc3: move phy operation to core.c Jean-Jacques Hiblot
2018-09-04 13:42 ` [U-Boot] [PATCH v2 6/7] dwc3-generic: Handle the PHYs, the clocks and the reset lines Jean-Jacques Hiblot
2018-09-04 13:42 ` [U-Boot] [PATCH v2 7/7] dwc3-generic: Add select_dr_mode operation Jean-Jacques Hiblot
2018-09-14 8:07 ` [U-Boot] [PATCH v2 0/7] [RESEND] Improvements for the dwc3_generic driver Jean-Jacques Hiblot
2018-09-21 8:26 ` Jean-Jacques Hiblot
2018-09-21 8:50 ` Lukasz Majewski [this message]
2018-09-21 8:54 ` Jean-Jacques Hiblot
2018-09-21 9:24 ` Lukasz Majewski
2018-09-21 10:18 ` Jean-Jacques Hiblot
2018-09-21 10:43 ` Lukasz Majewski
2018-09-21 13:02 ` Jean-Jacques Hiblot
2018-09-24 15:38 ` Jean-Jacques Hiblot
2018-09-25 7:35 ` Lukasz Majewski
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20180921105041.392f1d07@jawa \
--to=lukma@denx.de \
--cc=u-boot@lists.denx.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox