mturney mturney has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39362 )
Change subject: sc7180: Adjust memory allocations per upstream comments ......................................................................
Patch Set 4:
(2 comments)
https://review.coreboot.org/c/coreboot/+/39362/3/src/soc/qualcomm/sc7180/inc... File src/soc/qualcomm/sc7180/include/soc/memlayout.ld:
https://review.coreboot.org/c/coreboot/+/39362/3/src/soc/qualcomm/sc7180/inc... PS3, Line 65: 0x300000
Saurabh said that we don't need the TZ_STAT region for coreboot builds, so can we reduce this to 2MB […]
Will confirm that ATF team is good to move bl31 down to 0x80B00000.
https://review.coreboot.org/c/coreboot/+/39362/3/src/soc/qualcomm/sc7180/inc... PS3, Line 66: 2M
Comments on the bug said 1MB, any reason why it needs to be 2 now? (Remember we can always update th […]
Version of bl31 Google has requires 2MB. When new version is available that reduces hard-coded constant to 1M, this will be updated to only use 1M. We didn't want to break your builds.