Ting Shen has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31123 )
Change subject: WIP: bootmem: add new memory type for BL31 ......................................................................
Patch Set 6: Code-Review-1
(1 comment)
Patch Set 2:
Right, you need to actually update all the arm64 memlayouts to include this area, and you should also remove the corresponding mmio_resource() lines from SoC. Otherwise this looks good.
Is there any way to verify this? I can only test this on a mt8173 device I have, but each platform uses different approach to setup its reserved range.. (Or maybe it's not a good idea to break compatibility?)
https://review.coreboot.org/#/c/31123/2//COMMIT_MSG Commit Message:
https://review.coreboot.org/#/c/31123/2//COMMIT_MSG@8 PS2, Line 8:
Please elaborate, why this is needed.
Done