Re: [PATCH net-next v4 3/4] dt-bindings: net: Add RGMII internal delay for DP83869

[Dan Murphy]

Rob

On 5/29/20 2:03 PM, Rob Herring wrote:
> On Wed, May 27, 2020 at 11:49:33AM -0500, Dan Murphy wrote:
> > Add the internal delay values into the header and update the binding
> > with the internal delay properties.
> >
> > Signed-off-by: Dan Murphy <dmurphy@xxxxxx>
> > ---
> >   .../devicetree/bindings/net/ti,dp83869.yaml      | 16 ++++++++++++++++
> >   1 file changed, 16 insertions(+)
> >
> > diff --git a/Documentation/devicetree/bindings/net/ti,dp83869.yaml b/Documentation/devicetree/bindings/net/ti,dp83869.yaml
> > index 5b69ef03bbf7..2971dd3fc039 100644
> > --- a/Documentation/devicetree/bindings/net/ti,dp83869.yaml
> > +++ b/Documentation/devicetree/bindings/net/ti,dp83869.yaml
> > @@ -64,6 +64,20 @@ properties:
> >          Operational mode for the PHY.  If this is not set then the operational
> >          mode is set by the straps. see dt-bindings/net/ti-dp83869.h for values
> >   
> > +  rx-internal-delay-ps:
> > +    $ref: "#/properties/rx-internal-delay-ps"
> This just creates a circular reference which probably blows up.

dt_binding_check did not have an issue with this.

But I will remove it

Dan