summaryrefslogtreecommitdiff
path: root/src/drivers/intel
diff options
context:
space:
mode:
authorWim Vervoorn <wvervoorn@eltan.com>2020-01-02 11:52:59 +0100
committerNico Huber <nico.h@gmx.de>2020-01-10 14:48:22 +0000
commit9f363fc85aef290d20f7102a16af0df274beadd8 (patch)
tree05fe9a7c79f60a737593ab1238b47280906945e4 /src/drivers/intel
parent90a9db537b68ebb85deab0eaa2f5f19467532ee2 (diff)
downloadcoreboot-9f363fc85aef290d20f7102a16af0df274beadd8.tar.xz
drivers/intel/fsp2_0/logo.c: Correct check for logo_size
The check to validate if the logo file was loaded correctly was incorrect. Now check the actual logo size. BUG=N/A TEST=build Change-Id: Ib3a808dd831986e8347512892ee88983d376d34c Signed-off-by: Wim Vervoorn <wvervoorn@eltan.com> Reviewed-on: https://review.coreboot.org/c/coreboot/+/38124 Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net> Reviewed-by: Nico Huber <nico.h@gmx.de> Reviewed-by: Frans Hendriks <fhendriks@eltan.com> Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'src/drivers/intel')
-rw-r--r--src/drivers/intel/fsp2_0/logo.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/src/drivers/intel/fsp2_0/logo.c b/src/drivers/intel/fsp2_0/logo.c
index ba2b5dcbcd..1a9152f97b 100644
--- a/src/drivers/intel/fsp2_0/logo.c
+++ b/src/drivers/intel/fsp2_0/logo.c
@@ -26,7 +26,7 @@ const struct cbmem_entry *fsp_load_logo(UINT32 *logo_ptr, UINT32 *logo_size)
if (logo_buffer) {
*logo_size = cbfs_boot_load_file("logo.bmp", (void *)logo_buffer,
1 * MiB, CBFS_TYPE_RAW);
- if (logo_size)
+ if (*logo_size)
*logo_ptr = (UINT32)logo_buffer;
}
}