build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31652 )
Change subject: getac/p470: Let clang-format handle the coding style ......................................................................
Patch Set 2:
(6 comments)
https://review.coreboot.org/#/c/31652/2/src/mainboard/getac/p470/cstates.c File src/mainboard/getac/p470/cstates.c:
https://review.coreboot.org/#/c/31652/2/src/mainboard/getac/p470/cstates.c@2... PS2, Line 23: {ACPI_ADDRESS_SPACE_FIXED, 1, 2, 1, 0, 0}}, space required after that close brace '}'
https://review.coreboot.org/#/c/31652/2/src/mainboard/getac/p470/cstates.c@2... PS2, Line 28: {ACPI_ADDRESS_SPACE_IO, 8, 0, 0, DEFAULT_PMBASE + LV2, 0}}, space required after that close brace '}'
https://review.coreboot.org/#/c/31652/2/src/mainboard/getac/p470/cstates.c@3... PS2, Line 33: {ACPI_ADDRESS_SPACE_IO, 8, 0, 0, DEFAULT_PMBASE + LV3, 0}}, space required after that close brace '}'
https://review.coreboot.org/#/c/31652/2/src/mainboard/getac/p470/gpio.c File src/mainboard/getac/p470/gpio.c:
https://review.coreboot.org/#/c/31652/2/src/mainboard/getac/p470/gpio.c@103 PS2, Line 103: { that open brace { should be on the previous line
https://review.coreboot.org/#/c/31652/2/src/mainboard/getac/p470/gpio.c@111 PS2, Line 111: { that open brace { should be on the previous line
https://review.coreboot.org/#/c/31652/2/src/mainboard/getac/p470/irq_tables.... File src/mainboard/getac/p470/irq_tables.c:
https://review.coreboot.org/#/c/31652/2/src/mainboard/getac/p470/irq_tables.... PS2, Line 53: }}; space required after that close brace '}'