[coreboot-gerrit] Change in coreboot[master]: nb/amd/agesa: read 256 bytes to SPD buffer instead of 128

[Michał Żygowski (Code Review)]

Michał Żygowski has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/40484 )

Change subject: nb/amd/agesa: read 256 bytes to SPD buffer instead of 128
......................................................................

Patch Set 1:

(1 comment)

https://review.coreboot.org/c/coreboot/+/40484/1/src/northbridge/amd/agesa/f... 
File src/northbridge/amd/agesa/family14/dimmSpd.c:

https://review.coreboot.org/c/coreboot/+/40484/1/src/northbridge/amd/agesa/f... 
PS1, Line 44: 	int err = smbus_readSpd(spdAddress, (void *) info->Buffer, 256);
Shouldn't it be tied to CONFIG_DIMM_SPD_SIZE? If you would like to use XMP profile, you may add a Kconfig option to enable XMP profiles which will modify the DIMM_SPD_SIZE for your boards.