Hello Aaron Durbin, Patrick Rudolph, Vanny E, build bot (Jenkins), Furquan Shaikh, David Guckian, Patrick Georgi, Martin Roth,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/34776
to look at the new patch set (#5).
Change subject: cpu/x86/smm: Define single smm_subregion() ......................................................................
cpu/x86/smm: Define single smm_subregion()
At the moment we only have two splitting of TSEG, one with and one without IED. They can all use same implementation.
Make configuration problems of TSEG region assertion failures.
Rename file from stage_cache.c to tseg_region.c to reflect it's purpose.
Change-Id: I9daf0dec8fbaaa1f4e6004ea034869f43412d7d5 Signed-off-by: Kyösti Mälkki kyosti.malkki@gmail.com --- M src/cpu/x86/smm/Makefile.inc D src/cpu/x86/smm/stage_cache.c A src/cpu/x86/smm/tseg_region.c M src/soc/amd/picasso/ramtop.c M src/soc/amd/stoneyridge/ramtop.c M src/soc/intel/apollolake/memmap.c M src/soc/intel/braswell/memmap.c M src/soc/intel/cannonlake/memmap.c M src/soc/intel/denverton_ns/memmap.c M src/soc/intel/icelake/memmap.c M src/soc/intel/skylake/memmap.c 11 files changed, 95 insertions(+), 339 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/76/34776/5