diff mbox series

[U-Boot,v3,1/1] arm64: dts: marvell: armada-ap806: reserve PSCI area

Message ID 20181221163046.25769-1-xypron.glpk@gmx.de
State Accepted
Commit cf63dad014bae080445bccbf9cecbe05f2cbed45
Delegated to: Stefan Roese
Headers show
Series [U-Boot,v3,1/1] arm64: dts: marvell: armada-ap806: reserve PSCI area | expand

Commit Message

Heinrich Schuchardt Dec. 21, 2018, 4:30 p.m. UTC
The memory area [0x4000000-0x4200000[ is occupied by the PSCI firmware. Any
attempt to access it from U-Boot leads to an immediate crash.

So let's make the same memory reservation as the vendor device tree.

Signed-off-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
---
cf. https://lkml.org/lkml/2018/12/21/253

v3
	Move change from armada-8040-mcbin.dts to armada-ap806.dtsi.
	I only have a MACCHIATObin for testing. But in
	https://github.com/MarvellEmbeddedProcessors/linux-marvell
	it is done in the same way.
v2
	add missing right brace

---
 arch/arm/dts/armada-ap806.dtsi | 11 +++++++++++
 1 file changed, 11 insertions(+)

Comments

Heinrich Schuchardt Jan. 20, 2019, 8:25 a.m. UTC | #1
On 12/21/18 5:30 PM, Heinrich Schuchardt wrote:
> The memory area [0x4000000-0x4200000[ is occupied by the PSCI firmware. Any
> attempt to access it from U-Boot leads to an immediate crash.
> 
> So let's make the same memory reservation as the vendor device tree.
> 
> Signed-off-by: Heinrich Schuchardt <xypron.glpk@gmx.de>

The accompanying patch has been accepted for the Linux 5.0-rc2 kernel:
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?h=v5.0-rc2&id=132ac39cffbcfed80ada38ef0fc6d34d95da7be6

Best regards

Heinrich
Stefan Roese Jan. 21, 2019, 10:42 a.m. UTC | #2
On 21.12.18 17:30, Heinrich Schuchardt wrote:
> The memory area [0x4000000-0x4200000[ is occupied by the PSCI firmware. Any
> attempt to access it from U-Boot leads to an immediate crash.
> 
> So let's make the same memory reservation as the vendor device tree.
> 
> Signed-off-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
> ---
> cf. https://lkml.org/lkml/2018/12/21/253
> 
> v3
> 	Move change from armada-8040-mcbin.dts to armada-ap806.dtsi.
> 	I only have a MACCHIATObin for testing. But in
> 	https://github.com/MarvellEmbeddedProcessors/linux-marvell
> 	it is done in the same way.
> v2
> 	add missing right brace
> 
> ---
>   arch/arm/dts/armada-ap806.dtsi | 11 +++++++++++
>   1 file changed, 11 insertions(+)

Applied to u-boot-marvell/master

Thanks,
Stefan
diff mbox series

Patch

diff --git a/arch/arm/dts/armada-ap806.dtsi b/arch/arm/dts/armada-ap806.dtsi
index ebdee514c0..d28c0822f8 100644
--- a/arch/arm/dts/armada-ap806.dtsi
+++ b/arch/arm/dts/armada-ap806.dtsi
@@ -64,6 +64,17 @@ 
 		method = "smc";
 	};
 
+	reserved-memory {
+		#address-cells = <2>;
+		#size-cells = <2>;
+		ranges;
+
+		psci-area@4000000 {
+			reg = <0x0 0x4000000 0x0 0x200000>;
+			no-map;
+		};
+	};
+
 	ap806 {
 		#address-cells = <2>;
 		#size-cells = <2>;