[coreboot-gerrit] Change in coreboot[master]: src/drivers/intel/fsp1_1/raminit.c: Make check FSP HOBs independent o...

[Frans Hendriks (Code Review)]

Frans Hendriks has uploaded this change for review. ( https://review.coreboot.org/29371

Change subject: src/drivers/intel/fsp1_1/raminit.c: Make check FSP HOBs independent of CONFIG_DISPLAY_HOBS.
......................................................................

src/drivers/intel/fsp1_1/raminit.c: Make check FSP HOBs independent of CONFIG_DISPLAY_HOBS.

Check for FSP HOBS is disabled when CONFIG_DISPLAY_HOBS is disabled.
Use the CONFIG_DISPLAY_HOBS for determination of display HOB info only.

BUG=N/A
TEST=Intel CherryHill CRB

Change-Id: I3776fa37866c7ef3aea090842387660c22bbdd4d
Signed-off-by: Frans Hendriks 
---
M src/drivers/intel/fsp1_1/raminit.c
1 file changed, 2 insertions(+), 5 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/71/29371/1

diff --git a/src/drivers/intel/fsp1_1/raminit.c b/src/drivers/intel/fsp1_1/raminit.c
index c7d8722..3adced2 100644
--- a/src/drivers/intel/fsp1_1/raminit.c
+++ b/src/drivers/intel/fsp1_1/raminit.c
@@ -50,10 +50,8 @@
    VPD_DATA_REGION *vpd_ptr;
    UPD_DATA_REGION *upd_ptr;
    int fsp_verification_failure = 0;
-#if IS_ENABLED(CONFIG_DISPLAY_HOBS)
    unsigned long int data;
    EFI_PEI_HOB_POINTERS hob_ptr;
-#endif

/*
     * Find and copy the UPD region to the stack so the platform can modify
@@ -193,7 +191,6 @@
    		memory_info_hob);
    }

-#if IS_ENABLED(CONFIG_DISPLAY_HOBS)
    if (hob_list_ptr == NULL)
    	die("ERROR - HOB pointer is NULL!\n");

@@ -241,8 +238,8 @@
    		"ERROR - Missing one or more required FSP HOBs!\n");

/* Display the HOBs */
-	print_hob_type_structure(0, hob_list_ptr);
-#endif
+	if (IS_ENABLED(CONFIG_DISPLAY_HOBS))
+		print_hob_type_structure(0, hob_list_ptr);

/* Get the address of the CBMEM region for the FSP reserved memory */
    fsp_reserved_memory_area = cbmem_find(CBMEM_ID_FSP_RESERVED_MEMORY);