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

[Rocky Phagura (Code Review)]

Rocky Phagura 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:

(2 comments)

Does anyone know why we load the payload into memory and then notify FSP? Doesn't it make sense to:
1. notify FSP of boot state init
2. load payload into memory
3. run payload

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

https://review.coreboot.org/c/coreboot/+/43979/3//COMMIT_MSG@7 
PS3, Line 7: drivers/intel/fsp2_0: Change boot state notification
> Add an empty line after the commit summary

Done

https://review.coreboot.org/c/coreboot/+/43979/3//COMMIT_MSG@19 
PS3, Line 19: This prevents code corruption of payload.  At this stage of boot, there is no
> nit: wrap at 72 characters

Done