From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from na01-by2-obe.outbound.protection.outlook.com (mail-by2on0103.outbound.protection.outlook.com [207.46.100.103]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-SHA384 (256/256 bits)) (No client certificate requested) by lists.ozlabs.org (Postfix) with ESMTPS id 633D31A07C3 for ; Wed, 7 Jan 2015 11:28:54 +1100 (AEDT) Message-ID: <1420590520.4961.38.camel@freescale.com> Subject: Re: [PATCH] [v3] power/fsl: add MDIO dt binding for FMan From: Scott Wood To: Xie Shaohui-B21989 Date: Tue, 6 Jan 2015 18:28:40 -0600 In-Reply-To: References: <1419321466-5575-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" , Medve Emilian-EMMEDVE1 List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Mon, 2015-01-05 at 05:21 -0600, Xie Shaohui-B21989 wrote: > Hello Scott, > > Is this v3 OK? > > Best Regards, > Shaohui Xie > > -----Original Message----- > > From: shh.xie@gmail.com [mailto:shh.xie@gmail.com] > > Sent: Tuesday, December 23, 2014 3:58 PM > > To: linuxppc-dev@lists.ozlabs.org; devicetree@vger.kernel.org; Wood Scott-B07421 > > Cc: Medve Emilian-EMMEDVE1; Xie Shaohui-B21989 > > Subject: [PATCH] [v3] power/fsl: add MDIO dt binding for FMan > > > > From: Shaohui Xie > > > > This binding is for FMan MDIO, it covers FMan v2 & FMan v3. > > > > Signed-off-by: Shaohui Xie > > --- > > changes in v3: > > rephrase the 'Definition' of property 'bus-frequency', don't include the > > property in example. > > > > changes in v2: > > addressed comments from Scott in V1. > > > > .../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 edeea16..737aa4c 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 > > > > ============================================================================= > > @@ -357,6 +358,69 @@ 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: Specifies the external MDIO bus clock speed to > > + be used, if different from the standard 2.5 MHz. > > + This may be due to the standard speed being unsupported (e.g. > > + due to a hardware problem), or to advertise that all relevant > > + components in the system support a faster speed. > > + > > +- interrupts > > + Usage: optional > > + Value type: > > + Definition: Event interrupt of external MDIO controller. > > + 1 Gb/s MDIO and 10 Gb/s MDIO has one interrupt respectively. I'm confused by "respectively" here. Does fsl,fman-memac-mdio have two interrupts (one for 1 Gb/s and one for 10 Gb/s)? If there's only (at most) one interrupt per MDIO controller then just delete this line. What interrupt is used for internal MDIO? Does "optional" mean it's used if and only if external MDIO is used, or is it optional even with external MDIO? I see it's not present in the example -- do we not have a real example that has the interrupt? -Scott