build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/36143 )
Change subject: arch/x86/boot.c: Pass arguments when running programs ......................................................................
Patch Set 2:
(4 comments)
https://review.coreboot.org/c/coreboot/+/36143/2/src/arch/x86/boot.c File src/arch/x86/boot.c:
https://review.coreboot.org/c/coreboot/+/36143/2/src/arch/x86/boot.c@33 PS2, Line 33: void(* doit)(void *); missing space after return type
https://review.coreboot.org/c/coreboot/+/36143/2/src/arch/x86/boot.c@33 PS2, Line 33: void(* doit)(void *); Unnecessary space before function pointer name
https://review.coreboot.org/c/coreboot/+/36143/2/src/arch/x86/boot.c@33 PS2, Line 33: void(* doit)(void *); space prohibited after that '*' (ctx:BxW)
https://review.coreboot.org/c/coreboot/+/36143/2/src/arch/x86/boot.c@33 PS2, Line 33: void(* doit)(void *); function definition argument 'void *' should also have an identifier name