Marc Jones has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/46640 )
Change subject: vc/intel/FSP2_0/CPX-SP: update to ww34 release and add watermark option ......................................................................
Patch Set 2:
(1 comment)
https://review.coreboot.org/c/coreboot/+/46640/2/src/mainboard/ocp/deltalake... File src/mainboard/ocp/deltalake/romstage.c:
https://review.coreboot.org/c/coreboot/+/46640/2/src/mainboard/ocp/deltalake... PS2, Line 80: UnusedUpdSpace0 If there is no VPD setting,would it be safer to do nothing? I would expect the FSP to do something sane if this isn't set.