diff options
| author | Wim Vervoorn <wvervoorn@eltan.com> | 2020-01-02 11:51:48 +0100 |
|---|---|---|
| committer | Nico Huber <nico.h@gmx.de> | 2020-01-10 14:48:16 +0000 |
| commit | 90a9db537b68ebb85deab0eaa2f5f19467532ee2 (patch) | |
| tree | f056a2ec16d35303e5c4028f3e999df4b4b955c9 /src/drivers/intel | |
| parent | e6ac20b9d019a90ecaf330cebba94a1fe690db62 (diff) | |
| download | coreboot-90a9db537b68ebb85deab0eaa2f5f19467532ee2.tar.xz | |
drivers/intel/fsp1_1/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: I4df2076b2f0cc371848a912c622268dfec24e2ef
Signed-off-by: Wim Vervoorn <wvervoorn@eltan.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/38123
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Frans Hendriks <fhendriks@eltan.com>
Reviewed-by: Nico Huber <nico.h@gmx.de>
Diffstat (limited to 'src/drivers/intel')
| -rw-r--r-- | src/drivers/intel/fsp1_1/logo.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/drivers/intel/fsp1_1/logo.c b/src/drivers/intel/fsp1_1/logo.c index 23aad01c1b..b00406d2bd 100644 --- a/src/drivers/intel/fsp1_1/logo.c +++ b/src/drivers/intel/fsp1_1/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; } } |