* [PATCH v3 03/17] arm64: dts: marvell: fix CP110 ICU node size [not found] <20180622151432.1566-1-miquel.raynal@bootlin.com> @ 2018-06-22 15:14 ` Miquel Raynal 2018-06-25 15:05 ` Gregory CLEMENT 0 siblings, 1 reply; 3+ messages in thread From: Miquel Raynal @ 2018-06-22 15:14 UTC (permalink / raw) To: Thomas Gleixner, Jason Cooper, Marc Zyngier, Catalin Marinas, Will Deacon, Andrew Lunn, Gregory Clement, Sebastian Hesselbarth Cc: Rob Herring, Mark Rutland, devicetree, linux-arm-kernel, Thomas Petazzoni, Antoine Tenart, Maxime Chevallier, Nadav Haklai, Haim Boot, Hanna Hawa, Miquel Raynal, stable 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 <miquel.raynal@bootlin.com> Reviewed-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com> --- 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 ^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH v3 03/17] arm64: dts: marvell: fix CP110 ICU node size 2018-06-22 15:14 ` [PATCH v3 03/17] arm64: dts: marvell: fix CP110 ICU node size Miquel Raynal @ 2018-06-25 15:05 ` Gregory CLEMENT 2018-06-25 15:09 ` Miquel Raynal 0 siblings, 1 reply; 3+ messages in thread From: Gregory CLEMENT @ 2018-06-25 15:05 UTC (permalink / raw) To: Miquel Raynal Cc: Thomas Gleixner, Jason Cooper, Marc Zyngier, Catalin Marinas, Will Deacon, Andrew Lunn, Sebastian Hesselbarth, Rob Herring, Mark Rutland, devicetree, linux-arm-kernel, Thomas Petazzoni, Antoine Tenart, Maxime Chevallier, Nadav Haklai, Haim Boot, Hanna Hawa, stable Hi Miquel, On ven., juin 22 2018, Miquel Raynal <miquel.raynal@bootlin.com> 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 <miquel.raynal@bootlin.com> > Reviewed-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com> 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 ^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH v3 03/17] arm64: dts: marvell: fix CP110 ICU node size 2018-06-25 15:05 ` Gregory CLEMENT @ 2018-06-25 15:09 ` Miquel Raynal 0 siblings, 0 replies; 3+ messages in thread From: Miquel Raynal @ 2018-06-25 15:09 UTC (permalink / raw) To: Gregory CLEMENT Cc: Thomas Gleixner, Jason Cooper, Marc Zyngier, Catalin Marinas, Will Deacon, Andrew Lunn, Sebastian Hesselbarth, Rob Herring, Mark Rutland, devicetree, linux-arm-kernel, Thomas Petazzoni, Antoine Tenart, Maxime Chevallier, Nadav Haklai, Haim Boot, Hanna Hawa, stable Hi Gregory, On Mon, 25 Jun 2018 17:05:22 +0200, Gregory CLEMENT <gregory.clement@bootlin.com> wrote: > Hi Miquel, > > On ven., juin 22 2018, Miquel Raynal <miquel.raynal@bootlin.com> 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 <miquel.raynal@bootlin.com> > > Reviewed-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com> > > 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. > Sure, thanks for the reminder. Miquèl ^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2018-06-25 15:09 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <20180622151432.1566-1-miquel.raynal@bootlin.com>
2018-06-22 15:14 ` [PATCH v3 03/17] arm64: dts: marvell: fix CP110 ICU node size Miquel Raynal
2018-06-25 15:05 ` Gregory CLEMENT
2018-06-25 15:09 ` Miquel Raynal
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).