build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/33839 )
Change subject: src/mainboard/protectli/vault_kbl: Add FW6 boards support ......................................................................
Patch Set 1:
(4 comments)
https://review.coreboot.org/#/c/33839/1/src/mainboard/protectli/vault_kbl/ac... File src/mainboard/protectli/vault_kbl/acpi_tables.c:
https://review.coreboot.org/#/c/33839/1/src/mainboard/protectli/vault_kbl/ac... PS1, Line 15: */ adding a line without newline at end of file
https://review.coreboot.org/#/c/33839/1/src/mainboard/protectli/vault_kbl/gp... File src/mainboard/protectli/vault_kbl/gpio.h:
https://review.coreboot.org/#/c/33839/1/src/mainboard/protectli/vault_kbl/gp... PS1, Line 50: /* ISH_GP5 */ PAD_NC(GPP_A23, NONE ), space prohibited before that close parenthesis ')'
https://review.coreboot.org/#/c/33839/1/src/mainboard/protectli/vault_kbl/gp... PS1, Line 52: /* CORE_VID1 */ PAD_NC(GPP_B1, NONE), trailing whitespace
https://review.coreboot.org/#/c/33839/1/src/mainboard/protectli/vault_kbl/hd... File src/mainboard/protectli/vault_kbl/hda_verb.c:
https://review.coreboot.org/#/c/33839/1/src/mainboard/protectli/vault_kbl/hd... PS1, Line 26: AZALIA_ARRAY_SIZES; adding a line without newline at end of file