Re: [PATCH v5 2/5] usb: dwc3: add dis_u2_freeclk_exists_quirk[Involving remittance information, please pay attention to the safety of property]

[William Wu]

Dear Rob,

On 07/01/2016 10:32 AM, Rob Herring wrote:
> On Thu, Jun 30, 2016 at 07:12:53PM +0800, William Wu wrote:
> > Add a quirk to clear the GUSB2PHYCFG.U2_FREECLK_EXISTS bit,
> > which specifies whether the USB2.0 PHY provides a free-running
> > PHY clock, which is active when the clock control input is active.
> >
> > Signed-off-by: William Wu <william.wu@xxxxxxxxxxxxxx>
> > ---
> > Changes in v5:
> > - None
> >
> > Changes in v4:
> > - rebase on top of balbi testing/next, remove pdata (balbi)
> >
> > Changes in v3:
> > - None
> >
> > Changes in v2:
> > - None
> >
> >   Documentation/devicetree/bindings/usb/dwc3.txt | 3 +++
> >   drivers/usb/dwc3/core.c                        | 5 +++++
> >   drivers/usb/dwc3/core.h                        | 5 +++++
> >   3 files changed, 13 insertions(+)
> >
> > diff --git a/Documentation/devicetree/bindings/usb/dwc3.txt b/Documentation/devicetree/bindings/usb/dwc3.txt
> > index 7d7ce08..1ada121 100644
> > --- a/Documentation/devicetree/bindings/usb/dwc3.txt
> > +++ b/Documentation/devicetree/bindings/usb/dwc3.txt
> > @@ -39,6 +39,9 @@ Optional properties:
> >   			disabling the suspend signal to the PHY.
> >    - snps,dis_rxdet_inp3_quirk: when set core will disable receiver detection
> >   			in PHY P3 power state.
> > + - snps,dis_u2_freeclk_exists_quirk: when set, clear the u2_freeclk_exists
> Use '-', not '_'.
OK, I'll fix them in next patch.
Thanks very much for your help.
> > +			in GUSB2PHYCFG, specify that USB2 PHY doesn't provide
> > +			a free-running PHY clock.
> >    - snps,is-utmi-l1-suspend: true when DWC3 asserts output signal
> >   			utmi_l1_suspend_n, false when asserts utmi_sleep_n
> >    - snps,hird-threshold: HIRD threshold