[coreboot-gerrit] Change in coreboot[master]: soc/intel/skylake: use cpu_lt_lock_memory in cpu_lock_sgx_memory

[Michael Niewöhner (Code Review)]

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

Change subject: soc/intel/skylake: use cpu_lt_lock_memory in cpu_lock_sgx_memory
......................................................................

Patch Set 1:

(1 comment)

https://review.coreboot.org/c/coreboot/+/36354/1/src/soc/intel/skylake/cpu.c 
File src/soc/intel/skylake/cpu.c:

https://review.coreboot.org/c/coreboot/+/36354/1/src/soc/intel/skylake/cpu.c... 
PS1, Line 579: void cpu_lock_sgx_memory(void)
> Is this API still needed after the next patch?

Yep, because when SGX is enabled the lock *MUST* happen before SGX init; every platform after skl/kbl does not need to do the lock themselves, which is handled there MCHECK