[coreboot-gerrit] Change in coreboot[master]: arch/x86: Simplified postcar WB MTRR setup

[Aaron Durbin (Code Review)]

Aaron Durbin has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/34897 )

Change subject: arch/x86: Simplified postcar WB MTRR setup
......................................................................

Patch Set 1:

(1 comment)

https://review.coreboot.org/c/coreboot/+/34897/1/src/soc/intel/skylake/memma... 
File src/soc/intel/skylake/memmap.c:

https://review.coreboot.org/c/coreboot/+/34897/1/src/soc/intel/skylake/memma... 
PS1, Line 317: 	/* Do some magic alignments */
Why are we doing anything magic? FWIW, we can't lay down a MTRR over potential holes needed in the memory map.