build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/43816 )
Change subject: soc/intel/jasperlake: Disable multiphase SI init ......................................................................
Patch Set 7:
(2 comments)
https://review.coreboot.org/c/coreboot/+/43816/7/src/soc/intel/jasperlake/fs... File src/soc/intel/jasperlake/fsp_params.c:
https://review.coreboot.org/c/coreboot/+/43816/7/src/soc/intel/jasperlake/fs... PS7, Line 355: return 0; code indent should use tabs where possible
https://review.coreboot.org/c/coreboot/+/43816/7/src/soc/intel/jasperlake/fs... PS7, Line 355: return 0; please, no spaces at the start of a line