build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/37441 )
Change subject: src/mainboard/supermicro/x11-lga1151v2-series: Add Support for X11SCH-F ......................................................................
Patch Set 49:
(5 comments)
https://review.coreboot.org/c/coreboot/+/37441/49/src/mainboard/supermicro/x... File src/mainboard/supermicro/x11-lga1151v2-series/bootblock.c:
https://review.coreboot.org/c/coreboot/+/37441/49/src/mainboard/supermicro/x... PS49, Line 18: const struct pad_config * early_gpio_table = get_early_gpio_table(&num); "foo * bar" should be "foo *bar"
https://review.coreboot.org/c/coreboot/+/37441/49/src/mainboard/supermicro/x... File src/mainboard/supermicro/x11-lga1151v2-series/memory.c:
https://review.coreboot.org/c/coreboot/+/37441/49/src/mainboard/supermicro/x... PS49, Line 44: const struct cnl_mb_cfg * variant_memcfg_config(void) "foo * bar" should be "foo *bar"
https://review.coreboot.org/c/coreboot/+/37441/49/src/mainboard/supermicro/x... File src/mainboard/supermicro/x11-lga1151v2-series/ramstage.c:
https://review.coreboot.org/c/coreboot/+/37441/49/src/mainboard/supermicro/x... PS49, Line 10: size_t num = 0; please, no spaces at the start of a line
https://review.coreboot.org/c/coreboot/+/37441/49/src/mainboard/supermicro/x... PS49, Line 11: const struct pad_config * gpio_table = get_gpio_table(&num); please, no spaces at the start of a line
https://review.coreboot.org/c/coreboot/+/37441/49/src/mainboard/supermicro/x... PS49, Line 11: const struct pad_config * gpio_table = get_gpio_table(&num); "foo * bar" should be "foo *bar"