build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/26470 )
Change subject: mainboard/asrock/e350m1: Fix coding style ......................................................................
Patch Set 2:
(6 comments)
https://review.coreboot.org/#/c/26470/2/src/mainboard/asrock/e350m1/BiosCall... File src/mainboard/asrock/e350m1/BiosCallOuts.c:
https://review.coreboot.org/#/c/26470/2/src/mainboard/asrock/e350m1/BiosCall... PS2, Line 23: VOID *ConfigPtr); need consistent spacing around '*' (ctx:WxV)
https://review.coreboot.org/#/c/26470/2/src/mainboard/asrock/e350m1/BiosCall... PS2, Line 91: /* this seems to be just copy-pasted from the AMD reference boards and needs line over 80 characters
https://review.coreboot.org/#/c/26470/2/src/mainboard/asrock/e350m1/OemCusto... File src/mainboard/asrock/e350m1/OemCustomize.c:
https://review.coreboot.org/#/c/26470/2/src/mainboard/asrock/e350m1/OemCusto... PS2, Line 37: } Statements should start on a tabstop
https://review.coreboot.org/#/c/26470/2/src/mainboard/asrock/e350m1/OemCusto... PS2, Line 52: } Statements should start on a tabstop
https://review.coreboot.org/#/c/26470/2/src/mainboard/asrock/e350m1/mptable.... File src/mainboard/asrock/e350m1/mptable.c:
https://review.coreboot.org/#/c/26470/2/src/mainboard/asrock/e350m1/mptable.... PS2, Line 27: [0x00] = 0x10, 0x11, 0x12, 0x13, 0x14, 0x15, 0x16, 0x17, /* INTA# - INTH# */ line over 80 characters
https://review.coreboot.org/#/c/26470/2/src/mainboard/asrock/e350m1/mptable.... PS2, Line 28: [0x08] = 0x00, 0x00, 0x00, 0x00, 0x1F, 0x1F, 0x1F, 0x1F, /* Misc-nil,0,1,2, INT from Serial irq */ line over 80 characters
-
build bot (Jenkins) (Code Review)