[coreboot-gerrit] Change in coreboot[master]: drivers/intel/fsp2_0: Add callback after non-XIP FSP-M load

[Felix Held (Code Review)]

Felix Held has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/38700 )

Change subject: drivers/intel/fsp2_0: Add callback after non-XIP FSP-M load
......................................................................

Patch Set 5:

(3 comments)

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

https://review.coreboot.org/c/coreboot/+/38700/4//COMMIT_MSG@11 
PS4, Line 11: increases performance but requires the memory to be reserved later.
> I believe the intended design of non-XIP memory init may use a feature where executable code can be  […]

Done

https://review.coreboot.org/c/coreboot/+/38700/4//COMMIT_MSG@14 
PS4, Line 14: memory consumed by FSP-M.
> Looks here for a better explanation: https://review.coreboot. […]

Done

https://review.coreboot.org/c/coreboot/+/38700/4/src/drivers/intel/fsp2_0/in... 
File src/drivers/intel/fsp2_0/include/fsp/api.h:

https://review.coreboot.org/c/coreboot/+/38700/4/src/drivers/intel/fsp2_0/in... 
PS4, Line 56: fsp_reserve_fspm_mem
> I'm fine with changing the name, of course. […]

Done