build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35523 )
Change subject: mb/acer: Add Acer Aspire VN7-572G ......................................................................
Patch Set 53:
(4 comments)
https://review.coreboot.org/c/coreboot/+/35523/53/src/mainboard/acer/aspire_... File src/mainboard/acer/aspire_vn7_572g/ramstage.c:
https://review.coreboot.org/c/coreboot/+/35523/53/src/mainboard/acer/aspire_... PS53, Line 31: static void gpu_reset(void) { open brace '{' following function definitions go on the next line
https://review.coreboot.org/c/coreboot/+/35523/53/src/mainboard/acer/aspire_... PS53, Line 48: while (! gpio_get(DGPU_PWROK)) { space prohibited after that '!' (ctx:BxW)
https://review.coreboot.org/c/coreboot/+/35523/53/src/mainboard/acer/aspire_... PS53, Line 69: static void mainboard_init(struct device *dev) { open brace '{' following function definitions go on the next line
https://review.coreboot.org/c/coreboot/+/35523/53/src/mainboard/acer/aspire_... PS53, Line 75: static void mainboard_enable(struct device *dev) { open brace '{' following function definitions go on the next line