[coreboot-gerrit] Change in coreboot[master]: soc/intel/sgx: make SGX a Kconfig option

[Arthur Heymans (Code Review)]

Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35799 )

Change subject: soc/intel/sgx: make SGX a Kconfig option
......................................................................

Patch Set 2:

(1 comment)

https://review.coreboot.org/c/coreboot/+/35799/2/src/soc/intel/apollolake/me... 
File src/soc/intel/apollolake/memmap.c:

https://review.coreboot.org/c/coreboot/+/35799/2/src/soc/intel/apollolake/me... 
PS2, Line 41: PrmrrSize
I find it weird that this is an FSP-M parameter, but sgx_enable isn't. So the prmrrSize region is set up by FSP-M but is just ram unless sgx is enabled?

I have no knowledge on either but this deserves an explanation in comments.