[coreboot-gerrit] New patch to review for coreboot: drivers/intel/fsp2_0: Fix debug display support

[Lee Leahy (leroy.p.leahy@intel.com)]

Lee Leahy (leroy.p.leahy at intel.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/16809

-gerrit

commit 122f6ea1803cdbfcb75d2576459515170ccb9e50
Author: Lee Leahy <leroy.p.leahy at intel.com>
Date:   Wed Sep 28 17:15:00 2016 -0700

    drivers/intel/fsp2_0: Fix debug display support
    
    Fix errors in debug display support.
    
    BRANCH=none
    BUG=None
    TEST=Build FSP 2.0 (SEC/PEI core with all FSP debug on) and run on
    Galileo Gen2
    
    Change-Id: I2ece056d66dc8568a7b7206970f20368ec5bf147
    Signed-off-by: Lee Leahy <Leroy.P.Leahy at intel.com>
---
 src/drivers/intel/fsp2_0/graphics.c         |  5 -----
 src/drivers/intel/fsp2_0/hand_off_block.c   | 11 -----------
 src/drivers/intel/fsp2_0/hob_display.c      |  7 ++++++-
 src/drivers/intel/fsp2_0/include/fsp/util.h | 11 +++++++++++
 src/drivers/intel/fsp2_0/upd_display.c      | 10 +++++-----
 5 files changed, 22 insertions(+), 22 deletions(-)

diff --git a/src/drivers/intel/fsp2_0/graphics.c b/src/drivers/intel/fsp2_0/graphics.c
index caf7bfa..bc1deb5 100644
--- a/src/drivers/intel/fsp2_0/graphics.c
+++ b/src/drivers/intel/fsp2_0/graphics.c
@@ -20,11 +20,6 @@ enum pixel_format {
 	pixel_bitmask = 2,		/* defined by <rgb>_mask values */
 };
 
-const uint8_t fsp_graphics_info_guid[16] = {
-	0xce, 0x2c, 0xf6, 0x39, 0x25, 0x68, 0x69, 0x46,
-	0xbb, 0x56, 0x54, 0x1a, 0xba, 0x75, 0x3a, 0x07
-};
-
 struct hob_graphics_info {
 	uint64_t framebuffer_base;
 	uint32_t framebuffer_size;
diff --git a/src/drivers/intel/fsp2_0/hand_off_block.c b/src/drivers/intel/fsp2_0/hand_off_block.c
index 17adf46..a8b9878 100644
--- a/src/drivers/intel/fsp2_0/hand_off_block.c
+++ b/src/drivers/intel/fsp2_0/hand_off_block.c
@@ -22,17 +22,6 @@
 
 #define HOB_HEADER_LEN		8
 
-enum resource_type {
-	EFI_RESOURCE_SYSTEM_MEMORY		= 0,
-	EFI_RESOURCE_MEMORY_MAPPED_IO		= 1,
-	EFI_RESOURCE_IO				= 2,
-	EFI_RESOURCE_FIRMWARE_DEVICE		= 3,
-	EFI_RESOURCE_MEMORY_MAPPED_IO_PORT	= 4,
-	EFI_RESOURCE_MEMORY_RESERVED		= 5,
-	EFI_RESOURCE_IO_RESERVED		= 6,
-	EFI_RESOURCE_MAX_MEMORY_TYPE		= 7,
-};
-
 /* GUIDs in little-endian, so they can be used with memcmp() */
 const uint8_t fsp_bootloader_tolum_guid[16] = {
 	0x56, 0x4f, 0xff, 0x73, 0x8e, 0xaa, 0x51, 0x44,
diff --git a/src/drivers/intel/fsp2_0/hob_display.c b/src/drivers/intel/fsp2_0/hob_display.c
index 39a9d17..6e6a915 100644
--- a/src/drivers/intel/fsp2_0/hob_display.c
+++ b/src/drivers/intel/fsp2_0/hob_display.c
@@ -51,6 +51,11 @@ static const uint8_t empty_guid[16] = {
 	0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0
 };
 
+const uint8_t fsp_graphics_info_guid[16] = {
+	0xce, 0x2c, 0xf6, 0x39, 0x25, 0x68, 0x69, 0x46,
+	0xbb, 0x56, 0x54, 0x1a, 0xba, 0x75, 0x3a, 0x07
+};
+
 static const uint8_t fsp_info_header_guid[16] = {
 	0xbe, 0x40, 0x27, 0x91, 0x84, 0x22, 0x34, 0x47,
 	0xb9, 0x71, 0x84, 0xb0, 0x27, 0x35, 0x3f, 0x0c
@@ -78,7 +83,7 @@ static const struct guid_name_map  guid_names[] = {
 	{ fsp_info_header_guid,		"FSP_INFO_HEADER_GUID" },
 	{ fsp_reserved_memory_guid,	"FSP_RESERVED_MEMORY" },
 	{ fsp_nv_storage_guid,		"FSP_NV_STORAGE" },
-	{ graphics_info_guid,		"GRAPHICS INFO" },
+	{ fsp_graphics_info_guid,	"GRAPHICS INFO" },
 	{ smbios_memory_info_guid,	"FSP_SMBIOS_MEMORY_INFO_GUID" },
 	{ tseg_guid,			"TSEG" },
 };
diff --git a/src/drivers/intel/fsp2_0/include/fsp/util.h b/src/drivers/intel/fsp2_0/include/fsp/util.h
index acb41e0..5f08856 100644
--- a/src/drivers/intel/fsp2_0/include/fsp/util.h
+++ b/src/drivers/intel/fsp2_0/include/fsp/util.h
@@ -37,6 +37,17 @@ struct hob_resource {
 	uint64_t length;
 } __attribute__((packed));
 
+enum resource_type {
+	EFI_RESOURCE_SYSTEM_MEMORY		= 0,
+	EFI_RESOURCE_MEMORY_MAPPED_IO		= 1,
+	EFI_RESOURCE_IO				= 2,
+	EFI_RESOURCE_FIRMWARE_DEVICE		= 3,
+	EFI_RESOURCE_MEMORY_MAPPED_IO_PORT	= 4,
+	EFI_RESOURCE_MEMORY_RESERVED		= 5,
+	EFI_RESOURCE_IO_RESERVED		= 6,
+	EFI_RESOURCE_MAX_MEMORY_TYPE		= 7,
+};
+
 enum hob_type {
 	HOB_TYPE_HANDOFF			= 0x0001,
 	HOB_TYPE_MEMORY_ALLOCATION		= 0x0002,
diff --git a/src/drivers/intel/fsp2_0/upd_display.c b/src/drivers/intel/fsp2_0/upd_display.c
index a588358..1ef657d 100644
--- a/src/drivers/intel/fsp2_0/upd_display.c
+++ b/src/drivers/intel/fsp2_0/upd_display.c
@@ -19,15 +19,15 @@ void fsp_display_upd_value(const char *name, size_t size, uint64_t old,
 {
 	size *= 2;
 	if (old == new) {
-		printk(BIOS_SPEW, "  0x%0*llx: %s\n", size, new, name);
+		printk(BIOS_SPEW, "  0x%0*llx: %s\n", (int)size, new, name);
 	} else {
-		printk(BIOS_SPEW, "  0x%0*llx --> 0x%0*llx: %s\n", size, old,
-			size, new, name);
+		printk(BIOS_SPEW, "  0x%0*llx --> 0x%0*llx: %s\n", (int)size, old,
+			(int)size, new, name);
 	}
 }
 
-static void fspm_display_arch_params(const struct FSPM_ARCH_UPD *old,
-	const struct FSPM_ARCH_UPD *new)
+static void fspm_display_arch_params(const FSPM_ARCH_UPD *old,
+	const FSPM_ARCH_UPD *new)
 {
 	/* Display the architectural parameters for MemoryInit */
 	printk(BIOS_SPEW, "Architectural UPD values for MemoryInit at: 0x%p\n",