[PATCH] powerpc/fsl: Update fman dt binding with pcs-phy and tbi-phy

[Liberman Igal]

Regards,
Igal Liberman

> -----Original Message-----
> From: Joakim Tjernlund [mailto:Joakim.Tjernlund at infinera.com]
> Sent: Tuesday, December 22, 2015 5:57 PM
> To: linuxppc-dev at lists.ozlabs.org; Liberman Igal-B31950
> <Igal.Liberman at freescale.com>; devicetree at vger.kernel.org
> Cc: Wood Scott-B07421 <scottwood at freescale.com>; Xie Shaohui-B21989
> <Shaohui.Xie at freescale.com>; Bucur Madalin-Cristian-B32716
> <madalin.bucur at freescale.com>
> Subject: Re: [PATCH] powerpc/fsl: Update fman dt binding with pcs-phy and
> tbi-phy
> 
> On Tue, 2015-12-22 at 06:18 +0200, igal.liberman at freescale.com wrote:
> > From: Igal Liberman <igal.liberman at freescale.com>
> >
> > Signed-off-by: Igal Liberman <igal.liberman at freescale.com>
> > ---
> >  .../devicetree/bindings/powerpc/fsl/fman.txt       |   39
> > ++++++++++++++++++++
> >  1 file changed, 39 insertions(+)
> >
> > diff --git a/Documentation/devicetree/bindings/powerpc/fsl/fman.txt
> > b/Documentation/devicetree/bindings/powerpc/fsl/fman.txt
> > index 1fc5328..7a6d7c3 100644
> > --- a/Documentation/devicetree/bindings/powerpc/fsl/fman.txt
> > +++ b/Documentation/devicetree/bindings/powerpc/fsl/fman.txt
> > @@ -315,6 +315,16 @@ PROPERTIES
> >  		Value type: <phandle>
> >  		Definition: A phandle for 1EEE1588 timer.
> >
> > +- pcsphy-handle
> > +		Usage required for "fsl,fman-memac" MACs
> > +		Value type: <phandle>
> > +		Definition: A phandle for pcsphy.
> > +
> > +- tbi-handle
> > +		Usage required for "fsl,fman-dtsec" MACs
> > +		Value type: <phandle>
> > +		Definition: A phandle for tbiphy.
> > +
> >  EXAMPLE
> >
> >  fman1_tx28: port at a8000 {
> > @@ -340,6 +350,7 @@ ethernet at e0000 {
> >  	reg = <0xe0000 0x1000>;
> >  	fsl,fman-ports = <&fman1_rx8 &fman1_tx28>;
> >  	ptp-timer = <&ptp-timer>;
> > +	tbi-handle = <&tbi0>;
> >  };
> >
> >
> >
> ==========================================================
> ============
> > ======
> > @@ -415,6 +426,13 @@ PROPERTIES
> >  		The settings and programming routines for internal/external
> >  		MDIO are different. Must be included for internal MDIO.
> >
> > +For internal PHY device on internal mdio bus, a PHY node should be
> created.
> > +See the definition of the PHY node in booting-without-of.txt for an
> > +example of how to define a PHY (Internal PHY has no interrupt line).
> > +- For "fsl,fman-mdio" compatible internal mdio bus, the PHY is TBI PHY.
> > +- For "fsl,fman-memac-mdio" compatible internal mdio bus, the PHY is
> > +PCS PHY,
> > +  PCS PHY addr must be '0'.
> 
> Will this replace the need for fixed PHYs ?
> 

Hi Jocke,

No, this is internal PCS configuration which required for both types of links.
The configuration for fixed links is different, but we use the same method.

Igal

>    Jocke