[coreboot-gerrit] Change in coreboot[master]: arch/x86: Add option for using a custom assembly_entry

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

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

Change subject: arch/x86: Add option for using a custom assembly_entry
......................................................................

Patch Set 7:

(1 comment)

https://review.coreboot.org/c/coreboot/+/32414/7/src/arch/x86/memlayout.ld 
File src/arch/x86/memlayout.ld:

https://review.coreboot.org/c/coreboot/+/32414/7/src/arch/x86/memlayout.ld@3... 
PS7, Line 38: 	#include "car.ld"
As noted earlier, I would rather see all references to CAR removed early on. And if your verstage/vboot shall not run on x86 many of the sections do not need to be defined. The file is also the reason for keeping ill-named DCACHE_RAM_xx around.

I am leaving it to someone else to evaluate if this is clean enough for initial support. I would rather arrange it with preprocessor to replace every occurence of car.ld in this file.