[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.

Subrata Banik has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/81618?usp=email )

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

Patch Set 2:

(2 comments)

File src/lib/bmp_logo.c:

https://review.coreboot.org/c/coreboot/+/81618/comment/4b386411_e17ec758 :
PS1, Line 19: void bmp_load_logo(uintptr_t *logo_ptr, size_t *logo_size)
> Why does this function take two out-pointers and have no return value, anyway? I think that's a very […]

Acknowledged

File src/soc/amd/mendocino/fsp_s_params.c:

https://review.coreboot.org/c/coreboot/+/81618/comment/1e6f1be4_eef610b5 :
PS1, Line 59: (uintptr_t *)
> You can't do these kinds of things. […]

Acknowledged