Matt Delco has posted comments on this change. ( https://review.coreboot.org/27609 )
Change subject: soc/intel/skylake: permit Kconfig to set subsystem ID ......................................................................
Patch Set 1:
(1 comment)
https://review.coreboot.org/#/c/27609/1/src/soc/intel/skylake/chip_fsp20.c File src/soc/intel/skylake/chip_fsp20.c:
https://review.coreboot.org/#/c/27609/1/src/soc/intel/skylake/chip_fsp20.c@3... PS1, Line 380: ifdef
use "if (IS_ENABLED(CONFIG_SUBSYSTEM_VENDOR_ID))"
The IS_ENABLED only works with boolean parameters, and this setting is the actual value of the vendor ID.