[XS] Change in coreboot[main]: intel/cmm/block: use CONFIG_SMM_TSEG_SIZE in sa_get_tseg_size function

[Wonkyu Kim (Code Review)]

Attention is currently required from: Anil Kumar K, Bora Guvendik, Jamie Ryu, Jeremy Compostella, Jérémy Compostella, Paul Menzel, Subrata Banik.

Wonkyu Kim has posted comments on this change by Wonkyu Kim. ( https://review.coreboot.org/c/coreboot/+/84210?usp=email )

Change subject: intel/cmm/block: use CONFIG_SMM_TSEG_SIZE in sa_get_tseg_size function
......................................................................

Patch Set 2:

(5 comments)

Commit Message:

https://review.coreboot.org/c/coreboot/+/84210/comment/47b7b21c_bf06b1b1?usp... :
PS2, Line 7: src/
> remove  `src/`

Done

https://review.coreboot.org/c/coreboot/+/84210/comment/8726788b_af7ed5a5?usp... :
PS2, Line 7: Update sa_get_tseg_size function
> Please be more specific.

Done

https://review.coreboot.org/c/coreboot/+/84210/comment/5deeb6c1_3607115f?usp... :
PS2, Line 10: api
> API

Done

https://review.coreboot.org/c/coreboot/+/84210/comment/46ebf1b5_53073bdd?usp... :
PS2, Line 10: nagative
> negative

Done

File src/soc/intel/common/block/systemagent/systemagent_early.c:

https://review.coreboot.org/c/coreboot/+/84210/comment/008e022d_a14fc1b6?usp... :
PS2, Line 155: 	return sa_get_gsm_base() - sa_get_tseg_base();
> WDYT? I assume in absence of IGD the GSM size would be zero hence, we won't get into `if` case. […]

We checked internally and got recommendation not to calculate it as CONFIG_SMM_TSEG_SIZE is passed to FSP and memory map which based on calculation can be changed later.