[coreboot-gerrit] Change in coreboot[master]: mb/ocp/deltalake: Override DDR frequency limit via VPD variable

[Tim Chu (Code Review)]

Tim Chu has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/48946 )

Change subject: mb/ocp/deltalake: Override DDR frequency limit via VPD variable
......................................................................

Patch Set 4:

(2 comments)

https://review.coreboot.org/c/coreboot/+/48946/2/src/mainboard/ocp/deltalake... 
File src/mainboard/ocp/deltalake/romstage.c:

https://review.coreboot.org/c/coreboot/+/48946/2/src/mainboard/ocp/deltalake... 
PS2, Line 85: 	if (vpd_gets(FSP_DIMM_FREQ, val_str, VPD_LEN, VPD_RW_THEN_RO)) {
> Please refer to [CB:48908] and use vpd_get_int().

Done

https://review.coreboot.org/c/coreboot/+/48946/3/src/mainboard/ocp/deltalake... 
File src/mainboard/ocp/deltalake/romstage.c:

https://review.coreboot.org/c/coreboot/+/48946/3/src/mainboard/ocp/deltalake... 
PS3, Line 82: unsigned int
> DdrFreqLimit is UINT8, I think uint8_t instead of unsigned int is better. […]

Done