* Re: [PATCH 1/3] Documentation: dt: dwc3: Add snps,configure-fladj property
2015-07-23 10:09 [PATCH 1/3] Documentation: dt: dwc3: Add snps,configure-fladj property Nikhil Badola
@ 2015-07-23 9:56 ` Mark Rutland
2015-07-23 10:52 ` Badola Nikhil
0 siblings, 1 reply; 6+ messages in thread
From: Mark Rutland @ 2015-07-23 9:56 UTC (permalink / raw)
To: Nikhil Badola
Cc: linux-kernel@vger.kernel.org, devicetree@vger.kernel.org,
balbi@ti.com
On Thu, Jul 23, 2015 at 11:09:21AM +0100, Nikhil Badola wrote:
> Add property snps,configure-fladj for enabling post silicon
> frame length adjustment
>
> Signed-off-by: Nikhil Badola <nikhil.badola@freescale.com>
> ---
> Documentation/devicetree/bindings/usb/dwc3.txt | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/Documentation/devicetree/bindings/usb/dwc3.txt b/Documentation/devicetree/bindings/usb/dwc3.txt
> index 0815eac..90c3972 100644
> --- a/Documentation/devicetree/bindings/usb/dwc3.txt
> +++ b/Documentation/devicetree/bindings/usb/dwc3.txt
> @@ -40,6 +40,7 @@ Optional properties:
> - snps,hird-threshold: HIRD threshold
> - snps,hsphy_interface: High-Speed PHY interface selection between "utmi" for
> UTMI+ and "ulpi" for ULPI when the DWC_USB3_HSPHY_INTERFACE has value 3.
> + - snps,configure-fladj: enables post-silicon frame length adjustment
Could you elaborate on what this means and why you think it's necessary?
Mark.
>
> This is usually a subnode to DWC3 glue to which it is connected.
>
> --
> 2.1.0
>
> --
> To unsubscribe from this list: send the line "unsubscribe devicetree" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH 1/3] Documentation: dt: dwc3: Add snps,configure-fladj property
@ 2015-07-23 10:09 Nikhil Badola
2015-07-23 9:56 ` Mark Rutland
0 siblings, 1 reply; 6+ messages in thread
From: Nikhil Badola @ 2015-07-23 10:09 UTC (permalink / raw)
To: linux-kernel, devicetree; +Cc: balbi, Nikhil Badola
Add property snps,configure-fladj for enabling post silicon
frame length adjustment
Signed-off-by: Nikhil Badola <nikhil.badola@freescale.com>
---
Documentation/devicetree/bindings/usb/dwc3.txt | 1 +
1 file changed, 1 insertion(+)
diff --git a/Documentation/devicetree/bindings/usb/dwc3.txt b/Documentation/devicetree/bindings/usb/dwc3.txt
index 0815eac..90c3972 100644
--- a/Documentation/devicetree/bindings/usb/dwc3.txt
+++ b/Documentation/devicetree/bindings/usb/dwc3.txt
@@ -40,6 +40,7 @@ Optional properties:
- snps,hird-threshold: HIRD threshold
- snps,hsphy_interface: High-Speed PHY interface selection between "utmi" for
UTMI+ and "ulpi" for ULPI when the DWC_USB3_HSPHY_INTERFACE has value 3.
+ - snps,configure-fladj: enables post-silicon frame length adjustment
This is usually a subnode to DWC3 glue to which it is connected.
--
2.1.0
^ permalink raw reply related [flat|nested] 6+ messages in thread
* RE: [PATCH 1/3] Documentation: dt: dwc3: Add snps,configure-fladj property
2015-07-23 9:56 ` Mark Rutland
@ 2015-07-23 10:52 ` Badola Nikhil
2015-07-23 10:57 ` Mark Rutland
0 siblings, 1 reply; 6+ messages in thread
From: Badola Nikhil @ 2015-07-23 10:52 UTC (permalink / raw)
To: Mark Rutland
Cc: linux-kernel@vger.kernel.org, devicetree@vger.kernel.org,
balbi@ti.com
> -----Original Message-----
> From: Mark Rutland [mailto:mark.rutland@arm.com]
> Sent: Thursday, July 23, 2015 3:27 PM
> To: Badola Nikhil-B46172
> Cc: linux-kernel@vger.kernel.org; devicetree@vger.kernel.org; balbi@ti.com
> Subject: Re: [PATCH 1/3] Documentation: dt: dwc3: Add snps,configure-fladj
> property
>
> On Thu, Jul 23, 2015 at 11:09:21AM +0100, Nikhil Badola wrote:
> > Add property snps,configure-fladj for enabling post silicon frame
> > length adjustment
> >
> > Signed-off-by: Nikhil Badola <nikhil.badola@freescale.com>
> > ---
> > Documentation/devicetree/bindings/usb/dwc3.txt | 1 +
> > 1 file changed, 1 insertion(+)
> >
> > diff --git a/Documentation/devicetree/bindings/usb/dwc3.txt
> > b/Documentation/devicetree/bindings/usb/dwc3.txt
> > index 0815eac..90c3972 100644
> > --- a/Documentation/devicetree/bindings/usb/dwc3.txt
> > +++ b/Documentation/devicetree/bindings/usb/dwc3.txt
> > @@ -40,6 +40,7 @@ Optional properties:
> > - snps,hird-threshold: HIRD threshold
> > - snps,hsphy_interface: High-Speed PHY interface selection between
> "utmi" for
> > UTMI+ and "ulpi" for ULPI when the DWC_USB3_HSPHY_INTERFACE has
> value 3.
> > + - snps,configure-fladj: enables post-silicon frame length adjustment
>
> Could you elaborate on what this means and why you think it's necessary?
This property enables the use of GFLADJ_30MHZ field value of gfladj register for frame length
adjustment instead of considering from the sideband input signal fladj_30mhz_reg from SOC.
This is required when signal fladj_30mhz_reg is connected to a wrong value or is not valid as
in our case, hence post-silicon.
However this field can be used to adjust any offset ranging from 00h to 3Fh, from the
clock source generating SOF(start of frame) packets. Thus, this property can be added to device
tree with appropriate adjustment value.
>
> >
> > This is usually a subnode to DWC3 glue to which it is connected.
> >
> > --
> > 2.1.0
> >
> > --
> > To unsubscribe from this list: send the line "unsubscribe devicetree"
> > in the body of a message to majordomo@vger.kernel.org More
> majordomo
> > info at http://vger.kernel.org/majordomo-info.html
> >
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 1/3] Documentation: dt: dwc3: Add snps,configure-fladj property
2015-07-23 10:52 ` Badola Nikhil
@ 2015-07-23 10:57 ` Mark Rutland
2015-07-23 11:07 ` Badola Nikhil
0 siblings, 1 reply; 6+ messages in thread
From: Mark Rutland @ 2015-07-23 10:57 UTC (permalink / raw)
To: Badola Nikhil
Cc: linux-kernel@vger.kernel.org, devicetree@vger.kernel.org,
balbi@ti.com
On Thu, Jul 23, 2015 at 11:52:19AM +0100, Badola Nikhil wrote:
> > -----Original Message-----
> > From: Mark Rutland [mailto:mark.rutland@arm.com]
> > Sent: Thursday, July 23, 2015 3:27 PM
> > To: Badola Nikhil-B46172
> > Cc: linux-kernel@vger.kernel.org; devicetree@vger.kernel.org; balbi@ti.com
> > Subject: Re: [PATCH 1/3] Documentation: dt: dwc3: Add snps,configure-fladj
> > property
> >
> > On Thu, Jul 23, 2015 at 11:09:21AM +0100, Nikhil Badola wrote:
> > > Add property snps,configure-fladj for enabling post silicon frame
> > > length adjustment
> > >
> > > Signed-off-by: Nikhil Badola <nikhil.badola@freescale.com>
> > > ---
> > > Documentation/devicetree/bindings/usb/dwc3.txt | 1 +
> > > 1 file changed, 1 insertion(+)
> > >
> > > diff --git a/Documentation/devicetree/bindings/usb/dwc3.txt
> > > b/Documentation/devicetree/bindings/usb/dwc3.txt
> > > index 0815eac..90c3972 100644
> > > --- a/Documentation/devicetree/bindings/usb/dwc3.txt
> > > +++ b/Documentation/devicetree/bindings/usb/dwc3.txt
> > > @@ -40,6 +40,7 @@ Optional properties:
> > > - snps,hird-threshold: HIRD threshold
> > > - snps,hsphy_interface: High-Speed PHY interface selection between
> > "utmi" for
> > > UTMI+ and "ulpi" for ULPI when the DWC_USB3_HSPHY_INTERFACE has
> > value 3.
> > > + - snps,configure-fladj: enables post-silicon frame length adjustment
> >
> > Could you elaborate on what this means and why you think it's necessary?
>
> This property enables the use of GFLADJ_30MHZ field value of gfladj register for frame length
> adjustment instead of considering from the sideband input signal fladj_30mhz_reg from SOC.
> This is required when signal fladj_30mhz_reg is connected to a wrong value or is not valid as
> in our case, hence post-silicon.
Ok, so this is basically an override for the GFLADJ_30MHZ field of the
gfladj register when there was a problem at integration time.
> However this field can be used to adjust any offset ranging from 00h to 3Fh, from the
> clock source generating SOF(start of frame) packets. Thus, this property can be added to device
> tree with appropriate adjustment value.
It takes a value? The description above makes it sound like a boolean
property.
I'd expect a description more like:
- snps,fladj-override: Value for GFLADJ_30MHZ when the fladj_30mhz_reg
signal is invalid or incorrect.
Which makes it clear what the value is and when it should be set.
Thanks,
Mark.
^ permalink raw reply [flat|nested] 6+ messages in thread
* RE: [PATCH 1/3] Documentation: dt: dwc3: Add snps,configure-fladj property
2015-07-23 10:57 ` Mark Rutland
@ 2015-07-23 11:07 ` Badola Nikhil
2015-07-23 14:34 ` Felipe Balbi
0 siblings, 1 reply; 6+ messages in thread
From: Badola Nikhil @ 2015-07-23 11:07 UTC (permalink / raw)
To: Mark Rutland
Cc: linux-kernel@vger.kernel.org, devicetree@vger.kernel.org,
balbi@ti.com
> -----Original Message-----
> From: Mark Rutland [mailto:mark.rutland@arm.com]
> Sent: Thursday, July 23, 2015 4:27 PM
> To: Badola Nikhil-B46172
> Cc: linux-kernel@vger.kernel.org; devicetree@vger.kernel.org; balbi@ti.com
> Subject: Re: [PATCH 1/3] Documentation: dt: dwc3: Add snps,configure-fladj
> property
>
> On Thu, Jul 23, 2015 at 11:52:19AM +0100, Badola Nikhil wrote:
> > > -----Original Message-----
> > > From: Mark Rutland [mailto:mark.rutland@arm.com]
> > > Sent: Thursday, July 23, 2015 3:27 PM
> > > To: Badola Nikhil-B46172
> > > Cc: linux-kernel@vger.kernel.org; devicetree@vger.kernel.org;
> > > balbi@ti.com
> > > Subject: Re: [PATCH 1/3] Documentation: dt: dwc3: Add
> > > snps,configure-fladj property
> > >
> > > On Thu, Jul 23, 2015 at 11:09:21AM +0100, Nikhil Badola wrote:
> > > > Add property snps,configure-fladj for enabling post silicon frame
> > > > length adjustment
> > > >
> > > > Signed-off-by: Nikhil Badola <nikhil.badola@freescale.com>
> > > > ---
> > > > Documentation/devicetree/bindings/usb/dwc3.txt | 1 +
> > > > 1 file changed, 1 insertion(+)
> > > >
> > > > diff --git a/Documentation/devicetree/bindings/usb/dwc3.txt
> > > > b/Documentation/devicetree/bindings/usb/dwc3.txt
> > > > index 0815eac..90c3972 100644
> > > > --- a/Documentation/devicetree/bindings/usb/dwc3.txt
> > > > +++ b/Documentation/devicetree/bindings/usb/dwc3.txt
> > > > @@ -40,6 +40,7 @@ Optional properties:
> > > > - snps,hird-threshold: HIRD threshold
> > > > - snps,hsphy_interface: High-Speed PHY interface selection
> > > > between
> > > "utmi" for
> > > > UTMI+ and "ulpi" for ULPI when the DWC_USB3_HSPHY_INTERFACE
> > > > has
> > > value 3.
> > > > + - snps,configure-fladj: enables post-silicon frame length
> > > > + adjustment
> > >
> > > Could you elaborate on what this means and why you think it's
> necessary?
> >
> > This property enables the use of GFLADJ_30MHZ field value of gfladj
> > register for frame length adjustment instead of considering from the
> sideband input signal fladj_30mhz_reg from SOC.
> > This is required when signal fladj_30mhz_reg is connected to a wrong
> > value or is not valid as in our case, hence post-silicon.
>
> Ok, so this is basically an override for the GFLADJ_30MHZ field of the gfladj
> register when there was a problem at integration time.
>
That's right.
> > However this field can be used to adjust any offset ranging from 00h
> > to 3Fh, from the clock source generating SOF(start of frame) packets.
> > Thus, this property can be added to device tree with appropriate
> adjustment value.
>
> It takes a value? The description above makes it sound like a boolean
> property.
>
> I'd expect a description more like:
>
> - snps,fladj-override: Value for GFLADJ_30MHZ when the fladj_30mhz_reg
> signal is invalid or incorrect.
>
> Which makes it clear what the value is and when it should be set.
Agreed. Will change and send a new version of the patch-set.
>
> Thanks,
> Mark.
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 1/3] Documentation: dt: dwc3: Add snps,configure-fladj property
2015-07-23 11:07 ` Badola Nikhil
@ 2015-07-23 14:34 ` Felipe Balbi
0 siblings, 0 replies; 6+ messages in thread
From: Felipe Balbi @ 2015-07-23 14:34 UTC (permalink / raw)
To: Badola Nikhil
Cc: Mark Rutland, linux-kernel@vger.kernel.org,
devicetree@vger.kernel.org, balbi@ti.com
[-- Attachment #1: Type: text/plain, Size: 3401 bytes --]
On Thu, Jul 23, 2015 at 11:07:09AM +0000, Badola Nikhil wrote:
> > -----Original Message-----
> > From: Mark Rutland [mailto:mark.rutland@arm.com]
> > Sent: Thursday, July 23, 2015 4:27 PM
> > To: Badola Nikhil-B46172
> > Cc: linux-kernel@vger.kernel.org; devicetree@vger.kernel.org; balbi@ti.com
> > Subject: Re: [PATCH 1/3] Documentation: dt: dwc3: Add snps,configure-fladj
> > property
> >
> > On Thu, Jul 23, 2015 at 11:52:19AM +0100, Badola Nikhil wrote:
> > > > -----Original Message-----
> > > > From: Mark Rutland [mailto:mark.rutland@arm.com]
> > > > Sent: Thursday, July 23, 2015 3:27 PM
> > > > To: Badola Nikhil-B46172
> > > > Cc: linux-kernel@vger.kernel.org; devicetree@vger.kernel.org;
> > > > balbi@ti.com
> > > > Subject: Re: [PATCH 1/3] Documentation: dt: dwc3: Add
> > > > snps,configure-fladj property
> > > >
> > > > On Thu, Jul 23, 2015 at 11:09:21AM +0100, Nikhil Badola wrote:
> > > > > Add property snps,configure-fladj for enabling post silicon frame
> > > > > length adjustment
> > > > >
> > > > > Signed-off-by: Nikhil Badola <nikhil.badola@freescale.com>
> > > > > ---
> > > > > Documentation/devicetree/bindings/usb/dwc3.txt | 1 +
> > > > > 1 file changed, 1 insertion(+)
> > > > >
> > > > > diff --git a/Documentation/devicetree/bindings/usb/dwc3.txt
> > > > > b/Documentation/devicetree/bindings/usb/dwc3.txt
> > > > > index 0815eac..90c3972 100644
> > > > > --- a/Documentation/devicetree/bindings/usb/dwc3.txt
> > > > > +++ b/Documentation/devicetree/bindings/usb/dwc3.txt
> > > > > @@ -40,6 +40,7 @@ Optional properties:
> > > > > - snps,hird-threshold: HIRD threshold
> > > > > - snps,hsphy_interface: High-Speed PHY interface selection
> > > > > between
> > > > "utmi" for
> > > > > UTMI+ and "ulpi" for ULPI when the DWC_USB3_HSPHY_INTERFACE
> > > > > has
> > > > value 3.
> > > > > + - snps,configure-fladj: enables post-silicon frame length
> > > > > + adjustment
> > > >
> > > > Could you elaborate on what this means and why you think it's
> > necessary?
> > >
> > > This property enables the use of GFLADJ_30MHZ field value of gfladj
> > > register for frame length adjustment instead of considering from the
> > sideband input signal fladj_30mhz_reg from SOC.
> > > This is required when signal fladj_30mhz_reg is connected to a wrong
> > > value or is not valid as in our case, hence post-silicon.
> >
> > Ok, so this is basically an override for the GFLADJ_30MHZ field of the gfladj
> > register when there was a problem at integration time.
> >
>
> That's right.
>
> > > However this field can be used to adjust any offset ranging from 00h
> > > to 3Fh, from the clock source generating SOF(start of frame) packets.
> > > Thus, this property can be added to device tree with appropriate
> > adjustment value.
> >
> > It takes a value? The description above makes it sound like a boolean
> > property.
> >
> > I'd expect a description more like:
> >
> > - snps,fladj-override: Value for GFLADJ_30MHZ when the fladj_30mhz_reg
> > signal is invalid or incorrect.
> >
> > Which makes it clear what the value is and when it should be set.
>
> Agreed. Will change and send a new version of the patch-set.
BTW, I'm not going to accept this without a glue layer making use of it.
Also, this patch needs to go to linux-usb as well, please resend.
--
balbi
[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2015-07-23 14:35 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-07-23 10:09 [PATCH 1/3] Documentation: dt: dwc3: Add snps,configure-fladj property Nikhil Badola
2015-07-23 9:56 ` Mark Rutland
2015-07-23 10:52 ` Badola Nikhil
2015-07-23 10:57 ` Mark Rutland
2015-07-23 11:07 ` Badola Nikhil
2015-07-23 14:34 ` Felipe Balbi
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox