build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/48426 )
Change subject: asus/a88xm-e: add the PIRQ routing table ......................................................................
Patch Set 6:
(22 comments)
File src/mainboard/asus/a88xm-e/irq_tables.c:
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-131801): https://review.coreboot.org/c/coreboot/+/48426/comment/52ad0381_9ca5c47f PS6, Line 22: /* bus, dev | fn, {{link, bitmap}, {link, bitmap}, {link, bitmap}, {link, bitmap} }, slot, rfu */ line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-131801): https://review.coreboot.org/c/coreboot/+/48426/comment/361d3ab0_b540fb73 PS6, Line 24: {0x00, (0x00 << 3) | 0x2, {{0x01, 0xccf0}, {0x02, 0xccf0}, {0x03, 0xccf0}, {0x04, 0xccf0} }, 0x0, 0x0}, line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-131801): https://review.coreboot.org/c/coreboot/+/48426/comment/41433db3_56e9adb2 PS6, Line 27: {0x00, (0x01 << 3) | 0x0, {{0x01, 0xccf0}, {0x02, 0xccf0}, {0x00, 0x0000}, {0x00, 0x0000} }, 0x0, 0x0}, line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-131801): https://review.coreboot.org/c/coreboot/+/48426/comment/956e5865_7733145d PS6, Line 29: {0x00, (0x02 << 3) | 0x0, {{0x01, 0xccf0}, {0x02, 0xccf0}, {0x03, 0xccf0}, {0x04, 0xccf0} }, 0x0, 0x0}, line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-131801): https://review.coreboot.org/c/coreboot/+/48426/comment/40c2b5c6_77c90183 PS6, Line 31: {0x00, (0x04 << 3) | 0x0, {{0x01, 0xccf0}, {0x02, 0xccf0}, {0x03, 0xccf0}, {0x04, 0xccf0} }, 0x0, 0x0}, line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-131801): https://review.coreboot.org/c/coreboot/+/48426/comment/3d6ce1c0_2e7a2bb4 PS6, Line 33: {0x00, (0x05 << 3) | 0x0, {{0x01, 0xccf0}, {0x02, 0xccf0}, {0x03, 0xccf0}, {0x04, 0xccf0} }, 0x0, 0x0}, line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-131801): https://review.coreboot.org/c/coreboot/+/48426/comment/efaf5899_191fedfb PS6, Line 35: {0x00, (0x06 << 3) | 0x0, {{0x02, 0xccf0}, {0x03, 0xccf0}, {0x04, 0xccf0}, {0x01, 0xccf0} }, 0x0, 0x0}, line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-131801): https://review.coreboot.org/c/coreboot/+/48426/comment/4ea1a28a_4ae0d33d PS6, Line 37: {0x00, (0x10 << 3) | 0x0, {{0x03, 0xccf0}, {0x00, 0x0000}, {0x00, 0x0000}, {0x00, 0x0000} }, 0x0, 0x0}, line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-131801): https://review.coreboot.org/c/coreboot/+/48426/comment/1f783178_f836c4e0 PS6, Line 39: {0x00, (0x11 << 3) | 0x0, {{0x04, 0xccf0}, {0x00, 0x0000}, {0x00, 0x0000}, {0x00, 0x0000} }, 0x0, 0x0}, line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-131801): https://review.coreboot.org/c/coreboot/+/48426/comment/ea814ea4_57679277 PS6, Line 42: {0x00, (0x12 << 3) | 0x0, {{0x03, 0xccf0}, {0x02, 0xccf0}, {0x00, 0x0000}, {0x00, 0x0000} }, 0x0, 0x0}, line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-131801): https://review.coreboot.org/c/coreboot/+/48426/comment/70bf19e6_0dd8d661 PS6, Line 45: {0x00, (0x13 << 3) | 0x0, {{0x03, 0xccf0}, {0x02, 0xccf0}, {0x00, 0x0000}, {0x00, 0x0000} }, 0x0, 0x0}, line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-131801): https://review.coreboot.org/c/coreboot/+/48426/comment/2adc3a72_a09aae29 PS6, Line 48: {0x00, (0x16 << 3) | 0x0, {{0x03, 0xccf0}, {0x02, 0xccf0}, {0x00, 0x0000}, {0x00, 0x0000} }, 0x0, 0x0}, line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-131801): https://review.coreboot.org/c/coreboot/+/48426/comment/c7554de6_642ba3e0 PS6, Line 51: {0x00, (0x14 << 3) | 0x0, {{0x01, 0xccf0}, {0x02, 0xccf0}, {0x03, 0xccf0}, {0x04, 0xccf0} }, 0x0, 0x0}, line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-131801): https://review.coreboot.org/c/coreboot/+/48426/comment/f4df8b37_e5e85d9c PS6, Line 53: {0x00, (0x14 << 3) | 0x4, {{0x05, 0xccf0}, {0x06, 0xccf0}, {0x00, 0x0000}, {0x00, 0x0000} }, 0x0, 0x0}, line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-131801): https://review.coreboot.org/c/coreboot/+/48426/comment/78c3ffae_8a6bff4d PS6, Line 55: {0x00, (0x15 << 3) | 0x0, {{0x02, 0xccf0}, {0x03, 0xccf0}, {0x04, 0xccf0}, {0x01, 0xccf0} }, 0x0, 0x0}, line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-131801): https://review.coreboot.org/c/coreboot/+/48426/comment/9448014a_574a3808 PS6, Line 57: {0x00, (0x15 << 3) | 0x1, {{0x03, 0xccf0}, {0x04, 0xccf0}, {0x01, 0xccf0}, {0x02, 0xccf0} }, 0x0, 0x0}, line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-131801): https://review.coreboot.org/c/coreboot/+/48426/comment/33a9aa4b_0f5b7656 PS6, Line 59: {0x00, (0x15 << 3) | 0x2, {{0x04, 0xccf0}, {0x01, 0xccf0}, {0x02, 0xccf0}, {0x03, 0xccf0} }, 0x0, 0x0}, line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-131801): https://review.coreboot.org/c/coreboot/+/48426/comment/83e34db5_2e97e98d PS6, Line 61: {0x00, (0x15 << 3) | 0x3, {{0x01, 0xccf0}, {0x02, 0xccf0}, {0x03, 0xccf0}, {0x04, 0xccf0} }, 0x0, 0x0}, line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-131801): https://review.coreboot.org/c/coreboot/+/48426/comment/e45d689d_d5e14552 PS6, Line 63: {0x01, (0x00 << 3) | 0x0, {{0x01, 0xccf0}, {0x02, 0xccf0}, {0x03, 0xccf0}, {0x04, 0xccf0} }, 0x1, 0x0}, line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-131801): https://review.coreboot.org/c/coreboot/+/48426/comment/edff1c1a_41210621 PS6, Line 65: {0x02, (0x00 << 3) | 0x0, {{0x01, 0xccf0}, {0x02, 0xccf0}, {0x03, 0xccf0}, {0x04, 0xccf0} }, 0x2, 0x0}, line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-131801): https://review.coreboot.org/c/coreboot/+/48426/comment/bf06c037_7e210d20 PS6, Line 67: {0x03, (0x00 << 3) | 0x0, {{0x02, 0xccf0}, {0x03, 0xccf0}, {0x04, 0xccf0}, {0x01, 0xccf0} }, 0x3, 0x0}, line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-131801): https://review.coreboot.org/c/coreboot/+/48426/comment/0b3f8a8b_be6c61ee PS6, Line 69: {0x04, (0x00 << 3) | 0x0, {{0x02, 0xccf0}, {0x03, 0xccf0}, {0x04, 0xccf0}, {0x01, 0xccf0} }, 0x4, 0x0} line over 96 characters