build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/38507 )
Change subject: mb/intel/jasperlake_rvp: Add memory initialization parameters ......................................................................
Patch Set 2:
(2 comments)
https://review.coreboot.org/c/coreboot/+/38507/2/src/mainboard/intel/jasperl... File src/mainboard/intel/jasperlake_rvp/romstage_fsp_params.c:
https://review.coreboot.org/c/coreboot/+/38507/2/src/mainboard/intel/jasperl... PS2, Line 35: if(board_id == BOARD_ID_JSL_DDR4){ space required before the open brace '{'
https://review.coreboot.org/c/coreboot/+/38507/2/src/mainboard/intel/jasperl... PS2, Line 35: if(board_id == BOARD_ID_JSL_DDR4){ space required before the open parenthesis '('