build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/29333 )
Change subject: src/soc/intel/braswell/northcluster.c: Correct calculation of FSP memory area ......................................................................
Patch Set 2:
(4 comments)
https://review.coreboot.org/#/c/29333/2/src/soc/intel/braswell/northcluster.... File src/soc/intel/braswell/northcluster.c:
https://review.coreboot.org/#/c/29333/2/src/soc/intel/braswell/northcluster.... PS2, Line 110: if (fsp_reserved_memory_area){ space required before the open brace '{'
https://review.coreboot.org/#/c/29333/2/src/soc/intel/braswell/northcluster.... PS2, Line 111: fsp_res_base_k = (RES_IN_KIB((unsigned int)fsp_reserved_memory_area)); line over 80 characters
https://review.coreboot.org/#/c/29333/2/src/soc/intel/braswell/northcluster.... PS2, Line 113: else that open brace { should be on the previous line
https://review.coreboot.org/#/c/29333/2/src/soc/intel/braswell/northcluster.... PS2, Line 113: else else should follow close brace '}'