diff options
author | Matt DeVillier <matt.devillier@gmail.com> | 2018-05-27 21:51:49 -0500 |
---|---|---|
committer | Martin Roth <martinroth@google.com> | 2018-05-31 15:28:12 +0000 |
commit | 9aaf59aa6dc428fe7c405c479ec008f75306a30c (patch) | |
tree | c0b817b9f73ade5d655db3a6f9e11bca9771309e /payloads/libpayload/drivers/timer | |
parent | a89b9cd4933e913d84ece773935bc27370429c1c (diff) |
soc/intel/broadwell: decouple PEI memory struct from coreboot header
Recent changes to field lengths in include/memory_info.h resulted in
a mismatch between the memory_info struct the MRC blob writes to and
the struct used by coreboot to parse out data for the SMBIOS tables.
This mismatch caused type 17 SMBIOS tables to be filled incorrectly.
The solution used here is to define the memory_info struct as expected
by MRC in the pei_data header, and manually copy the data field by field
into the coreboot memory_info struct, observing the more restrictive
lengths for the two structs.
Test: build/boot google/lulu, verify SMBIOS type 17 tables correctly
populated.
Change-Id: I932b7b41ae1e3fd364d056a8c91f7ed5d25dbafc
Signed-off-by: Matt DeVillier <matt.devillier@gmail.com>
Reviewed-on: https://review.coreboot.org/26598
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Nico Huber <nico.h@gmx.de>
Diffstat (limited to 'payloads/libpayload/drivers/timer')
0 files changed, 0 insertions, 0 deletions