Wim Vervoorn has uploaded this change for review.

View Change

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>
---
M src/drivers/intel/fsp1_1/logo.c
1 file changed, 1 insertion(+), 1 deletion(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/23/38123/1

diff --git a/src/drivers/intel/fsp1_1/logo.c b/src/drivers/intel/fsp1_1/logo.c
index 23aad01..b00406d 100644
--- a/src/drivers/intel/fsp1_1/logo.c
+++ b/src/drivers/intel/fsp1_1/logo.c
@@ -26,7 +26,7 @@
 		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;
 		}
 	}

To view, visit change 38123. To unsubscribe, or for help writing mail filters, visit settings.