Attention is currently required from: Yi Chou, Yu-Ping Wu.
Julius Werner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/79144?usp=email )
Change subject: libpayload: Fix the stack and data labels ......................................................................
Patch Set 3:
(2 comments)
File payloads/libpayload/arch/arm/libpayload.ldscript:
https://review.coreboot.org/c/coreboot/+/79144/comment/0e83aa12_4dab63a8 : PS3, Line 50: *(.rodata.*) Can we just add `_text`/`_etext`, `_rodata`/`_erodata` and `_bss`/`_ebss` everywhere here as well for consistency?
https://review.coreboot.org/c/coreboot/+/79144/comment/d7e88f1c_5a5b4d42 : PS3, Line 77: _stack = .; Be careful, these are actually used, so you need to fix the references in the relevant assembly files as well (the three `arch/<arch>/head.S`).