diff options
| author | Lee Leahy <leroy.p.leahy@intel.com> | 2016-08-01 15:47:42 -0700 |
|---|---|---|
| committer | Lee Leahy <leroy.p.leahy@intel.com> | 2016-08-03 06:16:16 +0200 |
| commit | 52d0c682bf3bb39584a1edd6e7326a6f4c1267f7 (patch) | |
| tree | ca7e32dcbffa4f5b9922327347d70209139be01b /src/drivers/intel/fsp2_0/memory_init.c | |
| parent | ac3b0a6e9f78cf7c4f2b32a6f97a42e7528aedd6 (diff) | |
drivers/intel/fsp2_0: Verify HOBs returned by FspMemoryInit
Verify that FSP is properly returning:
* HOB list pointer
* FSP_BOOTLOADER_TOLUM_HOB
* FSP_RESERVED_MEMORY_RESOURCE_HOB
TEST=Build and run on Galileo Gen2
Change-Id: I23005d10f7f3ccf06a2e29dab5fa11c7ed79f187
Signed-off-by: Lee Leahy <leroy.p.leahy@intel.com>
Reviewed-on: https://review.coreboot.org/15850
Tested-by: build bot (Jenkins)
Reviewed-by: Aaron Durbin <adurbin@chromium.org>
Diffstat (limited to 'src/drivers/intel/fsp2_0/memory_init.c')
| -rw-r--r-- | src/drivers/intel/fsp2_0/memory_init.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/drivers/intel/fsp2_0/memory_init.c b/src/drivers/intel/fsp2_0/memory_init.c index e55b9e5e0b..1e14df2746 100644 --- a/src/drivers/intel/fsp2_0/memory_init.c +++ b/src/drivers/intel/fsp2_0/memory_init.c @@ -68,7 +68,8 @@ static enum fsp_status do_fsp_post_memory_init(bool s3wake, struct range_entry fsp_mem; struct romstage_handoff *handoff; - fsp_find_reserved_memory(&fsp_mem); + if (fsp_find_reserved_memory(&fsp_mem)) + die("Failed to find FSP_RESERVED_MEMORY_RESOURCE_HOB!\n"); /* initialize cbmem by adding FSP reserved memory first thing */ if (!s3wake) { |