From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753258AbcDSX00 (ORCPT ); Tue, 19 Apr 2016 19:26:26 -0400 Received: from mail-am1on0074.outbound.protection.outlook.com ([157.56.112.74]:57856 "EHLO emea01-am1-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1753117AbcDSX0Y convert rfc822-to-8bit (ORCPT ); Tue, 19 Apr 2016 19:26:24 -0400 X-Greylist: delayed 5667 seconds by postgrey-1.27 at vger.kernel.org; Tue, 19 Apr 2016 19:26:23 EDT From: Stuart Yoder To: Shawn Guo CC: "devicetree@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , "german.rivera@freescale.com" , "linux-kernel@vger.kernel.org" , "robh+dt@kernel.org" , Yang-Leo Li Subject: RE: [PATCH 2/2 v2] arm64: dts: ls2080a: fsl-mc dt node updates Thread-Topic: [PATCH 2/2 v2] arm64: dts: ls2080a: fsl-mc dt node updates Thread-Index: AQHRlcg4Tba0T60Z20+tWQTWsco3N5+PEDEAgALOWxA= Date: Tue, 19 Apr 2016 21:51:51 +0000 Message-ID: References: <1460581523-32146-1-git-send-email-stuart.yoder@nxp.com> <20160418025618.GU28903@tiger> In-Reply-To: <20160418025618.GU28903@tiger> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: authentication-results: kernel.org; dkim=none (message not signed) header.d=none;kernel.org; dmarc=none action=none header.from=nxp.com; x-originating-ip: [192.88.168.50] x-ms-office365-filtering-correlation-id: 28695638-70b6-45c8-128f-08d3689cd10b x-microsoft-exchange-diagnostics: 1;DB4PR04MB0781;5:vnQkrwX52gv9hsLLFSUOenuDL5SchKfyvT2vvI5ZuAe1sNEpOU3xpo5VOQIk/ZZOOZ9IPE1jSfGycwVlfbYxWn2xoNtQEtUd2zJ+9dOvODYzKkG65mwIMe67YDgx0jJ6RQcSWbsecclPgvdoCdqKslqtBZ/mctTCBq/2Qd40HnVVOxijHTV5ycil/8fZtXi/;24:1Fep8E5Tj/HbNY42k2v+P00NxQZJ6G5/IMWueMuAuF1ETiLahMW9FItO8HVbFD8GLxRLMy23VI/rTm1jgDa5MdK+vFTv3al7px3JmJNVcCM=;7:305pYnpGkI7J/8eTuSfv2iPJ4L0q0xx+Zz8HpXgOhoAeQb+xSIBj+39WxvGh0RfefCjwTo8Kls1FYxhitP4JME/4w+EK+JhpxNakKC+eCtO/LNiDahaUk58BbB2jYuyQFp0GsWUK+HLz1ElwSpr6zTLKRgJJGoOYItx87kt/bPS27HvLlOCkHdQxXQYsTtbUtAz0Gf3/psyGjvVsIBXc8eRw4jH+9lZDUEc1adEVeT8= x-microsoft-antispam: UriScan:;BCL:0;PCL:0;RULEID:;SRVR:DB4PR04MB0781; x-microsoft-antispam-prvs: x-exchange-antispam-report-test: UriScan:; x-exchange-antispam-report-cfa-test: BCL:0;PCL:0;RULEID:(9101521026)(601004)(2401047)(8121501046)(5005006)(3002001)(10201501046)(6055026);SRVR:DB4PR04MB0781;BCL:0;PCL:0;RULEID:;SRVR:DB4PR04MB0781; x-forefront-prvs: 0917DFAC67 x-forefront-antispam-report: SFV:NSPM;SFS:(10009020)(6009001)(377454003)(13464003)(24454002)(575784001)(5008740100001)(2906002)(2950100001)(2900100001)(86362001)(1096002)(1220700001)(6116002)(102836003)(3846002)(54356999)(50986999)(5003600100002)(77096005)(76176999)(10400500002)(33656002)(11100500001)(586003)(19580405001)(106116001)(87936001)(92566002)(122556002)(19580395003)(5002640100001)(66066001)(189998001)(110136002)(3280700002)(15650500001)(9686002)(74316001)(81166005)(3660700001)(4326007)(76576001)(7059030);DIR:OUT;SFP:1101;SCL:1;SRVR:DB4PR04MB0781;H:HE1PR04MB1641.eurprd04.prod.outlook.com;FPR:;SPF:None;MLV:sfv;LANG:en; spamdiagnosticoutput: 1:23 spamdiagnosticmetadata: NSPM Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 8BIT MIME-Version: 1.0 X-OriginatorOrg: nxp.com X-MS-Exchange-CrossTenant-originalarrivaltime: 19 Apr 2016 21:51:51.8847 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635 X-MS-Exchange-Transport-CrossTenantHeadersStamped: DB4PR04MB0781 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org > -----Original Message----- > From: Shawn Guo [mailto:shawnguo@kernel.org] > Sent: Sunday, April 17, 2016 9:56 PM > To: Stuart Yoder > Cc: devicetree@vger.kernel.org; linux-arm-kernel@lists.infradead.org; german.rivera@freescale.com; > linux-kernel@vger.kernel.org; robh+dt@kernel.org; Yang-Leo Li > Subject: Re: [PATCH 2/2 v2] arm64: dts: ls2080a: fsl-mc dt node updates > > On Wed, Apr 13, 2016 at 04:05:23PM -0500, Stuart Yoder wrote: > > updates to the fsl-mc node for full functionality: > > -msi-parent is needed for interrupt support > > -ranges is needed to enable the bus driver to translate bus addresses > > -dpmac nodes provide a basis for relating dpmac objects to PHYs > > > > Signed-off-by: Stuart Yoder > > --- > > > > -v2: updated comment to remove some extraneous info, added > > carriage returns after dpmac nodes > > > > arch/arm64/boot/dts/freescale/fsl-ls2080a.dtsi | 98 ++++++++++++++++++++++++++ > > 1 file changed, 98 insertions(+) > > > > diff --git a/arch/arm64/boot/dts/freescale/fsl-ls2080a.dtsi b/arch/arm64/boot/dts/freescale/fsl- > ls2080a.dtsi > > index 9d746c6..15498eb 100644 > > --- a/arch/arm64/boot/dts/freescale/fsl-ls2080a.dtsi > > +++ b/arch/arm64/boot/dts/freescale/fsl-ls2080a.dtsi > > @@ -265,6 +265,104 @@ > > compatible = "fsl,qoriq-mc"; > > reg = <0x00000008 0x0c000000 0 0x40>, /* MC portal base */ > > <0x00000000 0x08340000 0 0x40000>; /* MC control reg */ > > + msi-parent = <&its>; > > + #address-cells = <3>; > > + #size-cells = <1>; > > + > > + /* > > + * Region type 0x0 - MC portals > > + * Region type 0x1 - QBMAN portals > > + */ > > + ranges = <0x0 0x0 0x0 0x8 0x0c000000 0x4000000 > > + 0x1 0x0 0x0 0x8 0x18000000 0x8000000>; > > + > > + /* > > + * Define the maximum number of MACs present on the SoC. > > + */ > > + dpmacs { > > + #address-cells = <1>; > > + #size-cells = <0>; > > + > > + dpmac1: dpmac@1 { > > + compatible = "fsl,qoriq-mc-dpmac"; > > + reg = <1>; > > + }; > > + > > + dpmac2: dpmac@2 { > > + compatible = "fsl,qoriq-mc-dpmac"; > > + reg = <2>; > > + }; > > + > > + dpmac3: dpmac@3 { > > + compatible = "fsl,qoriq-mc-dpmac"; > > + reg = <3>; > > + }; > > + > > + dpmac4: dpmac@4 { > > + compatible = "fsl,qoriq-mc-dpmac"; > > + reg = <4>; > > + }; > > + > > + dpmac5: dpmac@5 { > > + compatible = "fsl,qoriq-mc-dpmac"; > > + reg = <5>; > > + }; > > + > > + dpmac6: dpmac@6 { > > + compatible = "fsl,qoriq-mc-dpmac"; > > + reg = <6>; > > + }; > > + > > + dpmac7: dpmac@7 { > > + compatible = "fsl,qoriq-mc-dpmac"; > > + reg = <7>; > > + }; > > + > > + dpmac8: dpmac@8 { > > + compatible = "fsl,qoriq-mc-dpmac"; > > + reg = <8>; > > + }; > > + > > + dpmac9: dpmac@9 { > > + compatible = "fsl,qoriq-mc-dpmac"; > > + reg = <9>; > > + }; > > + > > + dpmac10: dpmac@10 { > > The unit-address should be same as reg property, i.e. both are in hex > value. Ok, will update. I am going to leave the labels in decimal because that is how we refer to these mac devices in normal usage and in our documentation. ls2080a users expect dpmac10 to be the 10th mac not the 16th. Thanks, Stuart From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stuart Yoder Subject: RE: [PATCH 2/2 v2] arm64: dts: ls2080a: fsl-mc dt node updates Date: Tue, 19 Apr 2016 21:51:51 +0000 Message-ID: References: <1460581523-32146-1-git-send-email-stuart.yoder@nxp.com> <20160418025618.GU28903@tiger> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20160418025618.GU28903@tiger> Content-Language: en-US List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=m.gmane.org@lists.infradead.org To: Shawn Guo Cc: "devicetree@vger.kernel.org" , "german.rivera@freescale.com" , "linux-kernel@vger.kernel.org" , Yang-Leo Li , "robh+dt@kernel.org" , "linux-arm-kernel@lists.infradead.org" List-Id: devicetree@vger.kernel.org > -----Original Message----- > From: Shawn Guo [mailto:shawnguo@kernel.org] > Sent: Sunday, April 17, 2016 9:56 PM > To: Stuart Yoder > Cc: devicetree@vger.kernel.org; linux-arm-kernel@lists.infradead.org; german.rivera@freescale.com; > linux-kernel@vger.kernel.org; robh+dt@kernel.org; Yang-Leo Li > Subject: Re: [PATCH 2/2 v2] arm64: dts: ls2080a: fsl-mc dt node updates > > On Wed, Apr 13, 2016 at 04:05:23PM -0500, Stuart Yoder wrote: > > updates to the fsl-mc node for full functionality: > > -msi-parent is needed for interrupt support > > -ranges is needed to enable the bus driver to translate bus addresses > > -dpmac nodes provide a basis for relating dpmac objects to PHYs > > > > Signed-off-by: Stuart Yoder > > --- > > > > -v2: updated comment to remove some extraneous info, added > > carriage returns after dpmac nodes > > > > arch/arm64/boot/dts/freescale/fsl-ls2080a.dtsi | 98 ++++++++++++++++++++++++++ > > 1 file changed, 98 insertions(+) > > > > diff --git a/arch/arm64/boot/dts/freescale/fsl-ls2080a.dtsi b/arch/arm64/boot/dts/freescale/fsl- > ls2080a.dtsi > > index 9d746c6..15498eb 100644 > > --- a/arch/arm64/boot/dts/freescale/fsl-ls2080a.dtsi > > +++ b/arch/arm64/boot/dts/freescale/fsl-ls2080a.dtsi > > @@ -265,6 +265,104 @@ > > compatible = "fsl,qoriq-mc"; > > reg = <0x00000008 0x0c000000 0 0x40>, /* MC portal base */ > > <0x00000000 0x08340000 0 0x40000>; /* MC control reg */ > > + msi-parent = <&its>; > > + #address-cells = <3>; > > + #size-cells = <1>; > > + > > + /* > > + * Region type 0x0 - MC portals > > + * Region type 0x1 - QBMAN portals > > + */ > > + ranges = <0x0 0x0 0x0 0x8 0x0c000000 0x4000000 > > + 0x1 0x0 0x0 0x8 0x18000000 0x8000000>; > > + > > + /* > > + * Define the maximum number of MACs present on the SoC. > > + */ > > + dpmacs { > > + #address-cells = <1>; > > + #size-cells = <0>; > > + > > + dpmac1: dpmac@1 { > > + compatible = "fsl,qoriq-mc-dpmac"; > > + reg = <1>; > > + }; > > + > > + dpmac2: dpmac@2 { > > + compatible = "fsl,qoriq-mc-dpmac"; > > + reg = <2>; > > + }; > > + > > + dpmac3: dpmac@3 { > > + compatible = "fsl,qoriq-mc-dpmac"; > > + reg = <3>; > > + }; > > + > > + dpmac4: dpmac@4 { > > + compatible = "fsl,qoriq-mc-dpmac"; > > + reg = <4>; > > + }; > > + > > + dpmac5: dpmac@5 { > > + compatible = "fsl,qoriq-mc-dpmac"; > > + reg = <5>; > > + }; > > + > > + dpmac6: dpmac@6 { > > + compatible = "fsl,qoriq-mc-dpmac"; > > + reg = <6>; > > + }; > > + > > + dpmac7: dpmac@7 { > > + compatible = "fsl,qoriq-mc-dpmac"; > > + reg = <7>; > > + }; > > + > > + dpmac8: dpmac@8 { > > + compatible = "fsl,qoriq-mc-dpmac"; > > + reg = <8>; > > + }; > > + > > + dpmac9: dpmac@9 { > > + compatible = "fsl,qoriq-mc-dpmac"; > > + reg = <9>; > > + }; > > + > > + dpmac10: dpmac@10 { > > The unit-address should be same as reg property, i.e. both are in hex > value. Ok, will update. I am going to leave the labels in decimal because that is how we refer to these mac devices in normal usage and in our documentation. ls2080a users expect dpmac10 to be the 10th mac not the 16th. Thanks, Stuart From mboxrd@z Thu Jan 1 00:00:00 1970 From: stuart.yoder@nxp.com (Stuart Yoder) Date: Tue, 19 Apr 2016 21:51:51 +0000 Subject: [PATCH 2/2 v2] arm64: dts: ls2080a: fsl-mc dt node updates In-Reply-To: <20160418025618.GU28903@tiger> References: <1460581523-32146-1-git-send-email-stuart.yoder@nxp.com> <20160418025618.GU28903@tiger> Message-ID: To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org > -----Original Message----- > From: Shawn Guo [mailto:shawnguo at kernel.org] > Sent: Sunday, April 17, 2016 9:56 PM > To: Stuart Yoder > Cc: devicetree at vger.kernel.org; linux-arm-kernel at lists.infradead.org; german.rivera at freescale.com; > linux-kernel at vger.kernel.org; robh+dt at kernel.org; Yang-Leo Li > Subject: Re: [PATCH 2/2 v2] arm64: dts: ls2080a: fsl-mc dt node updates > > On Wed, Apr 13, 2016 at 04:05:23PM -0500, Stuart Yoder wrote: > > updates to the fsl-mc node for full functionality: > > -msi-parent is needed for interrupt support > > -ranges is needed to enable the bus driver to translate bus addresses > > -dpmac nodes provide a basis for relating dpmac objects to PHYs > > > > Signed-off-by: Stuart Yoder > > --- > > > > -v2: updated comment to remove some extraneous info, added > > carriage returns after dpmac nodes > > > > arch/arm64/boot/dts/freescale/fsl-ls2080a.dtsi | 98 ++++++++++++++++++++++++++ > > 1 file changed, 98 insertions(+) > > > > diff --git a/arch/arm64/boot/dts/freescale/fsl-ls2080a.dtsi b/arch/arm64/boot/dts/freescale/fsl- > ls2080a.dtsi > > index 9d746c6..15498eb 100644 > > --- a/arch/arm64/boot/dts/freescale/fsl-ls2080a.dtsi > > +++ b/arch/arm64/boot/dts/freescale/fsl-ls2080a.dtsi > > @@ -265,6 +265,104 @@ > > compatible = "fsl,qoriq-mc"; > > reg = <0x00000008 0x0c000000 0 0x40>, /* MC portal base */ > > <0x00000000 0x08340000 0 0x40000>; /* MC control reg */ > > + msi-parent = <&its>; > > + #address-cells = <3>; > > + #size-cells = <1>; > > + > > + /* > > + * Region type 0x0 - MC portals > > + * Region type 0x1 - QBMAN portals > > + */ > > + ranges = <0x0 0x0 0x0 0x8 0x0c000000 0x4000000 > > + 0x1 0x0 0x0 0x8 0x18000000 0x8000000>; > > + > > + /* > > + * Define the maximum number of MACs present on the SoC. > > + */ > > + dpmacs { > > + #address-cells = <1>; > > + #size-cells = <0>; > > + > > + dpmac1: dpmac at 1 { > > + compatible = "fsl,qoriq-mc-dpmac"; > > + reg = <1>; > > + }; > > + > > + dpmac2: dpmac at 2 { > > + compatible = "fsl,qoriq-mc-dpmac"; > > + reg = <2>; > > + }; > > + > > + dpmac3: dpmac at 3 { > > + compatible = "fsl,qoriq-mc-dpmac"; > > + reg = <3>; > > + }; > > + > > + dpmac4: dpmac at 4 { > > + compatible = "fsl,qoriq-mc-dpmac"; > > + reg = <4>; > > + }; > > + > > + dpmac5: dpmac at 5 { > > + compatible = "fsl,qoriq-mc-dpmac"; > > + reg = <5>; > > + }; > > + > > + dpmac6: dpmac at 6 { > > + compatible = "fsl,qoriq-mc-dpmac"; > > + reg = <6>; > > + }; > > + > > + dpmac7: dpmac at 7 { > > + compatible = "fsl,qoriq-mc-dpmac"; > > + reg = <7>; > > + }; > > + > > + dpmac8: dpmac at 8 { > > + compatible = "fsl,qoriq-mc-dpmac"; > > + reg = <8>; > > + }; > > + > > + dpmac9: dpmac at 9 { > > + compatible = "fsl,qoriq-mc-dpmac"; > > + reg = <9>; > > + }; > > + > > + dpmac10: dpmac at 10 { > > The unit-address should be same as reg property, i.e. both are in hex > value. Ok, will update. I am going to leave the labels in decimal because that is how we refer to these mac devices in normal usage and in our documentation. ls2080a users expect dpmac10 to be the 10th mac not the 16th. Thanks, Stuart