[XS] Change in coreboot[main]: driver/intel/fsp2_0: Prevent error logs during S3 wakeup

[Subrata Banik (Code Review)]

Attention is currently required from: Andrey Petrov, Krishna P Bhat D, Ronak Kanabar, Varun Upadhyay.

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

Change subject: driver/intel/fsp2_0: Prevent error logs during S3 wakeup
......................................................................

Patch Set 3:

(1 comment)

File src/drivers/intel/fsp2_0/fsp_gop_blt.c:

https://review.coreboot.org/c/coreboot/+/83055/comment/c64503b4_26f2c1a6?usp... :
PS2, Line 245: 	/* CBMEM is locked for S3 resume path. */
             : 	if (acpi_is_wakeup_s3())
             : 		return;
> The current implementation returns NULL from bmp_load_logo() when the system is waking up from an S3 sleep state so it doesnt goes to draw FW Splash screen, This change is just eliminating error messages that were being logged during the S3 resume process.

my point is shouldn't we draw FW splash screen during S3 resume ?