[coreboot-gerrit] Change in coreboot[master]: soc/intel/sgx: convert SGX and PRMRR devicetree options to Kconfig

[Arthur Heymans (Code Review)]

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

Change subject: soc/intel/sgx: convert SGX and PRMRR devicetree options to Kconfig
......................................................................

Patch Set 33:

(1 comment)

https://review.coreboot.org/c/coreboot/+/35799/5/src/mainboard/intel/glkrvp/... 
File src/mainboard/intel/glkrvp/Kconfig:

https://review.coreboot.org/c/coreboot/+/35799/5/src/mainboard/intel/glkrvp/... 
PS5, Line 89: config SOC_INTEL_COMMON_BLOCK_SGX_ENABLE
            : 	bool
            : 	default y
> Sorry, I don't get what you mean. sgx was enabled in the devicetree (yes, mb specific) before. now it gets enabled via the new Kconfig setting in the boards Kconfig (the mb specific one).
> I don't want to change the default behaviour as I don't know if SGX is working on all other boards.

Not sure what my train of thought was here. Is enabling SGX something that should be done by default on this platform or not? anyway... go ahead.