From mboxrd@z Thu Jan 1 00:00:00 1970 From: Gregory CLEMENT Subject: Re: [PATCH v3 03/17] arm64: dts: marvell: fix CP110 ICU node size Date: Mon, 25 Jun 2018 17:05:22 +0200 Message-ID: <87a7ric3f1.fsf@bootlin.com> References: <20180622151432.1566-1-miquel.raynal@bootlin.com> <20180622151432.1566-4-miquel.raynal@bootlin.com> Mime-Version: 1.0 Content-Type: text/plain Return-path: In-Reply-To: <20180622151432.1566-4-miquel.raynal@bootlin.com> (Miquel Raynal's message of "Fri, 22 Jun 2018 17:14:18 +0200") Sender: stable-owner@vger.kernel.org To: Miquel Raynal Cc: Thomas Gleixner , Jason Cooper , Marc Zyngier , Catalin Marinas , Will Deacon , Andrew Lunn , Sebastian Hesselbarth , Rob Herring , Mark Rutland , devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, Thomas Petazzoni , Antoine Tenart , Maxime Chevallier , Nadav Haklai , Haim Boot , Hanna Hawa , stable@vger.kernel.org List-Id: devicetree@vger.kernel.org Hi Miquel, On ven., juin 22 2018, Miquel Raynal wrote: > ICU size in CP110 is not 0x10 but at least 0x440 bytes long (from the > specification). > > Fixes: 6ef84a827c37 ("arm64: dts: marvell: enable GICP and ICU on Armada 7K/8K") > Cc: stable@vger.kernel.org > Signed-off-by: Miquel Raynal > Reviewed-by: Thomas Petazzoni This patch was part of the last PR from arm-soc for Linus and should be part of 4.18-rc3, so if there is new version you can drop it. Thanks, Gregory > --- > arch/arm64/boot/dts/marvell/armada-cp110.dtsi | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/arch/arm64/boot/dts/marvell/armada-cp110.dtsi b/arch/arm64/boot/dts/marvell/armada-cp110.dtsi > index 7dabe25f6774..1c6ff8197a88 100644 > --- a/arch/arm64/boot/dts/marvell/armada-cp110.dtsi > +++ b/arch/arm64/boot/dts/marvell/armada-cp110.dtsi > @@ -149,7 +149,7 @@ > > CP110_LABEL(icu): interrupt-controller@1e0000 { > compatible = "marvell,cp110-icu"; > - reg = <0x1e0000 0x10>; > + reg = <0x1e0000 0x440>; > #interrupt-cells = <3>; > interrupt-controller; > msi-parent = <&gicp>; > -- > 2.14.1 > -- Gregory Clement, Bootlin (formerly Free Electrons) Embedded Linux and Kernel engineering http://bootlin.com From mboxrd@z Thu Jan 1 00:00:00 1970 From: gregory.clement@bootlin.com (Gregory CLEMENT) Date: Mon, 25 Jun 2018 17:05:22 +0200 Subject: [PATCH v3 03/17] arm64: dts: marvell: fix CP110 ICU node size In-Reply-To: <20180622151432.1566-4-miquel.raynal@bootlin.com> (Miquel Raynal's message of "Fri, 22 Jun 2018 17:14:18 +0200") References: <20180622151432.1566-1-miquel.raynal@bootlin.com> <20180622151432.1566-4-miquel.raynal@bootlin.com> Message-ID: <87a7ric3f1.fsf@bootlin.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Hi Miquel, On ven., juin 22 2018, Miquel Raynal wrote: > ICU size in CP110 is not 0x10 but at least 0x440 bytes long (from the > specification). > > Fixes: 6ef84a827c37 ("arm64: dts: marvell: enable GICP and ICU on Armada 7K/8K") > Cc: stable at vger.kernel.org > Signed-off-by: Miquel Raynal > Reviewed-by: Thomas Petazzoni This patch was part of the last PR from arm-soc for Linus and should be part of 4.18-rc3, so if there is new version you can drop it. Thanks, Gregory > --- > arch/arm64/boot/dts/marvell/armada-cp110.dtsi | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/arch/arm64/boot/dts/marvell/armada-cp110.dtsi b/arch/arm64/boot/dts/marvell/armada-cp110.dtsi > index 7dabe25f6774..1c6ff8197a88 100644 > --- a/arch/arm64/boot/dts/marvell/armada-cp110.dtsi > +++ b/arch/arm64/boot/dts/marvell/armada-cp110.dtsi > @@ -149,7 +149,7 @@ > > CP110_LABEL(icu): interrupt-controller at 1e0000 { > compatible = "marvell,cp110-icu"; > - reg = <0x1e0000 0x10>; > + reg = <0x1e0000 0x440>; > #interrupt-cells = <3>; > interrupt-controller; > msi-parent = <&gicp>; > -- > 2.14.1 > -- Gregory Clement, Bootlin (formerly Free Electrons) Embedded Linux and Kernel engineering http://bootlin.com