Re: [PATCH] arm64: dts: mediatek: mt8195-demo: fix the memory size of

From: Miles Chen
Date: Tue Sep 27 2022 - 21:48:06 EST


>On 9/26/22 15:05, Miles Chen wrote:
>> Hi Macpaul,
>>
>>> The size of device tree node secmon (bl31_secmon_reserved) was
>>> incorrect. It should be increased to 2MiB (0x200000).
>>
>> 192K should work when the patch(6147314aeedc) was accepted.
>> Does the trusted-firmware-a get larger now, so we need to
>> increase the size to 2MiB?
>>
>> thanks,
>> Miles
>
>When mt8195-demo.dts sent to the upstream, at that time the size of
>BL31 was small. Because supported functions and modules in BL31 are
>basic sets when the board was under early development stage.
>
>Now BL31 includes more firmwares of coprocessors and maturer functions
>so the size has grown bigger in real applications. According to the
>value reported by customers, we think reserved 2MiB for BL31 might be
>enough for maybe the following 2 or 3 years.

Thanks for your explanation,
please add this to the commit message and submit next version.

With that, feel free to add:
Reviewed-by: Miles Chen <miles.chen@xxxxxxxxxxxx>

thanks,
Miles