Hello build bot (Jenkins), Furquan Shaikh, Caveh Jalali, Tim Wawrzynczak, 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 (#38).
Change subject: lib/spd: respect spd memory part name override if present ......................................................................
lib/spd: respect spd memory part name override if present
The BIOS log was looking in the spd data for the part name, but part names are stripped from generic SPDs. In those cases, devices define their DRAM part name in the CBI.
Check for the existence of a mainboard part name override and if present, use the override name.
Add weak implementation of mainboard_get_dram_part_num() to spd_bin.c and remove all other weak implementaitons.
BUG=b:168724473 TEST=cherry-pick change to enable this on Tiger Lake, then execute "emerge-volteer coreboot chromeos-bootimage", flash and boot volteer to kernel 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, 26 insertions(+), 20 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/59/45459/38