[coreboot-gerrit] Change in coreboot[master]: arch/x86: Simplify <arch/early_variables.h>

[Kyösti Mälkki (Code Review)]

Kyösti Mälkki has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35016 )

Change subject: arch/x86: Simplify <arch/early_variables.h>
......................................................................

Patch Set 5:

(1 comment)

https://review.coreboot.org/c/coreboot/+/35016/5/src/lib/program.ld 
File src/lib/program.ld:

https://review.coreboot.org/c/coreboot/+/35016/5/src/lib/program.ld@141 
PS5, Line 141: #if ENV_STAGE_HAS_HEAP_SECTION && !ENV_CACHE_AS_RAM
> nit: Isn't this one unnecessary?

If we did add .heap inside CAR, this becomes equally necessary as the .bss case above. Then again, for consistency STAGE_HAS_DATA should have this added too? We can go either way.