[M] Change in coreboot[main]: soc/intel/xeon_sp: Remove PAM lock and unlock operations

[Patrick Rudolph (Code Review)]

Attention is currently required from: Arthur Heymans, Christian Walter, Johnny Lin, Jonathan Zhang, Lean Sheng Tan, Shuo Liu, Tim Chu.

Patrick Rudolph has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/81349?usp=email )

Change subject: soc/intel/xeon_sp: Remove PAM lock and unlock operations
......................................................................

Patch Set 13:

(2 comments)

Commit Message:

https://review.coreboot.org/c/coreboot/+/81349/comment/3f957eb2_d48a3764 :
PS13, Line 11: some SoCs
which ones?

File src/soc/intel/xeon_sp/util.c:

https://review.coreboot.org/c/coreboot/+/81349/comment/992620ab_59962f40 :
PS5, Line 115: lock_pam0123
> In UEFI (in bootloader and in FSP), CPX/SKX need the lock, but SPR doesn't need nor do the lock. […]

This doesn't answer the question.
Does SPR FSP always lock it by default? If yes please update the commit message and made clear which FSP does what.
Reading FSP source it looks like LockChipset is always 0, thus SPR FSP would not lock it.