* [PATCH] arm64: dts: marvell: fix CP110 ICU node size
@ 2018-07-05 12:34 ` Miquel Raynal
0 siblings, 0 replies; 6+ messages in thread
From: Miquel Raynal @ 2018-07-05 12:34 UTC (permalink / raw)
To: linux-arm-kernel
commit 2f872ddcdb1e8e2186162616cea4581b8403849d upstream.
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 # 4.14.x
Signed-off-by: Miquel Raynal <miquel.raynal@bootlin.com>
Reviewed-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
Signed-off-by: Gregory CLEMENT <gregory.clement@bootlin.com>
---
Backport to 4.14 stable tree: the original patch did not applied
because of the effort of DT de-duplication that merged two files in one.
arch/arm64/boot/dts/marvell/armada-cp110-master.dtsi | 2 +-
arch/arm64/boot/dts/marvell/armada-cp110-slave.dtsi | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/arch/arm64/boot/dts/marvell/armada-cp110-master.dtsi b/arch/arm64/boot/dts/marvell/armada-cp110-master.dtsi
index 9a7b63cd63a3..7c19f32d0f44 100644
--- a/arch/arm64/boot/dts/marvell/armada-cp110-master.dtsi
+++ b/arch/arm64/boot/dts/marvell/armada-cp110-master.dtsi
@@ -169,7 +169,7 @@
cpm_icu: interrupt-controller at 1e0000 {
compatible = "marvell,cp110-icu";
- reg = <0x1e0000 0x10>;
+ reg = <0x1e0000 0x440>;
#interrupt-cells = <3>;
interrupt-controller;
msi-parent = <&gicp>;
diff --git a/arch/arm64/boot/dts/marvell/armada-cp110-slave.dtsi b/arch/arm64/boot/dts/marvell/armada-cp110-slave.dtsi
index faf28633a309..150e97a1dfea 100644
--- a/arch/arm64/boot/dts/marvell/armada-cp110-slave.dtsi
+++ b/arch/arm64/boot/dts/marvell/armada-cp110-slave.dtsi
@@ -169,7 +169,7 @@
cps_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
^ permalink raw reply related [flat|nested] 6+ messages in thread* [PATCH] arm64: dts: marvell: fix CP110 ICU node size @ 2018-07-05 12:34 ` Miquel Raynal 0 siblings, 0 replies; 6+ messages in thread From: Miquel Raynal @ 2018-07-05 12:34 UTC (permalink / raw) To: stable Cc: Jason Cooper, Andrew Lunn, Gregory Clement, Sebastian Hesselbarth, Rob Herring, Mark Rutland, linux-arm-kernel, devicetree, linux-kernel, Thomas Petazzoni, Nadav Haklai commit 2f872ddcdb1e8e2186162616cea4581b8403849d upstream. 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 # 4.14.x Signed-off-by: Miquel Raynal <miquel.raynal@bootlin.com> Reviewed-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com> Signed-off-by: Gregory CLEMENT <gregory.clement@bootlin.com> --- Backport to 4.14 stable tree: the original patch did not applied because of the effort of DT de-duplication that merged two files in one. arch/arm64/boot/dts/marvell/armada-cp110-master.dtsi | 2 +- arch/arm64/boot/dts/marvell/armada-cp110-slave.dtsi | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/arch/arm64/boot/dts/marvell/armada-cp110-master.dtsi b/arch/arm64/boot/dts/marvell/armada-cp110-master.dtsi index 9a7b63cd63a3..7c19f32d0f44 100644 --- a/arch/arm64/boot/dts/marvell/armada-cp110-master.dtsi +++ b/arch/arm64/boot/dts/marvell/armada-cp110-master.dtsi @@ -169,7 +169,7 @@ cpm_icu: interrupt-controller@1e0000 { compatible = "marvell,cp110-icu"; - reg = <0x1e0000 0x10>; + reg = <0x1e0000 0x440>; #interrupt-cells = <3>; interrupt-controller; msi-parent = <&gicp>; diff --git a/arch/arm64/boot/dts/marvell/armada-cp110-slave.dtsi b/arch/arm64/boot/dts/marvell/armada-cp110-slave.dtsi index faf28633a309..150e97a1dfea 100644 --- a/arch/arm64/boot/dts/marvell/armada-cp110-slave.dtsi +++ b/arch/arm64/boot/dts/marvell/armada-cp110-slave.dtsi @@ -169,7 +169,7 @@ cps_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] 6+ messages in thread
* [PATCH] arm64: dts: marvell: fix CP110 ICU node size 2018-07-05 12:34 ` Miquel Raynal @ 2018-07-05 13:26 ` Gregory CLEMENT -1 siblings, 0 replies; 6+ messages in thread From: Gregory CLEMENT @ 2018-07-05 13:26 UTC (permalink / raw) To: linux-arm-kernel Hi Miquel, On jeu., juil. 05 2018, Miquel Raynal <miquel.raynal@bootlin.com> wrote: > commit 2f872ddcdb1e8e2186162616cea4581b8403849d upstream. > > 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 # 4.14.x > Signed-off-by: Miquel Raynal <miquel.raynal@bootlin.com> > Reviewed-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com> > Signed-off-by: Gregory CLEMENT <gregory.clement@bootlin.com> See https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=2f872ddcdb1e8e2186162616cea4581b8403849d it is already in the linux kernel. Gregory > --- > > Backport to 4.14 stable tree: the original patch did not applied > because of the effort of DT de-duplication that merged two files in one. > > > arch/arm64/boot/dts/marvell/armada-cp110-master.dtsi | 2 +- > arch/arm64/boot/dts/marvell/armada-cp110-slave.dtsi | 2 +- > 2 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/arch/arm64/boot/dts/marvell/armada-cp110-master.dtsi b/arch/arm64/boot/dts/marvell/armada-cp110-master.dtsi > index 9a7b63cd63a3..7c19f32d0f44 100644 > --- a/arch/arm64/boot/dts/marvell/armada-cp110-master.dtsi > +++ b/arch/arm64/boot/dts/marvell/armada-cp110-master.dtsi > @@ -169,7 +169,7 @@ > > cpm_icu: interrupt-controller at 1e0000 { > compatible = "marvell,cp110-icu"; > - reg = <0x1e0000 0x10>; > + reg = <0x1e0000 0x440>; > #interrupt-cells = <3>; > interrupt-controller; > msi-parent = <&gicp>; > diff --git a/arch/arm64/boot/dts/marvell/armada-cp110-slave.dtsi b/arch/arm64/boot/dts/marvell/armada-cp110-slave.dtsi > index faf28633a309..150e97a1dfea 100644 > --- a/arch/arm64/boot/dts/marvell/armada-cp110-slave.dtsi > +++ b/arch/arm64/boot/dts/marvell/armada-cp110-slave.dtsi > @@ -169,7 +169,7 @@ > > cps_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 ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] arm64: dts: marvell: fix CP110 ICU node size @ 2018-07-05 13:26 ` Gregory CLEMENT 0 siblings, 0 replies; 6+ messages in thread From: Gregory CLEMENT @ 2018-07-05 13:26 UTC (permalink / raw) To: Miquel Raynal Cc: stable, Jason Cooper, Andrew Lunn, Sebastian Hesselbarth, Rob Herring, Mark Rutland, linux-arm-kernel, devicetree, linux-kernel, Thomas Petazzoni, Nadav Haklai Hi Miquel, On jeu., juil. 05 2018, Miquel Raynal <miquel.raynal@bootlin.com> wrote: > commit 2f872ddcdb1e8e2186162616cea4581b8403849d upstream. > > 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 # 4.14.x > Signed-off-by: Miquel Raynal <miquel.raynal@bootlin.com> > Reviewed-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com> > Signed-off-by: Gregory CLEMENT <gregory.clement@bootlin.com> See https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=2f872ddcdb1e8e2186162616cea4581b8403849d it is already in the linux kernel. Gregory > --- > > Backport to 4.14 stable tree: the original patch did not applied > because of the effort of DT de-duplication that merged two files in one. > > > arch/arm64/boot/dts/marvell/armada-cp110-master.dtsi | 2 +- > arch/arm64/boot/dts/marvell/armada-cp110-slave.dtsi | 2 +- > 2 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/arch/arm64/boot/dts/marvell/armada-cp110-master.dtsi b/arch/arm64/boot/dts/marvell/armada-cp110-master.dtsi > index 9a7b63cd63a3..7c19f32d0f44 100644 > --- a/arch/arm64/boot/dts/marvell/armada-cp110-master.dtsi > +++ b/arch/arm64/boot/dts/marvell/armada-cp110-master.dtsi > @@ -169,7 +169,7 @@ > > cpm_icu: interrupt-controller@1e0000 { > compatible = "marvell,cp110-icu"; > - reg = <0x1e0000 0x10>; > + reg = <0x1e0000 0x440>; > #interrupt-cells = <3>; > interrupt-controller; > msi-parent = <&gicp>; > diff --git a/arch/arm64/boot/dts/marvell/armada-cp110-slave.dtsi b/arch/arm64/boot/dts/marvell/armada-cp110-slave.dtsi > index faf28633a309..150e97a1dfea 100644 > --- a/arch/arm64/boot/dts/marvell/armada-cp110-slave.dtsi > +++ b/arch/arm64/boot/dts/marvell/armada-cp110-slave.dtsi > @@ -169,7 +169,7 @@ > > cps_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] 6+ messages in thread
* [PATCH] arm64: dts: marvell: fix CP110 ICU node size 2018-07-05 13:26 ` Gregory CLEMENT @ 2018-07-05 13:28 ` Gregory CLEMENT -1 siblings, 0 replies; 6+ messages in thread From: Gregory CLEMENT @ 2018-07-05 13:28 UTC (permalink / raw) To: linux-arm-kernel Hi, On jeu., juil. 05 2018, Gregory CLEMENT <gregory.clement@bootlin.com> wrote: > Hi Miquel, > > On jeu., juil. 05 2018, Miquel Raynal <miquel.raynal@bootlin.com> wrote: > >> commit 2f872ddcdb1e8e2186162616cea4581b8403849d upstream. >> >> 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 # 4.14.x >> Signed-off-by: Miquel Raynal <miquel.raynal@bootlin.com> >> Reviewed-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com> >> Signed-off-by: Gregory CLEMENT <gregory.clement@bootlin.com> > > See > https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=2f872ddcdb1e8e2186162616cea4581b8403849d > > it is already in the linux kernel. > > Gregory I missed the comment below and the fact it was for the stable branch. Sorryt for the noise! Gregory > >> --- >> >> Backport to 4.14 stable tree: the original patch did not applied >> because of the effort of DT de-duplication that merged two files in one. >> >> >> arch/arm64/boot/dts/marvell/armada-cp110-master.dtsi | 2 +- >> arch/arm64/boot/dts/marvell/armada-cp110-slave.dtsi | 2 +- >> 2 files changed, 2 insertions(+), 2 deletions(-) >> >> diff --git a/arch/arm64/boot/dts/marvell/armada-cp110-master.dtsi b/arch/arm64/boot/dts/marvell/armada-cp110-master.dtsi >> index 9a7b63cd63a3..7c19f32d0f44 100644 >> --- a/arch/arm64/boot/dts/marvell/armada-cp110-master.dtsi >> +++ b/arch/arm64/boot/dts/marvell/armada-cp110-master.dtsi >> @@ -169,7 +169,7 @@ >> >> cpm_icu: interrupt-controller at 1e0000 { >> compatible = "marvell,cp110-icu"; >> - reg = <0x1e0000 0x10>; >> + reg = <0x1e0000 0x440>; >> #interrupt-cells = <3>; >> interrupt-controller; >> msi-parent = <&gicp>; >> diff --git a/arch/arm64/boot/dts/marvell/armada-cp110-slave.dtsi b/arch/arm64/boot/dts/marvell/armada-cp110-slave.dtsi >> index faf28633a309..150e97a1dfea 100644 >> --- a/arch/arm64/boot/dts/marvell/armada-cp110-slave.dtsi >> +++ b/arch/arm64/boot/dts/marvell/armada-cp110-slave.dtsi >> @@ -169,7 +169,7 @@ >> >> cps_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 > > _______________________________________________ > linux-arm-kernel mailing list > linux-arm-kernel at lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel -- Gregory Clement, Bootlin (formerly Free Electrons) Embedded Linux and Kernel engineering http://bootlin.com ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] arm64: dts: marvell: fix CP110 ICU node size @ 2018-07-05 13:28 ` Gregory CLEMENT 0 siblings, 0 replies; 6+ messages in thread From: Gregory CLEMENT @ 2018-07-05 13:28 UTC (permalink / raw) To: Miquel Raynal Cc: Mark Rutland, Andrew Lunn, Jason Cooper, devicetree, linux-kernel, stable, Nadav Haklai, Rob Herring, Thomas Petazzoni, linux-arm-kernel, Sebastian Hesselbarth Hi, On jeu., juil. 05 2018, Gregory CLEMENT <gregory.clement@bootlin.com> wrote: > Hi Miquel, > > On jeu., juil. 05 2018, Miquel Raynal <miquel.raynal@bootlin.com> wrote: > >> commit 2f872ddcdb1e8e2186162616cea4581b8403849d upstream. >> >> 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 # 4.14.x >> Signed-off-by: Miquel Raynal <miquel.raynal@bootlin.com> >> Reviewed-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com> >> Signed-off-by: Gregory CLEMENT <gregory.clement@bootlin.com> > > See > https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=2f872ddcdb1e8e2186162616cea4581b8403849d > > it is already in the linux kernel. > > Gregory I missed the comment below and the fact it was for the stable branch. Sorryt for the noise! Gregory > >> --- >> >> Backport to 4.14 stable tree: the original patch did not applied >> because of the effort of DT de-duplication that merged two files in one. >> >> >> arch/arm64/boot/dts/marvell/armada-cp110-master.dtsi | 2 +- >> arch/arm64/boot/dts/marvell/armada-cp110-slave.dtsi | 2 +- >> 2 files changed, 2 insertions(+), 2 deletions(-) >> >> diff --git a/arch/arm64/boot/dts/marvell/armada-cp110-master.dtsi b/arch/arm64/boot/dts/marvell/armada-cp110-master.dtsi >> index 9a7b63cd63a3..7c19f32d0f44 100644 >> --- a/arch/arm64/boot/dts/marvell/armada-cp110-master.dtsi >> +++ b/arch/arm64/boot/dts/marvell/armada-cp110-master.dtsi >> @@ -169,7 +169,7 @@ >> >> cpm_icu: interrupt-controller@1e0000 { >> compatible = "marvell,cp110-icu"; >> - reg = <0x1e0000 0x10>; >> + reg = <0x1e0000 0x440>; >> #interrupt-cells = <3>; >> interrupt-controller; >> msi-parent = <&gicp>; >> diff --git a/arch/arm64/boot/dts/marvell/armada-cp110-slave.dtsi b/arch/arm64/boot/dts/marvell/armada-cp110-slave.dtsi >> index faf28633a309..150e97a1dfea 100644 >> --- a/arch/arm64/boot/dts/marvell/armada-cp110-slave.dtsi >> +++ b/arch/arm64/boot/dts/marvell/armada-cp110-slave.dtsi >> @@ -169,7 +169,7 @@ >> >> cps_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 > > _______________________________________________ > linux-arm-kernel mailing list > linux-arm-kernel@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel -- Gregory Clement, Bootlin (formerly Free Electrons) Embedded Linux and Kernel engineering http://bootlin.com ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2018-07-05 13:28 UTC | newest] Thread overview: 6+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2018-07-05 12:34 [PATCH] arm64: dts: marvell: fix CP110 ICU node size Miquel Raynal 2018-07-05 12:34 ` Miquel Raynal 2018-07-05 13:26 ` Gregory CLEMENT 2018-07-05 13:26 ` Gregory CLEMENT 2018-07-05 13:28 ` Gregory CLEMENT 2018-07-05 13:28 ` Gregory CLEMENT
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.