[coreboot-gerrit] Change in coreboot[master]: soc/intel/icelake: Add ENABLE_DISPLAY_OVER_EXT_PCIE_GFX kconfig

[Subrata Banik (Code Review)]

Subrata Banik has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/33738 )

Change subject: soc/intel/icelake: Add ENABLE_DISPLAY_OVER_EXT_PCIE_GFX kconfig
......................................................................

Patch Set 11:
> Patch Set 11: Code-Review-1
>
> I still do not see an argument in the commit message, why this is intel/icelake specific.
>
> If such a Kconfig option is introduced, it needs to be integrated into the existing options under `src/device`, where VGA_ROM_RUN and friends are.

HI Paul, if you look at gerrit discussion, Patrick had point that keeping such kconfig into common driver might increase overhead of maintenance hence he suggested to make it soc specific. So far we don't see any such soc/rvp going to use this feature. If we think going forward that we are keeping for all rvp/soc then we should move this code into common/driver. Do you agree ?