Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/48657 )
Change subject: soc/intel/xeon_sp: Move DMICTL lock ......................................................................
Patch Set 1: Code-Review+2
(1 comment)
https://review.coreboot.org/c/coreboot/+/48657/1/src/soc/intel/xeon_sp/cpx/c... File src/soc/intel/xeon_sp/cpx/chip.c:
https://review.coreboot.org/c/coreboot/+/48657/1/src/soc/intel/xeon_sp/cpx/c... PS1, Line 82: pch_lock_dmictl();
Could this be moved to chip_final() instead? […]
Ack