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>;
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
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
linux-stable-mirror@lists.linaro.org