diff options
author | Patrick Georgi <pgeorgi@google.com> | 2017-10-05 17:32:43 +0200 |
---|---|---|
committer | Patrick Georgi <pgeorgi@google.com> | 2017-10-06 16:59:20 +0000 |
commit | cba7316c2662ee1c916345b034ca08dad43984f7 (patch) | |
tree | 3a41c6c78ff9d0d1a67328c2f8cd336dd80ea205 /src/drivers | |
parent | c1ef5c1752cc440f4436968e54f8dd79f638951e (diff) |
soc/intel/common: refactor locate_vbt
All callers of locate_vbt just care about the file content and
immediately map the rdev for its content.
Instead of repeating this in all call sites, move that code to
locate_vbt.
Change-Id: I5b518e6c959437bd8f393269db7955358a786719
Signed-off-by: Patrick Georgi <pgeorgi@google.com>
Reviewed-on: https://review.coreboot.org/21896
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Aaron Durbin <adurbin@chromium.org>
Diffstat (limited to 'src/drivers')
-rw-r--r-- | src/drivers/intel/fsp2_0/graphics.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/src/drivers/intel/fsp2_0/graphics.c b/src/drivers/intel/fsp2_0/graphics.c index 84f2d6dc21..e2ff7cf4f8 100644 --- a/src/drivers/intel/fsp2_0/graphics.c +++ b/src/drivers/intel/fsp2_0/graphics.c @@ -99,8 +99,7 @@ uintptr_t fsp_load_vbt(void) struct region_device rdev; void *vbt_data = NULL; - if (locate_vbt(&rdev) != CB_ERR) - vbt_data = rdev_mmap_full(&rdev); + vbt_data = locate_vbt(&rdev); if (vbt_data == NULL) printk(BIOS_NOTICE, "Could not locate a VBT file in CBFS\n"); |