[S] Change in coreboot[main]: lib: Centralize logo.bmp inclusion in lib/Makefile.mk

[Julius Werner (Code Review)]

Attention is currently required from: Andrey Petrov, Intel coreboot Reviewers, Karthik Ramasubramanian, Ronak Kanabar, Subrata Banik.

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

Change subject: lib: Centralize logo.bmp inclusion in lib/Makefile.mk
......................................................................

Patch Set 3:

(1 comment)

File src/lib/Makefile.mk:

https://review.coreboot.org/c/coreboot/+/86367/comment/051fd309_9610c21c?usp... :
PS3, Line 427: CONFIG_FSP2_0_LOGO_FILE_NAME
> > Actually, why is this an FSP2_0 option? This should probably also move to `src/lib/Kconfig` and be […]

Oh, I see, *all* the BMP_LOGO Kconfigs are still in the FSP folder even if most of them aren't prefixed "FSP" anymore. Yeah that's not great, if the code is in a platform-independent directory the Kconfigs should be too.

You don't need to do it now but it would be good if it got done eventually.