Attention is currently required from: Kapil Porwal.
Pratikkumar V Prajapati has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/71569 )
Change subject: soc/intel/common: Check PRMRR dependent features ......................................................................
Patch Set 11:
(2 comments)
Commit Message:
https://review.coreboot.org/c/coreboot/+/71569/comment/5fa2d0b6_9692f32e PS10, Line 7: dd helper functions for SGX, Key Locker, and PRM
The primary purpose of this CL seems to be setting PRMRR to 0 if all PRM features are disabled.
updated commit msg
File src/soc/intel/common/block/cpu/cpulib.c:
https://review.coreboot.org/c/coreboot/+/71569/comment/f1f262eb_098bb4af PS10, Line 436: configured
Sounds misleading to me. 'configured' vs 'CONFIG enabled'.
Does not seem misleading but do you have any better recommendation? I can change the function names.
-
Pratikkumar V Prajapati (Code Review)