[coreboot-gerrit] [S] Change in coreboot[main]: lib: Refactor bmp_load_logo() implementation

[Subrata Banik (Code Review)]

Attention is currently required from: Andrey Petrov, Felix Held, Fred Reitberger, Jason Glenesk, Julius Werner, Matt DeVillier, Ronak Kanabar, Sean Rhodes.

Hello Andrey Petrov, Felix Held, Fred Reitberger, Jason Glenesk, Matt DeVillier, Ronak Kanabar, Sean Rhodes, build bot (Jenkins),

I'd like you to reexamine a change. Please visit

https://review.coreboot.org/c/coreboot/+/81618?usp=email

to look at the new patch set (#3).

Change subject: lib: Refactor bmp_load_logo() implementation
......................................................................

lib: Refactor bmp_load_logo() implementation

This refactoring ensures bmp_load_logo() takes logo_size as an
argument, returning a valid logo_ptr only if logo_size is non-zero.

This prevents potential errors from mismatched size assumption.

BUG=b:242829490
TEST=google/rex0 builds successfully.

Change-Id: I14bc54670a67980ec93bc366b274832d1f959e50
Signed-off-by: Subrata Banik 
---
M src/drivers/intel/fsp1_1/ramstage.c
M src/drivers/intel/fsp2_0/fsp_gop_blt.c
M src/include/bootsplash.h
M src/lib/bmp_logo.c
M src/soc/amd/mendocino/fsp_s_params.c
M src/soc/intel/apollolake/chip.c
M src/soc/intel/cannonlake/fsp_params.c
M src/soc/intel/skylake/chip.c
8 files changed, 29 insertions(+), 16 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/18/81618/3