Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1413892 > unrolled thread

[PATCH] Documentation/devicetree: document cavium-pip rx-delay/tx-delay properties

Started byAaro Koskinen <aaro.koskinen@iki.fi>
First post2016-06-05 00:30 +0200
Last post2016-06-09 21:10 +0200
Articles 4 — 2 participants

Back to article view | Back to linux.kernel


Contents

  [PATCH] Documentation/devicetree: document cavium-pip rx-delay/tx-delay properties Aaro Koskinen <aaro.koskinen@iki.fi> - 2016-06-05 00:30 +0200
    Re: [PATCH] Documentation/devicetree: document cavium-pip  rx-delay/tx-delay properties Rob Herring <robh@kernel.org> - 2016-06-08 21:20 +0200
      Re: [PATCH] Documentation/devicetree: document cavium-pip  rx-delay/tx-delay properties Aaro Koskinen <aaro.koskinen@iki.fi> - 2016-06-08 22:00 +0200
        Re: [PATCH] Documentation/devicetree: document cavium-pip  rx-delay/tx-delay properties Rob Herring <robh@kernel.org> - 2016-06-09 21:10 +0200

#1413892 — [PATCH] Documentation/devicetree: document cavium-pip rx-delay/tx-delay properties

FromAaro Koskinen <aaro.koskinen@iki.fi>
Date2016-06-05 00:30 +0200
Subject[PATCH] Documentation/devicetree: document cavium-pip rx-delay/tx-delay properties
Message-ID<rGseS-6NZ-7@gated-at.bofh.it>
Document cavium-pip rx-delay/tx-delay properties. Currently the board
specific values need to be hardcoded in the platform code, which we
want to avoid when moving to DT-only booting.

Signed-off-by: Aaro Koskinen <aaro.koskinen@iki.fi>
---
 Documentation/devicetree/bindings/net/cavium-pip.txt | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/Documentation/devicetree/bindings/net/cavium-pip.txt b/Documentation/devicetree/bindings/net/cavium-pip.txt
index 7dbd158..edbf411 100644
--- a/Documentation/devicetree/bindings/net/cavium-pip.txt
+++ b/Documentation/devicetree/bindings/net/cavium-pip.txt
@@ -37,6 +37,10 @@ Properties for PIP port which is a child the PIP interface:
 
 - phy-handle: Optional, see ethernet.txt file in the same directory.
 
