Patrick Rudolph 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: Code-Review+1
(2 comments)
https://review.coreboot.org/c/coreboot/+/35799/2/src/mainboard/intel/glkrvp/... File src/mainboard/intel/glkrvp/variants/baseboard/devicetree.cb:
https://review.coreboot.org/c/coreboot/+/35799/2/src/mainboard/intel/glkrvp/... PS2, Line 100: register "sgx_enable" = "1" add select SOC_INTEL_COMMON_BLOCK_SGX_ENABLE to keep existing behaviour
https://review.coreboot.org/c/coreboot/+/35799/2/src/soc/intel/common/block/... File src/soc/intel/common/block/sgx/Kconfig:
https://review.coreboot.org/c/coreboot/+/35799/2/src/soc/intel/common/block/... PS2, Line 10: n default y?