[coreboot-gerrit] Change in coreboot[master]: {drivers,soc}/intel/fsp2_0: Move chipset specific logo handling to SoC

[Wim Vervoorn (Code Review)]

Wim Vervoorn has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/37792 )

Change subject: {drivers,soc}/intel/fsp2_0: Move chipset specific logo handling to SoC
......................................................................

Patch Set 6:

(1 comment)

https://review.coreboot.org/c/coreboot/+/37792/4/src/drivers/intel/fsp2_0/lo... 
File src/drivers/intel/fsp2_0/logo.c:

https://review.coreboot.org/c/coreboot/+/37792/4/src/drivers/intel/fsp2_0/lo... 
PS4, Line 29: 			if (logo_size)
            : 				*logo_ptr = (UINT32)logo_buffer;
> Ok, this is what I said for a invalid logo instance we can remove the cbmem entry here itself and se […]

It is of course possible to do this but the benefit is marginal as the cbmem entry will be removed directly after the fsp call.