[coreboot-gerrit] Change in coreboot[master]: soc/intel/common/block/cpu: Fix boot failure

[Michael Niewöhner (Code Review)]

Michael Niewöhner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/45344 )

Change subject: soc/intel/common/block/cpu: Fix boot failure
......................................................................

Patch Set 4:

(1 comment)

https://review.coreboot.org/c/coreboot/+/45344/4/src/soc/intel/common/block/... 
File src/soc/intel/common/block/cpu/cpulib.c:

https://review.coreboot.org/c/coreboot/+/45344/4/src/soc/intel/common/block/... 
PS4, Line 349: SOC_INTEL_COMMON_BLOCK_SGX
> Checking for SOC_INTEL_COMMON_BLOCK_SGX_PRMRR_DISABLED is a bad idea anyway, since it defaults to 0.

I don't get what you mean. That option was a bool, not the actual selected size. However, I agree that we should check for SGX feature and enablement