[U-Boot] [PATCH 3/4] usb: xhci-dwc3: Add USB2 PHY configuration

Mark Kettenis mark.kettenis at xs4all.nl
Sun Jun 30 16:07:26 UTC 2019


> From: Marek Vasut <marex at denx.de>
> Date: Sun, 30 Jun 2019 16:31:41 +0200
> 
> On 6/30/19 4:25 PM, Mark Kettenis wrote:
> >> From: Marek Vasut <marex at denx.de>
> >> Date: Sun, 30 Jun 2019 15:36:03 +0200
> >>
> >> On 6/30/19 3:31 PM, Kever Yang wrote:
> >>> Hi Mark,
> >>>
> >>>     I add Marek and Bin Meng for this patch, I'm not sure where
> >>>
> >>> are these phy setting should go.
> >>>
> >>> Thanks,
> >>> - Kever
> >>> On 06/30/2019 12:29 AM, Mark Kettenis wrote:
> >>>> Configure USB2 PHY register based on "phy_type" property and
> >>>> handle all the quirks that are relevant for Rockchip RK3399 SoCs.
> >>
> >> Is this a documented DT property or some ad-hoc U-Boot one ?
> > 
> > Yes, these are all standard Linux DT properties.  See usb/generic.txt
> > for "phy_type", and usb/dwc3.txt for "snps,dis-enblslpm_quirk" and
> > "snps,dis-u2-freeclk-exists-quirk".
> > 
> >> Is this a fix for this release or feature for next ?
> > 
> > Good question.  The series fixes a regression, but that regression was
> > already present in 2019.04 and you can still boot from uSD, eMMC or
> > USB2 without this fixed.
> 
> Do we expect this to break anything ?

I don't.  The DT properties that are being checked are only present
for in the DTs for RK3399 boards, so this shouldn't affect any other
devices.

But while checking for this I noticed that I misspelled one of the
properties, which had a dash in the old U-Boot-specific binding but an
underscore in the official Linux binding.  So I sent out a v2 of the
series.

Cheers,

Mark


More information about the U-Boot mailing list