Frans Hendriks has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/37559 )
Change subject: vendorcode/intel/fsp: Add Jasperlake FSP headers for FSP v1433 ......................................................................
Patch Set 6:
(1 comment)
Any reason for not using 3rdparty/fsp?
https://review.coreboot.org/c/coreboot/+/37559/6/src/vendorcode/intel/fsp/fs... File src/vendorcode/intel/fsp/fsp2_0/jasperlake/FirmwareVersionInfoHob.h:
https://review.coreboot.org/c/coreboot/+/37559/6/src/vendorcode/intel/fsp/fs... PS6, Line 25: /// coreboot guide line '/*' for comment