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 1:
(6 comments)
https://review.coreboot.org/#/c/31652/1/src/mainboard/getac/p470/cstates.c File src/mainboard/getac/p470/cstates.c:
https://review.coreboot.org/#/c/31652/1/src/mainboard/getac/p470/cstates.c@2... PS1, Line 23: {ACPI_ADDRESS_SPACE_FIXED, 1, 2, {1}, 0, 0}}, space required after that close brace '}'
https://review.coreboot.org/#/c/31652/1/src/mainboard/getac/p470/cstates.c@2... PS1, 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/1/src/mainboard/getac/p470/cstates.c@3... PS1, 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/1/src/mainboard/getac/p470/gpio.c File src/mainboard/getac/p470/gpio.c:
https://review.coreboot.org/#/c/31652/1/src/mainboard/getac/p470/gpio.c@103 PS1, Line 103: { that open brace { should be on the previous line
https://review.coreboot.org/#/c/31652/1/src/mainboard/getac/p470/gpio.c@111 PS1, Line 111: { that open brace { should be on the previous line
https://review.coreboot.org/#/c/31652/1/src/mainboard/getac/p470/irq_tables.... File src/mainboard/getac/p470/irq_tables.c:
https://review.coreboot.org/#/c/31652/1/src/mainboard/getac/p470/irq_tables.... PS1, Line 53: }}; space required after that close brace '}'