build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/44076 )
Change subject: soc/intel/common/block/cpu: Refactor init_cpus function ......................................................................
Patch Set 1:
(4 comments)
https://review.coreboot.org/c/coreboot/+/44076/1/src/soc/intel/common/block/... File src/soc/intel/common/block/cpu/mp_init.c:
https://review.coreboot.org/c/coreboot/+/44076/1/src/soc/intel/common/block/... PS1, Line 150: if (CONFIG(USE_INTEL_FSP_MP_INIT)) code indent should use tabs where possible
https://review.coreboot.org/c/coreboot/+/44076/1/src/soc/intel/common/block/... PS1, Line 150: if (CONFIG(USE_INTEL_FSP_MP_INIT)) please, no spaces at the start of a line
https://review.coreboot.org/c/coreboot/+/44076/1/src/soc/intel/common/block/... PS1, Line 151: return; code indent should use tabs where possible
https://review.coreboot.org/c/coreboot/+/44076/1/src/soc/intel/common/block/... PS1, Line 151: return; please, no spaces at the start of a line