build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/52766 )
Change subject: mb/google/guybrush: Add temporary _AIE table ......................................................................
Patch Set 1:
(2 comments)
File src/mainboard/google/guybrush/variants/baseboard/include/baseboard/ec.h:
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-118064): https://review.coreboot.org/c/coreboot/+/52766/comment/b818040e_b5cb9d81 PS1, Line 61: #define EC_ENABLE_WAKE_PIN Package () {_SB.GPIO, 0} space prohibited between function name and open parenthesis '('
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-118064): https://review.coreboot.org/c/coreboot/+/52766/comment/9b74660d_4de69cf2 PS1, Line 61: #define EC_ENABLE_WAKE_PIN Package () {_SB.GPIO, 0} Macros with complex values should be enclosed in parentheses