[S] Change in coreboot[master]: soc/intel/alderlake: Check MANUF_LOCK when logging manufacturing mode

[Sridhar Siricilla (Code Review)]

Attention is currently required from: Tarun Tuli, Michał Żygowski, Kangheui Won, Reka Norman, Rizwan Qureshi.

Sridhar Siricilla has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/69324 )

Change subject: soc/intel/alderlake: Check MANUF_LOCK when logging manufacturing mode
......................................................................

Patch Set 1:

(1 comment)

File src/soc/intel/alderlake/me.c:

https://review.coreboot.org/c/coreboot/+/69324/comment/c7c0265e_8381777f 
PS1, Line 141: hfsts1.fields.mfg_mode
> This bit only reflects the regions' writability in flash descriptor. […]

Originally, the bit is named as spi_protection since it indicates writability to flash descriptor. But, recently it got updated incorrectly. This has to be reverted to avoid the confusion.
> So basically fpf_soc_lock and manuf_lock should be the only bits to be checked, shouldn't they?

As part of staged EOM. these 3 conditions together reflect the EOM state. Hence, firmware has to check these 3 conditions.