[coreboot-gerrit] Change in coreboot[master]: soc/amd/non_car/memlayout_x86.ld: Top aling the bootblock

[Arthur Heymans (Code Review)]

Attention is currently required from: Jason Glenesk, Raul Rangel, Marshall Dawson, Felix Held.
Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/55796 )

Change subject: soc/amd/non_car/memlayout_x86.ld: Top aling the bootblock
......................................................................

Patch Set 1:

(2 comments)

File src/soc/amd/common/block/cpu/noncar/memlayout_x86.ld:

https://review.coreboot.org/c/coreboot/+/55796/comment/e1d349b5_24864173 
PS1, Line 115: _bootblock
> Hrmm, what would it take to use arch/x86/bootblock.ld? It looks like we just need a KCONFIG to remove the .id/.fit pointer

If there is nothing in those sections that should work too, so only conditionally include id.S. Fit has already been moved out.

https://review.coreboot.org/c/coreboot/+/55796/comment/e537ee8f_d98a9597 
PS1, Line 124: 64
> Why the extra 64?

ALIGN(x) aligns downwards but we are aligning the bootblock to the top, so some extra bytes appended help.