From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from na01-by2-obe.outbound.protection.outlook.com (mail-by2on0129.outbound.protection.outlook.com [207.46.100.129]) (using TLSv1 with cipher ECDHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by lists.ozlabs.org (Postfix) with ESMTPS id 5260E1A0079 for ; Tue, 11 Nov 2014 11:23:16 +1100 (AEDT) Message-ID: <1415665384.15957.16.camel@freescale.com> Subject: Re: [PATCH] DT: add MDIO node for FMan node From: Scott Wood To: Date: Mon, 10 Nov 2014 18:23:04 -0600 In-Reply-To: <1415102196-9238-1-git-send-email-shh.xie@gmail.com> References: <1415102196-9238-1-git-send-email-shh.xie@gmail.com> Content-Type: text/plain; charset="UTF-8" MIME-Version: 1.0 Cc: devicetree@vger.kernel.org, linuxppc-dev@lists.ozlabs.org, Shaohui Xie , Emilian.Medve@freescale.com List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Tue, 2014-11-04 at 19:56 +0800, shh.xie@gmail.com wrote: > From: Shaohui Xie > > This binding is for FMan MDIO, it covers FMan v2 & FMan v3. > > Signed-off-by: Shaohui Xie > --- > based on http://patchwork.ozlabs.org/patch/390351/ > for 'next' of git://git.kernel.org/pub/scm/linux/kernel/git/scottwood/linux.git Are there any other FMan pieces that are missing from the above patch? > .../devicetree/bindings/powerpc/fsl/fman.txt | 69 ++++++++++++++++++++++ > 1 file changed, 69 insertions(+) > > diff --git a/Documentation/devicetree/bindings/powerpc/fsl/fman.txt b/Documentation/devicetree/bindings/powerpc/fsl/fman.txt > index da8e5f2..83c2f14 100644 > --- a/Documentation/devicetree/bindings/powerpc/fsl/fman.txt > +++ b/Documentation/devicetree/bindings/powerpc/fsl/fman.txt > @@ -7,6 +7,7 @@ CONTENTS > - FMan MURAM Node > - FMan dTSEC/XGEC/mEMAC Node > - FMan IEEE 1588 Node > + - FMan MDIO Node > - Example > > ============================================================================= > @@ -352,6 +353,67 @@ ptp-timer@fe000 { > }; > > ============================================================================= > +FMan MDIO Node > + > +DESCRIPTION > + > +The MDIO is a bus to which the PHY devices are connected. > + > +PROPERTIES > + > +- compatible > + Usage: required > + Value type: > + Definition: A standard property. > + Must include "fsl,fman-mdio" for 1 Gb/s MDIO from FMan v2. > + Must include "fsl,fman-xmdio" for 10 Gb/s MDIO from FMan v2. > + Must include "fsl,fman-memac-mdio" for 1/10 Gb/s MDIO from > + FMan v3. > + > +- reg > + Usage: required > + Value type: > + Definition: A standard property. > + > +- bus-frequency > + Usage: optional > + Value type: > + Definition: Default MDIO bus clock speed. Use clocks/clock-names > +- interrupts > + Usage: optional > + Value type: > + Definition: MDIO controller event interrupts. One interrupt or multiple? > + > +- type > + Usage: required for FMan v3 > + Value type: > + Definition: A standard property. What standard is "type" defined in? > + FMan v3 has internal MDIO for internal PCS(Physical Coding > + Sublayer) PHYs and external MDIO for external PHYs. > + The settings and programming routines for internal/external > + MDIO are different. Must include "internal" for internal MDIO, > + must include "external" for external MDIO. I assume fman v2 is always internal? How about a boolean "fsl,fman-internal-phy" property instead? > + > +EXAMPLE > + > +Example for FMan v2: > + > +mdio@f1000 { > + compatible = "fsl,fman-xmdio"; > + reg = <0xf1000 0x1000>; > +}; > + > +Example for FMan v3: > + > +mdio@fd000 { > + compatible = "fsl,fman-memac-mdio"; > + reg = <0xfd000 0x1000>; > + bus-frequency = <2500000>; > + type = "external"; > +}; > + > +============================================================================= > Example > > fman@400000 { > @@ -526,4 +588,11 @@ fman@400000 { > compatible = "fsl,fman-ptp-timer"; > reg = <0xfe000 0x1000>; > }; > + > + mdio@fd000 { > + compatible = "fsl,fman-memac-mdio"; > + reg = <0xfd000 0x1000>; > + bus-frequency = <2500000>; > + type = "external"; > + }; > }; The rest of the example is fman v2; don't mix an fman v3 node in with that. -Scott