Hung-Te Lin has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/46401 )
Change subject: soc/mediatek/mt8192: Reserved 44K SRAM for mcupm working buffer ......................................................................
Patch Set 1:
(1 comment)
https://review.coreboot.org/c/coreboot/+/46401/1/src/soc/mediatek/mt8192/inc... File src/soc/mediatek/mt8192/include/soc/memlayout.ld:
https://review.coreboot.org/c/coreboot/+/46401/1/src/soc/mediatek/mt8192/inc... PS1, Line 29: SRAM_END(0x00115000) The SRAM_END should be indicating real end of SRAM. You should preserve a new section instead of leaving that undefined.
Also, there's no need to move TIMESTAMP, STACK, ... etc. We can just define a new region before or after PRERAM_CBMEM_CONSOLE.