[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 7: Code-Review+2

(1 comment)

https://review.coreboot.org/c/coreboot/+/34809/7/src/arch/x86/include/arch/c... 
File src/arch/x86/include/arch/cpu.h:

https://review.coreboot.org/c/coreboot/+/34809/7/src/arch/x86/include/arch/c... 
PS7, Line 305: 	int skip_common_mtrr;
I do think we could have just done with opposite logic -- add_common_mtrr and then hidden postcar_frame_common_mtrrs() within the compilation unit and called it from run_postcar_phase(). However, that reduces the API surface area a bit and no callers care but the ones you are touching here. Let's see how this plays out, though.