build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/36087 )
Change subject: soc/intel/tigerlake: Do initial SoC commit ......................................................................
Patch Set 11:
(2 comments)
https://review.coreboot.org/c/coreboot/+/36087/11/src/soc/intel/tigerlake/in... File src/soc/intel/tigerlake/include/soc/ramstage.h:
https://review.coreboot.org/c/coreboot/+/36087/11/src/soc/intel/tigerlake/in... PS11, Line 24: void mainboard_silicon_init_params(FSP_S_CONFIG *params); need consistent spacing around '*' (ctx:WxV)
https://review.coreboot.org/c/coreboot/+/36087/11/src/soc/intel/tigerlake/in... File src/soc/intel/tigerlake/include/soc/romstage.h:
https://review.coreboot.org/c/coreboot/+/36087/11/src/soc/intel/tigerlake/in... PS11, Line 21: void mainboard_memory_init_params(FSPM_UPD *mupd); need consistent spacing around '*' (ctx:WxV)