Attention is currently required from: Arthur Heymans.
Jérémy Compostella has posted comments on this change by Arthur Heymans. ( https://review.coreboot.org/c/coreboot/+/84045?usp=email )
Change subject: arch/x86/bootblock.ld: Simplify the linker script ......................................................................
Patch Set 27:
(2 comments)
Patchset:
PS27: Is there any fundamental reason why you are doing that which could be documented in the commit message ?
File src/arch/x86/bootblock.ld:
https://review.coreboot.org/c/coreboot/+/84045/comment/dfd34e4d_1f041621?usp... : PS27, Line 32: 0xfffff000 The only thing I don't really like with this patch is that `0xfffff000` is used twice as a value which in my opinion calls for a variable.