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

[Aaron Durbin (Code Review)]

Aaron Durbin 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 7:
> Patch Set 7:
>
> I'll let Felix manage this stack of patches, so I'll not abandon it right now.
>
> However, now it seems like a cleaner approach is to allow the FSP to report its region (still reflective of its header) as reserved.

I completely disagree on this. FSP should not mark itself as reserved. It doesn't know how it's used. That's up the user of FSP to determine.