Attention is currently required from: Arthur Heymans, Chen, Gang C, David Hendricks, Jonathan Zhang, Lean Sheng Tan, Patrick Rudolph, Paul Menzel, Shuo Liu, TangYiwei.
Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/81322?usp=email )
Change subject: mb/intel/beechnutcity_crb: Add GNR/SRF-SP 2S server board Beechnut City ......................................................................
Patch Set 28:
(1 comment)
File configs/builder/config.intel.crb.bnc:
https://review.coreboot.org/c/coreboot/+/81322/comment/bbd3ba05_85e5cda9 : PS11, Line 27: # : CONFIG_IFD_BIN_PATH="site-local/beechnutcity/descriptor.bin" : CONFIG_CPU_UCODE_BINARIES="site-local/beechnutcity/ucode.mcb" : CONFIG_FSP_T_FILE="site-local/beechnutcity/Server_T.fd" : CONFIG_FSP_M_FILE="site-local/beechnutcity/Server_M.fd" : CONFIG_FSP_S_FILE="site-local/beechnutcity/Server_S.fd" : CONFIG_FSP_HEADER_PATH="src/vendorcode/intel/fsp/fsp2_0/graniterapids/sp/"
This is aligned with configs/builder/config.intel.crb. […]
This describes the settings needed to build a bootable image for this board. As not all binaries are publicly available (yet), this config has to use site-local binaries.
But when things eventually get published, the defaults in platform code should be updated. Ideally, the default config for a mainboard (just select board vendor and model, and maybe choose a compatible payload) should produce a bootable coreboot image. (Yes, it wouldn't include any IFD or ME regions, but you can flash the result to the BIOS region only and it should work).