Subrata Banik has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/36542 )
Change subject: soc/intel/icelake: Make TSEG_BASE align to TSEG_SIZE ......................................................................
Patch Set 2:
(1 comment)
https://review.coreboot.org/c/coreboot/+/36542/1/src/soc/intel/icelake/smmre... File src/soc/intel/icelake/smmrelocate.c:
https://review.coreboot.org/c/coreboot/+/36542/1/src/soc/intel/icelake/smmre... PS1, Line 181: tseg_base = ALIGN(tseg_base, tseg_size);
Btw the cpu/intel file is used over many different targets and looks very similar to this code. It might be worth looking into making this or the gen1/ssmrelocate.c file common?
oh yes, someone is our team already started working on making this SMM related module common for soc/intel/common/block/smm. May be we will get rid of this in coming time soon.