lkml.org 
[lkml]   [2020]   [Jan]   [16]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRe: [PATCH 4/4] dt-bindings: net: adin: document 1588 TX/RX SOP bindings
Date
On Thu, 2020-01-16 at 14:43 +0100, Andrew Lunn wrote:
> [External]
>
> > + adi,1588-rx-sop-delays-cycles:
> > + allOf:
> > + - $ref: /schemas/types.yaml#definitions/uint8-array
> > + - items:
> > + - minItems: 3
> > + maxItems: 3
> > + description: |
> > + Enables Start Packet detection (SOP) for received IEEE 1588 time
> > stamp
> > + controls, and configures the number of cycles (of the MII RX_CLK
> > clock)
> > + to delay the indication of RX SOP frames for 10/100/1000 BASE-T
> > links.
> > + The first element (in the array) configures the delay for
> > 10BASE-T,
> > + the second for 100BASE-T, and the third for 1000BASE-T.
>
> Do you know the clock frequency? It would be much better to express
> this in ns, as with adi,1588-tx-sop-delays-ns.

Yep.
We know the clock frequency here.
I'll take a look about converting this.

>
> > @@ -62,5 +116,11 @@ examples:
> > reg = <1>;
> >
> > adi,fifo-depth-bits = <16>;
> > +
> > + adi,1588-rx-sop-delays-cycles = [ 00 00 00 ];
> > + adi,1588-rx-sop-pin-name = "int_n";
> > +
> > + adi,1588-tx-sop-delays-ns = [ 00 08 10 ];
>
> 10 is not a multiple of 8!

My bad here.
I should point-out somewhere that these are hex-values.
It's kind of implied via the DT uint8 array type.

>
> Andrew
\
 
 \ /
  Last update: 2020-01-16 15:01    [W:0.069 / U:0.304 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site