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