build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/38461 )
Change subject: soc/intel/tigerlake: Update fsp params for Jasper Lake ......................................................................
Patch Set 11:
(2 comments)
https://review.coreboot.org/c/coreboot/+/38461/11/src/soc/intel/tigerlake/ch... File src/soc/intel/tigerlake/chip.h:
https://review.coreboot.org/c/coreboot/+/38461/11/src/soc/intel/tigerlake/ch... PS11, Line 214: }debuginterfaceflag; space required after that close brace '}'
https://review.coreboot.org/c/coreboot/+/38461/11/src/soc/intel/tigerlake/fs... File src/soc/intel/tigerlake/fsp_params_jsl.c:
https://review.coreboot.org/c/coreboot/+/38461/11/src/soc/intel/tigerlake/fs... PS11, Line 136: if (!dev) { braces {} are not necessary for any arm of this statement