[coreboot-gerrit] Change in coreboot[master]: drivers/intel/fsp2_0: Add logo support

[Angel Pons (Code Review)]

Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/37515 )

Change subject: drivers/intel/fsp2_0: Add logo support
......................................................................

Patch Set 2: Code-Review+1
> Patch Set 2:
>
> (1 comment)
> > Patch Set 1:
> >
> > Since this seems to be one of those "move responsibility into FSP" things: what does this provide over our coreboot-side splash screen feature?
>
> Basically it adds BMP support, which is not in the current coreboot implementation. This allows  us to use one logo as splash screen for coreboot and a logo published in the BGRT table. The EDK payloads also use the BMP format.
>
> So this is not a replacement, it is just an additional feature that can be used when needed.

Out of curiosity, which platform do you plan to use this on? With libgfxinit and tianocore, coreboot can be made to show a .bmp bootsplash already.