>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
