[coreboot-gerrit] Change in coreboot[master]: vc/intel/FSP2_0/CPX-SP: add FSPM DIMM watermark option

[Jonathan Zhang (Code Review)]

Jonathan Zhang has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/46480 )

Change subject: vc/intel/FSP2_0/CPX-SP: add FSPM DIMM watermark option
......................................................................

Patch Set 5:

(1 comment)

https://review.coreboot.org/c/coreboot/+/46480/5//COMMIT_MSG 
Commit Message:

https://review.coreboot.org/c/coreboot/+/46480/5//COMMIT_MSG@10 
PS5, Line 10: FSP binary, but not in Intel's CPX-SP FSP binary.
> Do we want to support such inconsistencies in coreboot? I think coreboot should only support one typ […]

We do not want to and we want to minimize the difference. That being said, this become unavoidable. Since Intel CPX-SP FSP binary is not released yet, and since FB is probably the only Intel customer who uses CPX-SP FSP binary, we could go with not adding a Kconfig, and not do conditional compilation at this time. But sooner or later we will need to deal with such situation.