diff mbox series

arm64: dts: qcom: msm8998: Extend reserved memory area

Message ID 89492342-018b-3e5d-563c-026ad83458c4@free.fr (mailing list archive)
State Superseded
Headers show
Series arm64: dts: qcom: msm8998: Extend reserved memory area | expand

Commit Message

Marc Gonzalez Feb. 12, 2019, 3:16 p.m. UTC
My console locks up as soon as Linux writes to [88800000,88f00000[
AFAIU, that memory area is reserved for trustzone.

Extend the reserved memory range, to prevent Linux from stepping on
trustzone's feet.

[    0.000000] early_memtest: # of tests: 17
[    0.000000]   0x0000000080000000 - 0x0000000080080000 pattern 4c494e5558726c7a
[    0.000000]   0x0000000080c7a000 - 0x0000000083200000 pattern 4c494e5558726c7a
[    0.000000]   0x0000000083205b94 - 0x0000000085800000 pattern 4c494e5558726c7a
[    0.000000]   0x0000000088f00000 - 0x00000000a1e00000 pattern 4c494e5558726c7a
[    0.000000]   0x00000000a2000000 - 0x000000017dbc06c8 pattern 4c494e5558726c7a
[    0.000000]   0x000000017dbc06f7 - 0x000000017dbc06f8 pattern 4c494e5558726c7a

Cc: stable@vger.kernel.org # 4.20+
Fixes: c7833949564ec ("arm64: dts: qcom: msm8998: Add smem related nodes")
Signed-off-by: Marc Gonzalez <marc.w.gonzalez@free.fr>
---
 arch/arm64/boot/dts/qcom/msm8998.dtsi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Marc Gonzalez Feb. 13, 2019, 9:38 a.m. UTC | #1
On 12/02/2019 16:16, Marc Gonzalez wrote:

> My console locks up as soon as Linux writes to [88800000,88f00000[
> AFAIU, that memory area is reserved for trustzone.
> 
> Extend the reserved memory range, to prevent Linux from stepping on
> trustzone's feet.

s/feet/toes/

Bjorn/Andy, could you fold this vital fixup into the final patch? ^_^

> [    0.000000] early_memtest: # of tests: 17
> [    0.000000]   0x0000000080000000 - 0x0000000080080000 pattern 4c494e5558726c7a
> [    0.000000]   0x0000000080c7a000 - 0x0000000083200000 pattern 4c494e5558726c7a
> [    0.000000]   0x0000000083205b94 - 0x0000000085800000 pattern 4c494e5558726c7a
> [    0.000000]   0x0000000088f00000 - 0x00000000a1e00000 pattern 4c494e5558726c7a
> [    0.000000]   0x00000000a2000000 - 0x000000017dbc06c8 pattern 4c494e5558726c7a
> [    0.000000]   0x000000017dbc06f7 - 0x000000017dbc06f8 pattern 4c494e5558726c7a
> 
> Cc: stable@vger.kernel.org # 4.20+
> Fixes: c7833949564ec ("arm64: dts: qcom: msm8998: Add smem related nodes")
> Signed-off-by: Marc Gonzalez <marc.w.gonzalez@free.fr>
> ---
>  arch/arm64/boot/dts/qcom/msm8998.dtsi | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/arch/arm64/boot/dts/qcom/msm8998.dtsi b/arch/arm64/boot/dts/qcom/msm8998.dtsi
> index d69401152089..f5ec83a91fe4 100644
> --- a/arch/arm64/boot/dts/qcom/msm8998.dtsi
> +++ b/arch/arm64/boot/dts/qcom/msm8998.dtsi
> @@ -38,7 +38,7 @@
>  		};
>  
>  		memory@86200000 {
> -			reg = <0x0 0x86200000 0x0 0x2600000>;
> +			reg = <0x0 0x86200000 0x0 0x2d00000>;
>  			no-map;
>  		};
>
Sibi Sankar Feb. 19, 2019, 1:30 p.m. UTC | #2
Hey Marc,

Reviewed-by: Sibi Sankar <sibis@codeaurora.org>

On 2019-02-12 20:46, Marc Gonzalez wrote:
> My console locks up as soon as Linux writes to [88800000,88f00000[
> AFAIU, that memory area is reserved for trustzone.
> 
> Extend the reserved memory range, to prevent Linux from stepping on
> trustzone's feet.
> 
> [    0.000000] early_memtest: # of tests: 17
> [    0.000000]   0x0000000080000000 - 0x0000000080080000 pattern
> 4c494e5558726c7a
> [    0.000000]   0x0000000080c7a000 - 0x0000000083200000 pattern
> 4c494e5558726c7a
> [    0.000000]   0x0000000083205b94 - 0x0000000085800000 pattern
> 4c494e5558726c7a
> [    0.000000]   0x0000000088f00000 - 0x00000000a1e00000 pattern
> 4c494e5558726c7a
> [    0.000000]   0x00000000a2000000 - 0x000000017dbc06c8 pattern
> 4c494e5558726c7a
> [    0.000000]   0x000000017dbc06f7 - 0x000000017dbc06f8 pattern
> 4c494e5558726c7a
> 
> Cc: stable@vger.kernel.org # 4.20+
> Fixes: c7833949564ec ("arm64: dts: qcom: msm8998: Add smem related 
> nodes")
> Signed-off-by: Marc Gonzalez <marc.w.gonzalez@free.fr>
> ---
>  arch/arm64/boot/dts/qcom/msm8998.dtsi | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/arch/arm64/boot/dts/qcom/msm8998.dtsi
> b/arch/arm64/boot/dts/qcom/msm8998.dtsi
> index d69401152089..f5ec83a91fe4 100644
> --- a/arch/arm64/boot/dts/qcom/msm8998.dtsi
> +++ b/arch/arm64/boot/dts/qcom/msm8998.dtsi
> @@ -38,7 +38,7 @@
>  		};
> 
>  		memory@86200000 {
> -			reg = <0x0 0x86200000 0x0 0x2600000>;
> +			reg = <0x0 0x86200000 0x0 0x2d00000>;
>  			no-map;
>  		};
diff mbox series

Patch

diff --git a/arch/arm64/boot/dts/qcom/msm8998.dtsi b/arch/arm64/boot/dts/qcom/msm8998.dtsi
index d69401152089..f5ec83a91fe4 100644
--- a/arch/arm64/boot/dts/qcom/msm8998.dtsi
+++ b/arch/arm64/boot/dts/qcom/msm8998.dtsi
@@ -38,7 +38,7 @@ 
 		};
 
 		memory@86200000 {
-			reg = <0x0 0x86200000 0x0 0x2600000>;
+			reg = <0x0 0x86200000 0x0 0x2d00000>;
 			no-map;
 		};