[S] Change in coreboot[main]: vc/intel/fsp: Refactor FSP header inclusion for EDK2 compatibility

[Arthur Heymans (Code Review)]

Attention is currently required from: Appukuttan V K, Dinesh Gehlot, Kapil Porwal, Nick Vaccaro, Ronak Kanabar, Subrata Banik.

Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/81623?usp=email )

Change subject: vc/intel/fsp: Refactor FSP header inclusion for EDK2 compatibility
......................................................................

Patch Set 5:

(1 comment)

File src/vendorcode/intel/fsp/fsp2_0/IntelFspPkg/Include/FspInfoHeader.h:

https://review.coreboot.org/c/coreboot/+/81623/comment/f6577c42_3daad7a8 :
PS5, Line 7: vendorcode/intel/edk2/UDK2017/
> > This prefix is used in other places in the tree. Can we use a Kconfig param for it that depends on the UDK kconfig? That would make this file trivial and same with vendorcode/intel/Makefile.mk
>
> can you please help to share a little more details which I can take as an incremental change.
>
> I'm sensing that you are asking to introduce a kconfig for each `UDK version` to mask the `vendorcode/intel/edk2/UDK2017` path ?
>
> Then, this file only includes only one header like
>
> ```#include <CONFIG(UDK_PATH)/Include/Guid/FspHeaderFile.h>```
>
> UDK_PATH would depend on UDK configs like UDK_2017_VERSION etc.?

Yes that would be my suggestion. It's used in other places (makefile) too so that's why I think it might be a good idea.