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

[Furquan Shaikh (Code Review)]

Furquan Shaikh 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:

(1 comment)

https://review.coreboot.org/c/coreboot/+/43979/4//COMMIT_MSG 
Commit Message:

https://review.coreboot.org/c/coreboot/+/43979/4//COMMIT_MSG@21 
PS4, Line 21: This prevents code corruption of payload.  At this stage of boot, there
            : is no concept of memory management, therefore FSP can use any memory it
            : needs.
> I think FSP can produce multiple reserved memory HOBs, but we only handle one.

If we are missing the handling of multiple reserved memory HOBs published by FSP, then I think that needs to be fixed rather than changing the order here. This reserved memory HOB details will also have to be used to reserve the memory from OS as well.