Attention is currently required from: Julius Werner, Paul Menzel.
Yi Chou has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/79800?usp=email )
Change subject: libpayload: Move back the ttb_buffer section ......................................................................
Patch Set 5:
(5 comments)
Commit Message:
https://review.coreboot.org/c/coreboot/+/79800/comment/ca897162_14ea6332 : PS3, Line 9: If the move it into the .ttb_buffer section
Sorry, I do not understand this part. […]
Done
https://review.coreboot.org/c/coreboot/+/79800/comment/c6a05fd3_8f3d1189 : PS3, Line 10: We should move
So, move it back …
Done
https://review.coreboot.org/c/coreboot/+/79800/comment/8dd337e5_356a1679 : PS3, Line 13: Boot the device.
It didn’t boot before?
Rephrased.
File payloads/libpayload/arch/arm64/libpayload.ldscript:
https://review.coreboot.org/c/coreboot/+/79800/comment/1e27f13e_aafce608 : PS3, Line 65: *(.bss.ttb_buffer)
I think you might as well remove this completely. […]
Done
File payloads/libpayload/arch/arm64/mmu.c:
https://review.coreboot.org/c/coreboot/+/79800/comment/27fdd667_c8f09904 : PS3, Line 45: /* Don't chage the name of this variable without also changing the name in the
(Would need to change this text a bit then to explain that we refer to this in the linker script for […]
Done