[coreboot-gerrit] [XS] Change in coreboot[main]: soc/intel/xeon_sp: Allow Memory POR independent of RMT

[Martin L Roth (Code Review)]

Attention is currently required from: Arthur Heymans, Christian Walter, Johnny Lin, Jonathan Zhang, Lean Sheng Tan, Naresh Solanki, Patrick Rudolph, Tim Chu.

Martin L Roth has posted comments on this change by Naresh Solanki. ( https://review.coreboot.org/c/coreboot/+/84741?usp=email )

Change subject: soc/intel/xeon_sp: Allow Memory POR independent of RMT
......................................................................

Patch Set 3:

(1 comment)

File src/soc/intel/xeon_sp/spr/Kconfig:

https://review.coreboot.org/c/coreboot/+/84741/comment/ca8fe454_5e770e3e?usp... :
PS2, Line 177: MEM_POR_FREQ
This could be FORCE_MEM_POR.

FREQ isn't accurate since it enforces voltage as well, and when you see it in the code "MEM_POR_FREQ" doesn't tell you what it does. For all we can tell, it could mean "relax mem por".