[coreboot-gerrit] Change in coreboot[master]: drivers/intel/fsp2_0: Add mb hooks before & after FSP calls

[Furquan Shaikh (Code Review)]

Attention is currently required from: Martin Roth.
Furquan Shaikh has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/52866 )

Change subject: drivers/intel/fsp2_0: Add mb hooks before & after FSP calls
......................................................................

Patch Set 1:

(1 comment)

File src/drivers/intel/fsp2_0/include/fsp/api.h:

https://review.coreboot.org/c/coreboot/+/52866/comment/aa83078f_95790d93 
PS1, Line 75: save and restore registers
> Sure, but this is being done on the mainboard level, not the platform level. […]

I think the workarounds that are required would generally be applicable to all mainboards for a platform. If FSP absolutely cannot be fixed, I think it is better to apply it at the SoC level so that all mainboards benefit from it.