[coreboot-gerrit] Change in coreboot[master]: drivers/intel/fsp2_0: Change boot state notification

[Angel Pons (Code Review)]

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

Change subject: drivers/intel/fsp2_0: Change boot state notification
......................................................................

Patch Set 4:
> Patch Set 4:
>
> (2 comments)
>
> Does anyone know why we load the payload into memory and then notify FSP? Doesn't it make sense to:
> notify FSP of boot state init
> load payload into memory
> run payload

I'm not sure why we do it like that, but if FSP unexpectedly uses some memory, we have a problem. Maybe FSP is doing something wrong, or we're not properly handling its reserved memory HOBs (there can be more than one).