Change in coreboot[master]: soc/intel/xeon_sp/cpx: add GUID and name for more HOBs

[Jonathan Zhang (Code Review)]

Jonathan Zhang has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/43337 )

Change subject: soc/intel/xeon_sp/cpx: add GUID and name for more HOBs
......................................................................

soc/intel/xeon_sp/cpx: add GUID and name for more HOBs

Change-Id: I46637839452d6f7fba7453c19f18efef338f2ad6
---
M src/soc/intel/xeon_sp/cpx/hob_display.c
M src/vendorcode/intel/fsp/fsp2_0/cooperlake_sp/hob_memmap.h
2 files changed, 8 insertions(+), 1 deletion(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/37/43337/1

diff --git a/src/soc/intel/xeon_sp/cpx/hob_display.c b/src/soc/intel/xeon_sp/cpx/hob_display.c
index 3d61a82..2865fa2 100644
--- a/src/soc/intel/xeon_sp/cpx/hob_display.c
+++ b/src/soc/intel/xeon_sp/cpx/hob_display.c
@@ -8,8 +8,14 @@
 #include <lib.h>
 #include <soc/soc_util.h>

+#define FSP_PREV_BOOT_ERR_HOB_GUID { \
+  0xc5, 0xb5, 0x38, 0x51, 0x69, 0x93, 0xec, 0x48, \
+  0x5b, 0x97, 0x38, 0xa2, 0xf7, 0x09, 0x66, 0x75  \
+}
+
 static const uint8_t fsp_hob_iio_uds_guid[16] = FSP_HOB_IIO_UNIVERSAL_DATA_GUID;
 static const uint8_t fsp_hob_memmap_guid[16] = FSP_SYSTEM_MEMORYMAP_HOB_GUID;
+static const uint8_t fsp_prev_boot_err_hob_guid[16] = FSP_PREV_BOOT_ERR_HOB_GUID;

struct guid_name_map {
    const void *guid;
@@ -19,6 +25,7 @@
 static const struct guid_name_map  guid_names[] = {
    { fsp_hob_iio_uds_guid, "FSP_HOB_IIO_UNIVERSAL_DATA_GUID" },
    { fsp_hob_memmap_guid, "FSP_SYSTEM_MEMORYMAP_HOB_GUID" },
+	{ fsp_prev_boot_err_hob_guid, "FSP_PREV_BOOT_ERR_HOB_GUID" },
 };

const char *soc_get_guid_name(const uint8_t *guid)
diff --git a/src/vendorcode/intel/fsp/fsp2_0/cooperlake_sp/hob_memmap.h b/src/vendorcode/intel/fsp/fsp2_0/cooperlake_sp/hob_memmap.h
index 51aa051..cbb81a6 100644
--- a/src/vendorcode/intel/fsp/fsp2_0/cooperlake_sp/hob_memmap.h
+++ b/src/vendorcode/intel/fsp/fsp2_0/cooperlake_sp/hob_memmap.h
@@ -96,7 +96,7 @@
   UINT32   memSize;                               // Total physical memory size
   UINT16   memFreq;                               // Mem Frequency

-  UINT8    reserved2[61];
+  UINT8    reserved2[62];

UINT8    NumChPerMC;
   UINT8    numberEntries;                         // Number of Memory Map Elements

[Jonathan Zhang (Code Review)]

Jonathan Zhang has abandoned this change. ( https://review.coreboot.org/c/coreboot/+/43337 )

Change subject: soc/intel/xeon_sp/cpx: add GUID and name for more HOBs
......................................................................

Abandoned

More work is in the way.