Attention is currently required from: Christian Walter, Arthur Heymans, Michael Niewöhner. build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/37441 )
Change subject: mb/supermicro/x11-lga1151v2-series: Add support for X11SCH-F ......................................................................
Patch Set 72:
(9 comments)
File src/mainboard/supermicro/x11-lga1151v2-series/Kconfig:
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-137348): https://review.coreboot.org/c/coreboot/+/37441/comment/224da36c_94fa6822 PS72, Line 12: select SUPERIO_ASPEED_AST2400 'SUPERIO' may be misspelled - perhaps ''?
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-137348): https://review.coreboot.org/c/coreboot/+/37441/comment/f5be7343_54fb8180 PS72, Line 13: select SUPERIO_ASPEED_COMMON_PRE_RAM 'SUPERIO' may be misspelled - perhaps ''?
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-137348): https://review.coreboot.org/c/coreboot/+/37441/comment/038ce7b3_9153e1dd PS72, Line 14: select SUPERIO_ASPEED_HAS_UART_DELAY_WORKAROUND 'SUPERIO' may be misspelled - perhaps ''?
File src/mainboard/supermicro/x11-lga1151v2-series/bootblock.c:
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-137348): https://review.coreboot.org/c/coreboot/+/37441/comment/b3ba78a9_23134606 PS72, Line 4: #include <superio/aspeed/ast2400/ast2400.h> 'superio' may be misspelled - perhaps ''?
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-137348): https://review.coreboot.org/c/coreboot/+/37441/comment/422af2a5_1b9041c7 PS72, Line 5: #include <superio/aspeed/common/aspeed.h> 'superio' may be misspelled - perhaps ''?
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-137348): https://review.coreboot.org/c/coreboot/+/37441/comment/3254e7a8_55f97cef PS72, Line 9: static void early_config_superio(void) 'superio' may be misspelled - perhaps ''?
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-137348): https://review.coreboot.org/c/coreboot/+/37441/comment/baf2012c_9fc05768 PS72, Line 19: early_config_superio(); 'superio' may be misspelled - perhaps ''?
File src/mainboard/supermicro/x11-lga1151v2-series/devicetree.cb:
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-137348): https://review.coreboot.org/c/coreboot/+/37441/comment/82cbe804_31a878a1 PS72, Line 69: chip superio/common 'superio' may be misspelled - perhaps ''?
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-137348): https://review.coreboot.org/c/coreboot/+/37441/comment/dc65d3f9_8c147039 PS72, Line 71: chip superio/aspeed/ast2400 'superio' may be misspelled - perhaps ''?