[coreboot-gerrit] Change in coreboot[master]: arch/x86: Add postcar_frame_common_mtrrs()

[Aaron Durbin (Code Review)]

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

Change subject: arch/x86: Add postcar_frame_common_mtrrs()
......................................................................

Patch Set 2:

(1 comment)

https://review.coreboot.org/c/coreboot/+/34809/2/src/cpu/intel/car/romstage.... 
File src/cpu/intel/car/romstage.c:

https://review.coreboot.org/c/coreboot/+/34809/2/src/cpu/intel/car/romstage.... 
PS2, Line 36: 	if (!pcf->skip_common_mtrr)
I don't think we should be making these decisions here. This field should be checked in postcar_frame_common_mtrrs(). Or why not do this inside run_postcar_phase() and make the field inverted (fill common mtrrs).