From mboxrd@z Thu Jan 1 00:00:00 1970 From: Rob Herring Subject: Re: [PATCH v1] arm64: dts: dwc3: Add description of 'configure-gfladj' Date: Tue, 25 Sep 2018 11:48:39 -0500 Message-ID: <20180925164838.GA25852@bogus> References: <20180829073324.36866-1-yinbo.zhu@nxp.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <20180829073324.36866-1-yinbo.zhu@nxp.com> Sender: linux-kernel-owner@vger.kernel.org To: Yinbo Zhu Cc: Shawn Guo , Mark Rutland , xiaobo.xie@nxp.com, Ran Wang , Greg Kroah-Hartman , linux-usb@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, festevam@gmail.com List-Id: devicetree@vger.kernel.org On Wed, Aug 29, 2018 at 03:33:24PM +0800, Yinbo Zhu wrote: > This patch is to add description of 'configure-gfladj' to binding > so that configuring devicetree Not a complete sentence. The subject should be: "dt-bindings: usb: ..." > > Signed-off-by: Yinbo Zhu > --- > Documentation/devicetree/bindings/usb/dwc3.txt | 1 + > 1 files changed, 1 insertions(+), 0 deletions(-) > > diff --git a/Documentation/devicetree/bindings/usb/dwc3.txt b/Documentation/devicetree/bindings/usb/dwc3.txt > index 3e4c38b..40c6568 100644 > --- a/Documentation/devicetree/bindings/usb/dwc3.txt > +++ b/Documentation/devicetree/bindings/usb/dwc3.txt > @@ -101,6 +101,7 @@ Optional properties: > When just one value, which means INCRX burst mode enabled. When > more than one value, which means undefined length INCR burst type > enabled. The values can be 1, 4, 8, 16, 32, 64, 128 and 256. > + - configure-gfladj: determine whether frame length adjustment is required or not. Needs a vendor prefix. What's the type? bool? What determines if this is needed. If fixed for an SoC or IP version, then the compatible string should imply this. Rob