Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/46658 )
Change subject: soc/intel/xeon_sp: Use common cpu/intel romstage entry ......................................................................
Patch Set 2:
(1 comment)
https://review.coreboot.org/c/coreboot/+/46658/1/src/soc/intel/xeon_sp/memma... File src/soc/intel/xeon_sp/memmap.c:
https://review.coreboot.org/c/coreboot/+/46658/1/src/soc/intel/xeon_sp/memma... PS1, Line 42: *
nit: spacing around operators?
Done