diff mbox series

[V1,net-next,2/5] dt-bindings: fec: add RGMII delayed clock property

Message ID 20210709081823.18696-3-qiangqing.zhang@nxp.com
State Superseded
Headers show
Series net: fec: add support for i.MX8MQ and i.MX8QM | expand

Commit Message

Joakim Zhang July 9, 2021, 8:18 a.m. UTC
From: Fugang Duan <fugang.duan@nxp.com>

Add property for RGMII delayed clock.

Signed-off-by: Fugang Duan <fugang.duan@nxp.com>
Signed-off-by: Joakim Zhang <qiangqing.zhang@nxp.com>
---
 Documentation/devicetree/bindings/net/fsl-fec.txt | 4 ++++
 1 file changed, 4 insertions(+)

Comments

Rob Herring (Arm) July 14, 2021, 11:19 p.m. UTC | #1
On Fri, Jul 09, 2021 at 04:18:20PM +0800, Joakim Zhang wrote:
> From: Fugang Duan <fugang.duan@nxp.com>
> 
> Add property for RGMII delayed clock.
> 
> Signed-off-by: Fugang Duan <fugang.duan@nxp.com>
> Signed-off-by: Joakim Zhang <qiangqing.zhang@nxp.com>
> ---
>  Documentation/devicetree/bindings/net/fsl-fec.txt | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/net/fsl-fec.txt b/Documentation/devicetree/bindings/net/fsl-fec.txt
> index 6754be1b91c4..f93b9552cfc5 100644
> --- a/Documentation/devicetree/bindings/net/fsl-fec.txt
> +++ b/Documentation/devicetree/bindings/net/fsl-fec.txt
> @@ -50,6 +50,10 @@ Optional properties:
>      SOC internal PLL.
>    - "enet_out"(option), output clock for external device, like supply clock
>      for PHY. The clock is required if PHY clock source from SOC.
> +  - "enet_2x_txclk"(option), for RGMII sampleing clock which fixed at 250Mhz.
> +    The clock is required if SOC RGMII enable clock delay.
> +- fsl,rgmii_txc_dly: add RGMII TXC delayed clock from MAC.
> +- fsl,rgmii_rxc_dly: add RGMII RXC delayed clock from MAC.

Don't we have standard properties for this?

>  
>  Optional subnodes:
>  - mdio : specifies the mdio bus in the FEC, used as a container for phy nodes
> -- 
> 2.17.1
> 
>
Andrew Lunn July 14, 2021, 11:34 p.m. UTC | #2
On Wed, Jul 14, 2021 at 05:19:37PM -0600, Rob Herring wrote:
> On Fri, Jul 09, 2021 at 04:18:20PM +0800, Joakim Zhang wrote:

> > From: Fugang Duan <fugang.duan@nxp.com>

> > 

> > Add property for RGMII delayed clock.

> > 

> > Signed-off-by: Fugang Duan <fugang.duan@nxp.com>

> > Signed-off-by: Joakim Zhang <qiangqing.zhang@nxp.com>

> > ---

> >  Documentation/devicetree/bindings/net/fsl-fec.txt | 4 ++++

> >  1 file changed, 4 insertions(+)

> > 

> > diff --git a/Documentation/devicetree/bindings/net/fsl-fec.txt b/Documentation/devicetree/bindings/net/fsl-fec.txt

> > index 6754be1b91c4..f93b9552cfc5 100644

> > --- a/Documentation/devicetree/bindings/net/fsl-fec.txt

> > +++ b/Documentation/devicetree/bindings/net/fsl-fec.txt

> > @@ -50,6 +50,10 @@ Optional properties:

> >      SOC internal PLL.

> >    - "enet_out"(option), output clock for external device, like supply clock

> >      for PHY. The clock is required if PHY clock source from SOC.

> > +  - "enet_2x_txclk"(option), for RGMII sampleing clock which fixed at 250Mhz.

> > +    The clock is required if SOC RGMII enable clock delay.

> > +- fsl,rgmii_txc_dly: add RGMII TXC delayed clock from MAC.

> > +- fsl,rgmii_rxc_dly: add RGMII RXC delayed clock from MAC.

> 

> Don't we have standard properties for this?


Yes, rx-internal-delay-ps and tx-internal-delay-ps defined in
ethernet-controller.yaml

     Andrew
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/net/fsl-fec.txt b/Documentation/devicetree/bindings/net/fsl-fec.txt
index 6754be1b91c4..f93b9552cfc5 100644
--- a/Documentation/devicetree/bindings/net/fsl-fec.txt
+++ b/Documentation/devicetree/bindings/net/fsl-fec.txt
@@ -50,6 +50,10 @@  Optional properties:
     SOC internal PLL.
   - "enet_out"(option), output clock for external device, like supply clock
     for PHY. The clock is required if PHY clock source from SOC.
+  - "enet_2x_txclk"(option), for RGMII sampleing clock which fixed at 250Mhz.
+    The clock is required if SOC RGMII enable clock delay.
+- fsl,rgmii_txc_dly: add RGMII TXC delayed clock from MAC.
+- fsl,rgmii_rxc_dly: add RGMII RXC delayed clock from MAC.
 
 Optional subnodes:
 - mdio : specifies the mdio bus in the FEC, used as a container for phy nodes