Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/44240 )
Change subject: cpu/intel/model_6xx: Align cosmetics ......................................................................
Patch Set 5:
(1 comment)
https://review.coreboot.org/c/coreboot/+/44240/5/src/cpu/intel/model_6xx/mod... File src/cpu/intel/model_6xx/model_6xx_init.c:
https://review.coreboot.org/c/coreboot/+/44240/5/src/cpu/intel/model_6xx/mod... PS5, Line 18: /* Setup MTRRs */
I'd remove this form the other files.
I'd say this is a matter of personal preference, and I wouldn't want to re-do the patch train again just to change this. Plus, Keith (or someone else with this hardware) still needs to test a few things.
Maybe we can handle this once the bottom half of this patch train (yes, these commits without any reviews) is merged?