From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from foss.arm.com ([217.140.101.70]:43124 "EHLO foss.arm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751661AbbIDKyz (ORCPT ); Fri, 4 Sep 2015 06:54:55 -0400 Message-ID: <55E9787A.7050304@arm.com> Date: Fri, 04 Sep 2015 11:54:50 +0100 From: Marc Zyngier MIME-Version: 1.0 To: Bhupesh Sharma , arnd@arndb.de, mark.rutland@arm.com, linux-arm-kernel@lists.infradead.org, linux-clk@vger.kernel.org CC: bhupesh.linux@gmail.com, Catalin.Marinas@arm.com, will.deacon@arm.com, olof@lixom.net, Jaiprakash Singh , Alison Wang , Liu Gang , Minghuan Lian , Shaohui Xie , Nikhil Badola , Yangbo Lu , Scott Wood Subject: Re: [PATCH v2 07/10] dts/ls2085a: Update DTSI to add support of various peripherals References: <1441350322-4671-1-git-send-email-bhupesh.sharma@freescale.com> In-Reply-To: <1441350322-4671-1-git-send-email-bhupesh.sharma@freescale.com> Content-Type: text/plain; charset=windows-1252 Sender: linux-clk-owner@vger.kernel.org List-ID: On 04/09/15 08:05, Bhupesh Sharma wrote: > This patch updates the LS2085a DTSI (DTS Include) file to add > support for various peripherals supported by FSL LS2085a SoC, for e.g.: > - USB 3.0 Host > - PMU > - CCN-504 > - Watchdog > - SATA > - SPI > - PCIe > - etc. > > Signed-off-by: Bhupesh Sharma > Signed-off-by: Jaiprakash Singh > Signed-off-by: Alison Wang > Signed-off-by: Liu Gang > Signed-off-by: Minghuan Lian > Signed-off-by: Shaohui Xie > Signed-off-by: Nikhil Badola > Signed-off-by: Yangbo Lu > Signed-off-by: Scott Wood > --- > arch/arm64/boot/dts/freescale/fsl-ls2080a.dtsi | 469 +++++++++++++++++++++++- > 1 file changed, 459 insertions(+), 10 deletions(-) > > diff --git a/arch/arm64/boot/dts/freescale/fsl-ls2080a.dtsi b/arch/arm64/boot/dts/freescale/fsl-ls2080a.dtsi > index 333d942..5fee0a7 100644 > --- a/arch/arm64/boot/dts/freescale/fsl-ls2080a.dtsi > +++ b/arch/arm64/boot/dts/freescale/fsl-ls2080a.dtsi [...] > gic: interrupt-controller@6000000 { > compatible = "arm,gic-v3"; > reg = <0x0 0x06000000 0 0x10000>, /* GIC Dist */ > - <0x0 0x06100000 0 0x100000>; /* GICR (RD_base + SGI_base) */ > + <0x0 0x06100000 0 0x100000>, /* GICR (RD_base + SGI_base) */ > + <0x0 0x0c0c0000 0 0x2000>, /* GICC */ > + <0x0 0x0c0d0000 0 0x1000>, /* GICH */ > + <0x0 0x0c0e0000 0 0x20000>; /* GICV */ > #interrupt-cells = <3>; > + #address-cells = <2>; > + #size-cells = <2>; > + ranges; > interrupt-controller; > interrupts = <1 9 0x4>; > + > + its: gic-its@6020000 { > + compatible = "arm,gic-v3-its"; > + msi-controller; > + reg = <0x0 0x6020000 0 0x20000>; > + }; > + }; [...] > fsl_mc: fsl-mc@80c000000 { > compatible = "fsl,qoriq-mc"; > + #stream-id-cells = <2>; > reg = <0x00000008 0x0c000000 0 0x40>, /* MC portal base */ > <0x00000000 0x08340000 0 0x40000>; /* MC control reg */ > + lpi-parent = <&its>; I'm afraid there is no such thing as "lpi_parent" as a property. We have msi_parent, which is (I think) what you want, and is dealt with in the ITS driver. Also, you'll need to describe the DeviceIDs that can be generated by this device so that the ITS can be correctly programmed. Thanks, M. -- Jazz is not dead. It just smells funny...