Richard Spiegel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/29580 )
Change subject: nb/amd/amdmct/{mct,mct_ddr3}: Replace MTRR addresses with macros ......................................................................
Patch Set 2:
(1 comment)
https://review.coreboot.org/#/c/29580/2//COMMIT_MSG Commit Message:
https://review.coreboot.org/#/c/29580/2//COMMIT_MSG@9 PS2, Line 9: Replace numbers by macro Replace "magic" numbers with macros.
Also, normally the description should be longer then the title... add something else, maybe that the MTRR addresses are now publicly available?