+- rx-delay: Delay value for RGMII receive clock. Optional. Disabled if 0.
+
+- tx-delay: Delay value for RGMII transmit clock. Optional. Disabled if 0.
+
 Example:
 
 	pip@11800a0000000 {
-- 
2.7.2

[toc] | [next] | [standalone]


#1417785 — Re: [PATCH] Documentation/devicetree: document cavium-pip rx-delay/tx-delay properties

FromRob Herring <robh@kernel.org>
Date2016-06-08 21:20 +0200
SubjectRe: [PATCH] Documentation/devicetree: document cavium-pip rx-delay/tx-delay properties
Message-ID<rHRbc-4cI-11@gated-at.bofh.it>
In reply to#1413892
On Sun, Jun 05, 2016 at 01:24:02AM +0300, Aaro Koskinen wrote:
> Document cavium-pip rx-delay/tx-delay properties. Currently the board
> specific values need to be hardcoded in the platform code, which we
> want to avoid when moving to DT-only booting.
> 
> Signed-off-by: Aaro Koskinen <aaro.koskinen@iki.fi>
> ---
>  Documentation/devicetree/bindings/net/cavium-pip.txt | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/net/cavium-pip.txt b/Documentation/devicetree/bindings/net/cavium-pip.txt
> index 7dbd158..edbf411 100644
> --- a/Documentation/devicetree/bindings/net/cavium-pip.txt
> +++ b/Documentation/devicetree/bindings/net/cavium-pip.txt
> @@ -37,6 +37,10 @@ Properties for PIP port which is a child the PIP interface:
>  
>  - phy-handle: Optional, see ethernet.txt file in the same directory.
>  
> +- rx-delay: Delay value for RGMII receive clock. Optional. Disabled if 0.
> +
> +- tx-delay: Delay value for RGMII transmit clock. Optional. Disabled if 0.

What are the units? Use standard unit suffix if in ns, us, etc.

> +
>  Example:
>  
>  	pip@11800a0000000 {
> -- 
> 2.7.2
> 
> --
> 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

[toc] | [prev] | [next] | [standalone]


#1417828 — Re: [PATCH] Documentation/devicetree: document cavium-pip rx-delay/tx-delay properties

FromAaro Koskinen <aaro.koskinen@iki.fi>
Date2016-06-08 22:00 +0200
SubjectRe: [PATCH] Documentation/devicetree: document cavium-pip rx-delay/tx-delay properties
Message-ID<rHRNU-4rG-25@gated-at.bofh.it>
In reply to#1417785
Hi,

On Wed, Jun 08, 2016 at 02:18:42PM -0500, Rob Herring wrote:
> On Sun, Jun 05, 2016 at 01:24:02AM +0300, Aaro Koskinen wrote:
> > Document cavium-pip rx-delay/tx-delay properties. Currently the board
> > specific values need to be hardcoded in the platform code, which we
> > want to avoid when moving to DT-only booting.
> > 
> > Signed-off-by: Aaro Koskinen <aaro.koskinen@iki.fi>
> > ---
> >  Documentation/devicetree/bindings/net/cavium-pip.txt | 4 ++++
> >  1 file changed, 4 insertions(+)
> > 
> > diff --git a/Documentation/devicetree/bindings/net/cavium-pip.txt b/Documentation/devicetree/bindings/net/cavium-pip.txt
> > index 7dbd158..edbf411 100644
> > --- a/Documentation/devicetree/bindings/net/cavium-pip.txt
> > +++ b/Documentation/devicetree/bindings/net/cavium-pip.txt
> > @@ -37,6 +37,10 @@ Properties for PIP port which is a child the PIP interface:
> >  
> >  - phy-handle: Optional, see ethernet.txt file in the same directory.
> >  
> > +- rx-delay: Delay value for RGMII receive clock. Optional. Disabled if 0.
> > +
> > +- tx-delay: Delay value for RGMII transmit clock. Optional. Disabled if 0.
> 
> What are the units? Use standard unit suffix if in ns, us, etc.

This is a raw register value, and the mapping to actual delay varies
depending on HW, AFAIK. 

A.

[toc] | [prev] | [next] | [standalone]


#1418595 — Re: [PATCH] Documentation/devicetree: document cavium-pip rx-delay/tx-delay properties

FromRob Herring <robh@kernel.org>
Date2016-06-09 21:10 +0200
SubjectRe: [PATCH] Documentation/devicetree: document cavium-pip rx-delay/tx-delay properties
Message-ID<rIdv3-27R-3@gated-at.bofh.it>
In reply to#1417828
On Wed, Jun 8, 2016 at 2:51 PM, Aaro Koskinen <aaro.koskinen@iki.fi> wrote:
> Hi,
>
> On Wed, Jun 08, 2016 at 02:18:42PM -0500, Rob Herring wrote:
>> On Sun, Jun 05, 2016 at 01:24:02AM +0300, Aaro Koskinen wrote:
>> > Document cavium-pip rx-delay/tx-delay properties. Currently the board
>> > specific values need to be hardcoded in the platform code, which we
>> > want to avoid when moving to DT-only booting.
>> >
>> > Signed-off-by: Aaro Koskinen <aaro.koskinen@iki.fi>
>> > ---
>> >  Documentation/devicetree/bindings/net/cavium-pip.txt | 4 ++++
>> >  1 file changed, 4 insertions(+)
>> >
>> > diff --git a/Documentation/devicetree/bindings/net/cavium-pip.txt b/Documentation/devicetree/bindings/net/cavium-pip.txt
>> > index 7dbd158..edbf411 100644
>> > --- a/Documentation/devicetree/bindings/net/cavium-pip.txt
>> > +++ b/Documentation/devicetree/bindings/net/cavium-pip.txt
>> > @@ -37,6 +37,10 @@ Properties for PIP port which is a child the PIP interface:
>> >
>> >  - phy-handle: Optional, see ethernet.txt file in the same directory.
>> >
>> > +- rx-delay: Delay value for RGMII receive clock. Optional. Disabled if 0.
>> > +
>> > +- tx-delay: Delay value for RGMII transmit clock. Optional. Disabled if 0.
>>
>> What are the units? Use standard unit suffix if in ns, us, etc.
>
> This is a raw register value, and the mapping to actual delay varies
> depending on HW, AFAIK.

Then state that in the description.

Rob

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web