| Message ID | 20241204190524.21862-5-val@packett.cool (mailing list archive) |
|---|---|
| State | New |
| Headers | show |
| Series | MT8516/MT8167 dtsi fixes | expand |
Il 04/12/24 20:05, Val Packett ha scritto: > The Android DTB for the related MT8167 reserves 0x30000. This is likely > correct for MT8516 Android devices as well, and there's never any harm > in reserving 64KiB more. > > Fixes: 5236347bde42 ("arm64: dts: mediatek: add dtsi for MT8516") > Signed-off-by: Val Packett <val@packett.cool> That has changed over time with new bootloaders, yes, so there's no harm and it's actually a necessary change. Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>
diff --git a/arch/arm64/boot/dts/mediatek/mt8516.dtsi b/arch/arm64/boot/dts/mediatek/mt8516.dtsi index dd17d8a88c19..e30623ebac0e 100644 --- a/arch/arm64/boot/dts/mediatek/mt8516.dtsi +++ b/arch/arm64/boot/dts/mediatek/mt8516.dtsi @@ -144,10 +144,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>; }; };
The Android DTB for the related MT8167 reserves 0x30000. This is likely correct for MT8516 Android devices as well, and there's never any harm in reserving 64KiB more. Fixes: 5236347bde42 ("arm64: dts: mediatek: add dtsi for MT8516") Signed-off-by: Val Packett <val@packett.cool> --- arch/arm64/boot/dts/mediatek/mt8516.dtsi | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)