diff options
author | Nick Vaccaro <nvaccaro@google.com> | 2020-09-16 17:08:00 -0700 |
---|---|---|
committer | Nick Vaccaro <nvaccaro@google.com> | 2020-10-09 07:34:50 +0000 |
commit | 88d4e82b33e5b47cc5b19d5f516272ff03a41546 (patch) | |
tree | 8bd5004380a8f5e8ed45536dbfb2a1450214a346 /util/msrtool/msrtool.c | |
parent | 93d483db89da3554afe01958e9f80a76251f2477 (diff) |
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>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/45459
Reviewed-by: Furquan Shaikh <furquan@google.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'util/msrtool/msrtool.c')
0 files changed, 0 insertions, 0 deletions