build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/34065 )
Change subject: commonlib: Apply coding style for multiboot.h ......................................................................
Patch Set 2:
(6 comments)
https://review.coreboot.org/c/coreboot/+/34065/2/src/commonlib/include/commo... File src/commonlib/include/commonlib/multiboot.h:
https://review.coreboot.org/c/coreboot/+/34065/2/src/commonlib/include/commo... PS2, Line 157: { open brace '{' following union go on the same line
https://review.coreboot.org/c/coreboot/+/34065/2/src/commonlib/include/commo... PS2, Line 193: #define MULTIBOOT_FRAMEBUFFER_TYPE_INDEXED 0 please, no space before tabs
https://review.coreboot.org/c/coreboot/+/34065/2/src/commonlib/include/commo... PS2, Line 194: #define MULTIBOOT_FRAMEBUFFER_TYPE_RGB 1 please, no space before tabs
https://review.coreboot.org/c/coreboot/+/34065/2/src/commonlib/include/commo... PS2, Line 198: { open brace '{' following union go on the same line
https://review.coreboot.org/c/coreboot/+/34065/2/src/commonlib/include/commo... PS2, Line 200: { open brace '{' following struct go on the same line
https://review.coreboot.org/c/coreboot/+/34065/2/src/commonlib/include/commo... PS2, Line 205: { open brace '{' following struct go on the same line