[coreboot-gerrit] [S] Change in coreboot[master]: drivers/intel/fsp2_0: include IioPcieConfigUpd.h

[Jonathan Zhang (Code Review)]

Attention is currently required from: Simon Chou, Johnny Lin, Subrata Banik, Arthur Heymans, Ray Han Lim Ng, Srinidhi N Kaushik, Andrey Petrov.

Jonathan Zhang has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/71951 )

Change subject: drivers/intel/fsp2_0: include IioPcieConfigUpd.h
......................................................................

Patch Set 3:

(1 comment)

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

https://review.coreboot.org/c/coreboot/+/71951/comment/5a862756_8fc6b659 
PS2, Line 39: #if CONFIG(SOC_INTEL_SAPPHIRERAPID_SP)
            : #include <IioPcieConfigUpd.h>
            : #endif
> This is really not the place for SOC dependent code. […]

With SPR-SP, the IIO PCIe config becomes comprehensive, so the definitions are separated out from the FSP header file. In other words, IioPcieConfigUpd.h is actually part of FSP header. Another option is for FspmUpd.h to include this file.