Hello build bot (Jenkins), Furquan Shaikh, Patrick Rudolph,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/45459
to look at the new patch set (#58).
Change subject: lib/spd: respect spd memory part name override ......................................................................
lib/spd: respect spd memory part name override
The BIOS log was looking in the spd data for the part name, but part names are stripped from generic SPDs. For these cases, a mainboard can override the dram part number string, so the spd logging code needs to check for an override string when logging the dram part number.
Change print_spd_info() to use an override string if declared.
BUG=b:168724473 TEST="emerge-volteer coreboot chromeos-bootimage", flash and boot volteer2 and verify that the BIOS log shows a part name when logging SPD information:
SPD: module part number is K4U6E3S4AA-MGCL
I also modified volteer to not override the part name and verified that this change did as expected and printed a blank string.
Change-Id: I91971e07c450492dbb0588abd1c3c692ee0d3bb0 Signed-off-by: Nick Vaccaro nvaccaro@google.com --- M src/include/spd_bin.h M src/lib/spd_bin.c M src/soc/intel/cannonlake/cnl_memcfg_init.c M src/soc/intel/elkhartlake/meminit.c M src/soc/intel/jasperlake/meminit.c M src/soc/intel/tigerlake/meminit.c 6 files changed, 29 insertions(+), 22 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/59/45459/58