2023-03-29 08:57:27

by Alexandre Mergnat

[permalink] [raw]
Subject: [PATCH v3 03/17] arm64: dts: mediatek: Increase the size BL31 reserved memory

From: Alexandre Bailon <[email protected]>

The reserved size for BL31 is too small.
This has been highlighted by the MPU that now restrict access to BL31
memory to secure world only.
This increase the size of the reserved memory.

Signed-off-by: Alexandre Bailon <[email protected]>
Signed-off-by: Alexandre Mergnat <[email protected]>
---
arch/arm64/boot/dts/mediatek/mt8365-evk.dts | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/arm64/boot/dts/mediatek/mt8365-evk.dts b/arch/arm64/boot/dts/mediatek/mt8365-evk.dts
index b68aee8f229f..fc7f6d8ae173 100644
--- a/arch/arm64/boot/dts/mediatek/mt8365-evk.dts
+++ b/arch/arm64/boot/dts/mediatek/mt8365-evk.dts
@@ -65,10 +65,10 @@ reserved-memory {
#size-cells = <2>;
ranges;

- /* 128 KiB reserved for ARM Trusted Firmware (BL31) */
+ /* 192 KiB reserved for ARM Trusted Firmware (BL31) */
bl31_secmon_reserved: secmon@43000000 {
no-map;
- reg = <0 0x43000000 0 0x20000>;
+ reg = <0 0x43000000 0 0x30000>;
};

/* 12 MiB reserved for OP-TEE (BL32)

--
2.25.1


Subject: Re: [PATCH v3 03/17] arm64: dts: mediatek: Increase the size BL31 reserved memory

Il 29/03/23 10:54, [email protected] ha scritto:
> From: Alexandre Bailon <[email protected]>
>
> The reserved size for BL31 is too small.
> This has been highlighted by the MPU that now restrict access to BL31
> memory to secure world only.
> This increase the size of the reserved memory.
>
> Signed-off-by: Alexandre Bailon <[email protected]>
> Signed-off-by: Alexandre Mergnat <[email protected]>

Reviewed-by: AngeloGioacchino Del Regno <[email protected]>

2023-03-30 17:25:26

by Matthias Brugger

[permalink] [raw]
Subject: Re: [PATCH v3 03/17] arm64: dts: mediatek: Increase the size BL31 reserved memory



On 29/03/2023 10:54, [email protected] wrote:
> From: Alexandre Bailon <[email protected]>
>
> The reserved size for BL31 is too small.
> This has been highlighted by the MPU that now restrict access to BL31
> memory to secure world only.
> This increase the size of the reserved memory.
>
> Signed-off-by: Alexandre Bailon <[email protected]>
> Signed-off-by: Alexandre Mergnat <[email protected]>

Applied, thanks!

> ---
> arch/arm64/boot/dts/mediatek/mt8365-evk.dts | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/mediatek/mt8365-evk.dts b/arch/arm64/boot/dts/mediatek/mt8365-evk.dts
> index b68aee8f229f..fc7f6d8ae173 100644
> --- a/arch/arm64/boot/dts/mediatek/mt8365-evk.dts
> +++ b/arch/arm64/boot/dts/mediatek/mt8365-evk.dts
> @@ -65,10 +65,10 @@ reserved-memory {
> #size-cells = <2>;
> ranges;
>
> - /* 128 KiB reserved for ARM Trusted Firmware (BL31) */
> + /* 192 KiB reserved for ARM Trusted Firmware (BL31) */
> bl31_secmon_reserved: secmon@43000000 {
> no-map;
> - reg = <0 0x43000000 0 0x20000>;
> + reg = <0 0x43000000 0 0x30000>;
> };
>
> /* 12 MiB reserved for OP-TEE (BL32)
>