Subrata Banik has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/33739 )
Change subject: soc/intel/icelake: Add option to enable display over PCI external GFX ......................................................................
Patch Set 10:
(1 comment)
https://review.coreboot.org/c/coreboot/+/33739/9/src/soc/intel/icelake/Kconf... File src/soc/intel/icelake/Kconfig:
https://review.coreboot.org/c/coreboot/+/33739/9/src/soc/intel/icelake/Kconf... PS9, Line 211: config HAVE_EXT_GFX
- This is not any valid point, we have proper protection in CB code and FSP to run opRom. […]
can you please help to write the same for all PCIE card vendors NVDIA, AMD, ATI whoever and say please stop selling cards with OpRom, that would help. Right now my problem statement is that, i need to bring display over PCIE GFX for my board and i need to run OpRom, what is my priority.
I have protection taken to lock down all my protected range registers what any OpRom can manipulate, i have all Chipset registers lockdown as part of POSTBOOT_SAI thats the whole purpose of security that no 3rd party code can break into chipset space.
What you mean by not running OpRom code. still today there are many PC running thousands of OpRom code, all are BAD ? and they should stop their business. Please try to be reasonable with other patches and understand what is the need of the hour.