Attention is currently required from: Arthur Heymans.
Julius Werner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/80639?usp=email )
Change subject: soc/qualcomm/sc7{1,2}80: Increase romstage/verstage section for clang ......................................................................
Patch Set 3:
(2 comments)
File src/soc/qualcomm/sc7180/memlayout.ld:
https://review.coreboot.org/c/coreboot/+/80639/comment/b0b678aa_ab824e33 : PS3, Line 27: REGION(qcsdi, 0x1469C000, 55K, 4K) You're actually adjusting this downward here, you don't need to do that. There's intentionally some space left so the romstage can be grown if needed.
File src/soc/qualcomm/sc7280/memlayout.ld:
https://review.coreboot.org/c/coreboot/+/80639/comment/7dac169c_beffbfb8 : PS3, Line 30: REGION(modem_id, 0x146A7000, 4, 4) Unfortunately modem_id is used as a communication buffer with an external blob and cannot be moved independently. I think the best thing you can do here is reduce qcsdi to the remaining size, and if anyone ever needs it again (which will likely be never) they'll have to deal with the potential fallout.