Attention is currently required from: Alexander Couzens.
build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/69066 )
Change subject: mb/*/*/irq_tables.c: Use PCI_DEVFN(dev, fn) ......................................................................
Patch Set 1:
(122 comments)
File src/mainboard/asus/am1i-a/irq_tables.c:
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/5185ffc0_87bb203f PS1, Line 23: /* bus,PCI_DEVFN(dev, fn), {link, bitmap}, {link, bitmap}, {link, bitmap}, {link, bitmap}, slot, rfu */ line length of 121 exceeds 114 columns
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/0e60f0d1_b8c7cd88 PS1, Line 24: {0x00, PCI_DEVFN(0x1, 0), {{0x01, 0x9cb8}, {0x02, 0x9cb8}, {0x00, 0x0000}, {0x00, 0x0000}}, 0x0, 0x0}, line length of 118 exceeds 115 columns
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/f9364ce4_1e8c3f3d PS1, Line 25: {0x00, PCI_DEVFN(0x02, 0), {{0x01, 0x9cb8}, {0x02, 0x9cb8}, {0x03, 0x9cb8}, {0x04, 0x9cb8}}, 0x0, 0x0}, line length of 119 exceeds 116 columns
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/20ef9a8c_824451c1 PS1, Line 26: {0x00, PCI_DEVFN(0x10, 0), {{0x03, 0x9cb8}, {0x00, 0x0000}, {0x00, 0x0000}, {0x00, 0x0000}}, 0x0, 0x0}, line length of 119 exceeds 117 columns
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/0006ca81_c168a1ff PS1, Line 27: {0x00, PCI_DEVFN(0x11, 0), {{0x04, 0x9cb8}, {0x00, 0x0000}, {0x00, 0x0000}, {0x00, 0x0000}}, 0x0, 0x0}, line length of 119 exceeds 118 columns
File src/mainboard/asus/p2b/variants/p2b-f/irq_tables.c:
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/1b620a13_a23a0cae PS1, Line 21: {0x00,PCI_DEVFN(0x0c, 0), {{0x60, 0x1eb8}, {0x61, 0x1eb8}, {0x62, 0x1eb8}, {0x63, 0x01eb8}}, 0x1, 0x0}, space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/b3c3f693_f1507da8 PS1, Line 22: {0x00,PCI_DEVFN(0x0b, 0), {{0x61, 0x1eb8}, {0x62, 0x1eb8}, {0x63, 0x1eb8}, {0x60, 0x01eb8}}, 0x2, 0x0}, space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/eea95bc8_f94c0cc1 PS1, Line 23: {0x00,PCI_DEVFN(0x0a, 0), {{0x62, 0x1eb8}, {0x63, 0x1eb8}, {0x60, 0x1eb8}, {0x61, 0x01eb8}}, 0x3, 0x0}, space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/9e6be685_4bba67d2 PS1, Line 24: {0x00,PCI_DEVFN(0x09, 0), {{0x63, 0x1eb8}, {0x60, 0x1eb8}, {0x61, 0x1eb8}, {0x62, 0x01eb8}}, 0x4, 0x0}, space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/8887a5c4_7b1d5280 PS1, Line 25: {0x00,PCI_DEVFN(0x0d, 0), {{0x63, 0x1eb8}, {0x60, 0x1eb8}, {0x61, 0x1eb8}, {0x62, 0x01eb8}}, 0x5, 0x0}, space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/169f5a1b_2e3b5297 PS1, Line 26: {0x00,PCI_DEVFN(0x04, 0), {{0x60, 0x1eb8}, {0x61, 0x1eb8}, {0x62, 0x1eb8}, {0x63, 0x01eb8}}, 0x0, 0x0}, space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/1d3f3b30_87301ccc PS1, Line 27: {0x00,PCI_DEVFN(0x01, 0), {{0x60, 0x1eb8}, {0x61, 0x1eb8}, {0x62, 0x1eb8}, {0x63, 0x01eb8}}, 0x0, 0x0}, space required after that ',' (ctx:VxV)
File src/mainboard/asus/p2b/variants/p2b/irq_tables.c:
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/4031742f_c14c60b6 PS1, Line 21: {0x00,PCI_DEVFN(0x0c, 0), {{0x60, 0x1eb8}, {0x61, 0x1eb8}, {0x62, 0x1eb8}, {0x63, 0x01eb8}}, 0x1, 0x0}, space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/8c123924_9ab25efa PS1, Line 22: {0x00,PCI_DEVFN(0x0b, 0), {{0x61, 0x1eb8}, {0x62, 0x1eb8}, {0x63, 0x1eb8}, {0x60, 0x01eb8}}, 0x2, 0x0}, space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/7469bac7_1b7962c6 PS1, Line 23: {0x00,PCI_DEVFN(0x0a, 0), {{0x62, 0x1eb8}, {0x63, 0x1eb8}, {0x60, 0x1eb8}, {0x61, 0x01eb8}}, 0x3, 0x0}, space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/44777030_06883fd0 PS1, Line 24: {0x00,PCI_DEVFN(0x09, 0), {{0x63, 0x1eb8}, {0x60, 0x1eb8}, {0x61, 0x1eb8}, {0x62, 0x01eb8}}, 0x4, 0x0}, space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/853b17e1_8b69a8de PS1, Line 25: {0x00,PCI_DEVFN(0x04, 0), {{0x60, 0x1eb8}, {0x61, 0x1eb8}, {0x62, 0x1eb8}, {0x63, 0x01eb8}}, 0x0, 0x0}, space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/42bf4038_9e8e309d PS1, Line 26: {0x00,PCI_DEVFN(0x01, 0), {{0x60, 0x1eb8}, {0x61, 0x1eb8}, {0x62, 0x1eb8}, {0x63, 0x01eb8}}, 0x0, 0x0}, space required after that ',' (ctx:VxV)
File src/mainboard/asus/p2b/variants/p3b-f/irq_tables.c:
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/4316521d_c92dc5a2 PS1, Line 21: {0x00,PCI_DEVFN(0x0c, 0), {{0x60, 0x1eb8}, {0x61, 0x1eb8}, {0x62, 0x1eb8}, {0x63, 0x01eb8}}, 0x1, 0x0}, space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/f69c36e5_44b3ab5e PS1, Line 22: {0x00,PCI_DEVFN(0x0b, 0), {{0x61, 0x1eb8}, {0x62, 0x1eb8}, {0x63, 0x1eb8}, {0x60, 0x01eb8}}, 0x2, 0x0}, space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/12eb1e6c_445697e2 PS1, Line 23: {0x00,PCI_DEVFN(0x0a, 0), {{0x62, 0x1eb8}, {0x63, 0x1eb8}, {0x60, 0x1eb8}, {0x61, 0x01eb8}}, 0x3, 0x0}, space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/5e64df83_edd07e42 PS1, Line 24: {0x00,PCI_DEVFN(0x09, 0), {{0x63, 0x1eb8}, {0x60, 0x1eb8}, {0x61, 0x1eb8}, {0x62, 0x01eb8}}, 0x4, 0x0}, space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/942445e2_979404de PS1, Line 25: {0x00,PCI_DEVFN(0x0d, 0), {{0x63, 0x1eb8}, {0x60, 0x1eb8}, {0x61, 0x1eb8}, {0x62, 0x01eb8}}, 0x5, 0x0}, space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/bdce7950_bc051608 PS1, Line 26: {0x00,PCI_DEVFN(0x0e, 0), {{0x62, 0x1eb8}, {0x63, 0x1eb8}, {0x60, 0x1eb8}, {0x61, 0x01eb8}}, 0x6, 0x0}, space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/70d4faa8_af0ef9be PS1, Line 27: {0x00,PCI_DEVFN(0x04, 0), {{0x60, 0x1eb8}, {0x61, 0x1eb8}, {0x62, 0x1eb8}, {0x63, 0x01eb8}}, 0x0, 0x0}, space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/3fdb0323_ab385903 PS1, Line 28: {0x00,PCI_DEVFN(0x01, 0), {{0x60, 0x1eb8}, {0x61, 0x1eb8}, {0x62, 0x1eb8}, {0x63, 0x01eb8}}, 0x0, 0x0}, space required after that ',' (ctx:VxV)
File src/mainboard/emulation/qemu-i440fx/irq_tables.c:
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/bc29cf3f_b90881ce PS1, Line 21: {0x00,PCI_DEVFN(0x01, 0), {{0x60, 0xdef8}, {0x61, 0xdef8}, {0x62, 0xdef8}, {0x63, 0x0def8}}, 0x0, 0x0}, space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/242eb649_13acc1fd PS1, Line 22: {0x00,PCI_DEVFN(0x02, 0), {{0x61, 0xdef8}, {0x62, 0xdef8}, {0x63, 0xdef8}, {0x60, 0x0def8}}, 0x1, 0x0}, space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/b21882e6_fb921d8b PS1, Line 23: {0x00,PCI_DEVFN(0x03, 0), {{0x62, 0xdef8}, {0x63, 0xdef8}, {0x60, 0xdef8}, {0x61, 0x0def8}}, 0x2, 0x0}, space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/2d9a54b0_70614549 PS1, Line 24: {0x00,PCI_DEVFN(0x04, 0), {{0x63, 0xdef8}, {0x60, 0xdef8}, {0x61, 0xdef8}, {0x62, 0x0def8}}, 0x3, 0x0}, space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/c2bd95c7_c87644df PS1, Line 25: {0x00,PCI_DEVFN(0x05, 0), {{0x60, 0xdef8}, {0x61, 0xdef8}, {0x62, 0xdef8}, {0x63, 0x0def8}}, 0x4, 0x0}, space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/de4a3406_156b0b49 PS1, Line 26: {0x00,PCI_DEVFN(0x06, 0), {{0x61, 0xdef8}, {0x62, 0xdef8}, {0x63, 0xdef8}, {0x60, 0x0def8}}, 0x5, 0x0}, space required after that ',' (ctx:VxV)
File src/mainboard/getac/p470/irq_tables.c:
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/d71cfca4_708c5088 PS1, Line 21: {0x00,PCI_DEVFN(0x01, 0), {{0x60, 0xdcf8}, {0x61, 0xdcf8}, {0x62, 0xdcf8}, {0x63, 0x0dcd8}}, 0x0, 0x0}, // PCIe? space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/110eb6db_6ec76cc9 PS1, Line 22: {0x00,PCI_DEVFN(0x02, 0), {{0x60, 0xdcf8}, {0x00, 0x0000}, {0x00, 0x0000}, {0x00, 0x00000}}, 0x0, 0x0}, // VGA space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/a2b992b9_5924993b PS1, Line 23: {0x00,PCI_DEVFN(0x1e, 0), {{0x61, 0xdcf8}, {0x68, 0xdcf8}, {0x00, 0x0000}, {0x00, 0x00000}}, 0x0, 0x0}, // PCI bridge space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/2b4b83ec_339c7571 PS1, Line 24: {0x00,PCI_DEVFN(0x1f, 0), {{0x62, 0xdcf8}, {0x63, 0xdcd8}, {0x00, 0x0000}, {0x00, 0x00000}}, 0x0, 0x0}, // LPC space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/8693b716_546f0530 PS1, Line 25: {0x00,PCI_DEVFN(0x1d, 0), {{0x6b, 0xdcf8}, {0x63, 0xdcd8}, {0x62, 0xdcf8}, {0x60, 0x0dcf8}}, 0x0, 0x0}, // USB#1 space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/5e29792f_42ff81fa PS1, Line 26: {0x00,PCI_DEVFN(0x1b, 0), {{0x60, 0xdcf8}, {0x00, 0x0000}, {0x00, 0x0000}, {0x00, 0x00000}}, 0x0, 0x0}, // Audio device space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/d9fcddd4_a04b16eb PS1, Line 27: {0x00,PCI_DEVFN(0x1c, 0), {{0x60, 0xdcf8}, {0x61, 0xdcf8}, {0x62, 0xdcf8}, {0x63, 0x0dcd8}}, 0x2, 0x0}, // PCIe bridge space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/80ebebc7_9f06c1da PS1, Line 28: {0x04,PCI_DEVFN(0, 0), {{0x60, 0xdcf8}, {0x00, 0x0000}, {0x00, 0x0000}, {0x00, 0x00000}}, 0x0, 0x0}, // Firewire space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/09f5b0ac_d7855534 PS1, Line 29: {0x04,PCI_DEVFN(0x01, 0), {{0x68, 0xdcf8}, {0x69, 0xdcf8}, {0x6a, 0xdcf8}, {0x6b, 0x0dcf8}}, 0x1, 0x0}, // PCI Bridge space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/4ce124bf_d4371562 PS1, Line 30: {0x04,PCI_DEVFN(0x02, 0), {{0x69, 0xdcf8}, {0x6a, 0xdcf8}, {0x6b, 0xdcf8}, {0x68, 0x0dcf8}}, 0x2, 0x0}, space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/d7668434_f4269a6a PS1, Line 31: {0x04,PCI_DEVFN(0x03, 0), {{0x6a, 0xdcf8}, {0x6b, 0xdcf8}, {0x68, 0xdcf8}, {0x69, 0x0dcf8}}, 0x3, 0x0}, space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/e18be361_59a9df34 PS1, Line 32: {0x04,PCI_DEVFN(0x04, 0), {{0x6b, 0xdcf8}, {0x68, 0xdcf8}, {0x69, 0xdcf8}, {0x6a, 0x0dcf8}}, 0x4, 0x0}, space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/83776cfa_6bf2af6f PS1, Line 33: {0x04,PCI_DEVFN(0x05, 0), {{0x63, 0xdcd8}, {0x62, 0xdcf8}, {0x61, 0xdcf8}, {0x60, 0x0dcf8}}, 0x5, 0x0}, space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/26c4ff72_e6959037 PS1, Line 34: {0x04,PCI_DEVFN(0x06, 0), {{0x62, 0xdcf8}, {0x61, 0xdcf8}, {0x60, 0xdcf8}, {0x63, 0x0dcd8}}, 0x6, 0x0}, space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/bba7c83c_baebbeda PS1, Line 35: {0x04,PCI_DEVFN(0x09, 0), {{0x69, 0xdcf8}, {0x6a, 0xdcf8}, {0x6b, 0xdcf8}, {0x68, 0x0dcf8}}, 0x9, 0x0}, space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/10273366_aadd5bc6 PS1, Line 36: {0x01,PCI_DEVFN(0, 0), {{0x60, 0xdcf8}, {0x61, 0xdcf8}, {0x62, 0xdcf8}, {0x63, 0x0dcd8}}, 0x0, 0x0}, // Ethernet 8168 space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/989a84fe_c014eb69 PS1, Line 37: {0x02,PCI_DEVFN(0, 0), {{0x60, 0xdcf8}, {0x61, 0xdcf8}, {0x62, 0xdcf8}, {0x63, 0x0dcd8}}, 0x9, 0x0}, space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/76add2bc_4b764942 PS1, Line 38: {0x03,PCI_DEVFN(0, 0), {{0x61, 0xdcf8}, {0x62, 0xdcf8}, {0x63, 0xdcd8}, {0x60, 0x0dcf8}}, 0xa, 0x0}, space required after that ',' (ctx:VxV)
File src/mainboard/ibase/mb899/irq_tables.c:
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/031f10f9_b372f81c PS1, Line 21: {0x00,PCI_DEVFN(0x01, 0), {{0x60, 0xdcf8}, {0x61, 0xdcf8}, {0x62, 0xdcf8}, {0x63, 0x0dcd8}}, 0x0, 0x0}, // PCIe? space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/adc8d02d_acbeb14b PS1, Line 22: {0x00,PCI_DEVFN(0x02, 0), {{0x60, 0xdcf8}, {0x00, 0x0000}, {0x00, 0x0000}, {0x00, 0x00000}}, 0x0, 0x0}, // VGA space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/c7384566_3c4db15f PS1, Line 23: {0x00,PCI_DEVFN(0x1e, 0), {{0x61, 0xdcf8}, {0x68, 0xdcf8}, {0x00, 0x0000}, {0x00, 0x00000}}, 0x0, 0x0}, // PCI bridge space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/3c1b806e_505dc3f4 PS1, Line 24: {0x00,PCI_DEVFN(0x1f, 0), {{0x62, 0xdcf8}, {0x63, 0xdcd8}, {0x00, 0x0000}, {0x00, 0x00000}}, 0x0, 0x0}, // LPC space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/8ffc8e51_3a9c19bf PS1, Line 25: {0x00,PCI_DEVFN(0x1d, 0), {{0x6b, 0xdcf8}, {0x63, 0xdcd8}, {0x62, 0xdcf8}, {0x60, 0x0dcf8}}, 0x0, 0x0}, // USB#1 space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/5c1b7f9b_20409572 PS1, Line 26: {0x00,PCI_DEVFN(0x1b, 0), {{0x60, 0xdcf8}, {0x00, 0x0000}, {0x00, 0x0000}, {0x00, 0x00000}}, 0x0, 0x0}, // Audio device space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/d09271c3_ec8c1a84 PS1, Line 27: {0x00,PCI_DEVFN(0x1c, 0), {{0x60, 0xdcf8}, {0x61, 0xdcf8}, {0x62, 0xdcf8}, {0x63, 0x0dcd8}}, 0x2, 0x0}, // PCIe bridge space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/8111c716_9cdb34cd PS1, Line 28: {0x04,PCI_DEVFN(0, 0), {{0x60, 0xdcf8}, {0x00, 0x0000}, {0x00, 0x0000}, {0x00, 0x00000}}, 0x0, 0x0}, // Firewire space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/4a434b69_a01f1bfa PS1, Line 29: {0x04,PCI_DEVFN(0x01, 0), {{0x68, 0xdcf8}, {0x69, 0xdcf8}, {0x6a, 0xdcf8}, {0x6b, 0x0dcf8}}, 0x1, 0x0}, // PCI Bridge space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/e2f595b9_0ad55ac9 PS1, Line 30: {0x04,PCI_DEVFN(0x02, 0), {{0x69, 0xdcf8}, {0x6a, 0xdcf8}, {0x6b, 0xdcf8}, {0x68, 0x0dcf8}}, 0x2, 0x0}, space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/100abcfc_5f2caeac PS1, Line 31: {0x04,PCI_DEVFN(0x03, 0), {{0x6a, 0xdcf8}, {0x6b, 0xdcf8}, {0x68, 0xdcf8}, {0x69, 0x0dcf8}}, 0x3, 0x0}, space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/a7aa0c17_9d79bbf3 PS1, Line 32: {0x04,PCI_DEVFN(0x04, 0), {{0x6b, 0xdcf8}, {0x68, 0xdcf8}, {0x69, 0xdcf8}, {0x6a, 0x0dcf8}}, 0x4, 0x0}, space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/a8692d64_42bed439 PS1, Line 33: {0x04,PCI_DEVFN(0x05, 0), {{0x63, 0xdcd8}, {0x62, 0xdcf8}, {0x61, 0xdcf8}, {0x60, 0x0dcf8}}, 0x5, 0x0}, space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/e0de3b61_e6948114 PS1, Line 34: {0x04,PCI_DEVFN(0x06, 0), {{0x62, 0xdcf8}, {0x61, 0xdcf8}, {0x60, 0xdcf8}, {0x63, 0x0dcd8}}, 0x6, 0x0}, space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/c6720bbe_a3a1cceb PS1, Line 35: {0x04,PCI_DEVFN(0x09, 0), {{0x69, 0xdcf8}, {0x6a, 0xdcf8}, {0x6b, 0xdcf8}, {0x68, 0x0dcf8}}, 0x9, 0x0}, space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/2020bb70_5a079e3b PS1, Line 36: {0x01,PCI_DEVFN(0, 0), {{0x60, 0xdcf8}, {0x61, 0xdcf8}, {0x62, 0xdcf8}, {0x63, 0x0dcd8}}, 0x0, 0x0}, // Ethernet Marvell 88E8053 space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/d28fed0c_970f9c10 PS1, Line 37: {0x02,PCI_DEVFN(0, 0), {{0x60, 0xdcf8}, {0x61, 0xdcf8}, {0x62, 0xdcf8}, {0x63, 0x0dcd8}}, 0x9, 0x0}, space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/e6a1d065_b74d7ffa PS1, Line 38: {0x03,PCI_DEVFN(0, 0), {{0x61, 0xdcf8}, {0x62, 0xdcf8}, {0x63, 0xdcd8}, {0x60, 0x0dcf8}}, 0xa, 0x0}, space required after that ',' (ctx:VxV)
File src/mainboard/intel/d945gclf/irq_tables.c:
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/36bcad06_939cd32f PS1, Line 21: {0x00,PCI_DEVFN(0x01, 0), {{0x60, 0xdcf8}, {0x61, 0xdcf8}, {0x62, 0xdcf8}, {0x63, 0x0dcd8}}, 0x0, 0x0}, // PCIe? space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/da8ad386_5a04335b PS1, Line 22: {0x00,PCI_DEVFN(0x02, 0), {{0x60, 0xdcf8}, {0x00, 0x0000}, {0x00, 0x0000}, {0x00, 0x00000}}, 0x0, 0x0}, // VGA space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/856bf62a_b7ec45fb PS1, Line 23: {0x00,PCI_DEVFN(0x1e, 0), {{0x61, 0xdcf8}, {0x68, 0xdcf8}, {0x00, 0x0000}, {0x00, 0x00000}}, 0x0, 0x0}, // PCI bridge space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/9f604963_5d32d8ff PS1, Line 24: {0x00,PCI_DEVFN(0x1f, 0), {{0x62, 0xdcf8}, {0x63, 0xdcd8}, {0x00, 0x0000}, {0x00, 0x00000}}, 0x0, 0x0}, // LPC space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/f4faeeb8_a39953e3 PS1, Line 25: {0x00,PCI_DEVFN(0x1d, 0), {{0x6b, 0xdcf8}, {0x63, 0xdcd8}, {0x62, 0xdcf8}, {0x60, 0x0dcf8}}, 0x0, 0x0}, // USB#1 space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/9b12b46b_ad39acc3 PS1, Line 26: {0x00,PCI_DEVFN(0x1b, 0), {{0x60, 0xdcf8}, {0x00, 0x0000}, {0x00, 0x0000}, {0x00, 0x00000}}, 0x0, 0x0}, // Audio device space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/c4a74fd8_dc101fb1 PS1, Line 27: {0x00,PCI_DEVFN(0x1c, 0), {{0x60, 0xdcf8}, {0x61, 0xdcf8}, {0x62, 0xdcf8}, {0x63, 0x0dcd8}}, 0x2, 0x0}, // PCIe bridge space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/21bfb423_157111e4 PS1, Line 28: {0x04,PCI_DEVFN(0, 0), {{0x60, 0xdcf8}, {0x00, 0x0000}, {0x00, 0x0000}, {0x00, 0x00000}}, 0x0, 0x0}, // Firewire space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/16e2facd_3d8753c0 PS1, Line 29: {0x04,PCI_DEVFN(0x01, 0), {{0x68, 0xdcf8}, {0x69, 0xdcf8}, {0x6a, 0xdcf8}, {0x6b, 0x0dcf8}}, 0x1, 0x0}, // PCI Bridge space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/c440c7eb_ffda8759 PS1, Line 30: {0x04,PCI_DEVFN(0x02, 0), {{0x69, 0xdcf8}, {0x6a, 0xdcf8}, {0x6b, 0xdcf8}, {0x68, 0x0dcf8}}, 0x2, 0x0}, space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/0582c993_0ea83dc5 PS1, Line 31: {0x04,PCI_DEVFN(0x03, 0), {{0x6a, 0xdcf8}, {0x6b, 0xdcf8}, {0x68, 0xdcf8}, {0x69, 0x0dcf8}}, 0x3, 0x0}, space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/d102656f_dabd1fd9 PS1, Line 32: {0x04,PCI_DEVFN(0x04, 0), {{0x6b, 0xdcf8}, {0x68, 0xdcf8}, {0x69, 0xdcf8}, {0x6a, 0x0dcf8}}, 0x4, 0x0}, space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/882e064f_4c6c972f PS1, Line 33: {0x04,PCI_DEVFN(0x05, 0), {{0x63, 0xdcd8}, {0x62, 0xdcf8}, {0x61, 0xdcf8}, {0x60, 0x0dcf8}}, 0x5, 0x0}, space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/29b76a8d_573027b9 PS1, Line 34: {0x04,PCI_DEVFN(0x06, 0), {{0x62, 0xdcf8}, {0x61, 0xdcf8}, {0x60, 0xdcf8}, {0x63, 0x0dcd8}}, 0x6, 0x0}, space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/439bb3d7_4e816ee9 PS1, Line 35: {0x04,PCI_DEVFN(0x09, 0), {{0x69, 0xdcf8}, {0x6a, 0xdcf8}, {0x6b, 0xdcf8}, {0x68, 0x0dcf8}}, 0x9, 0x0}, space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/5bf4c5f2_cf8ccc16 PS1, Line 36: {0x01,PCI_DEVFN(0, 0), {{0x60, 0xdcf8}, {0x61, 0xdcf8}, {0x62, 0xdcf8}, {0x63, 0x0dcd8}}, 0x0, 0x0}, // Ethernet 8168 space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/a1ee8d1d_80b87ab4 PS1, Line 37: {0x02,PCI_DEVFN(0, 0), {{0x60, 0xdcf8}, {0x61, 0xdcf8}, {0x62, 0xdcf8}, {0x63, 0x0dcd8}}, 0x9, 0x0}, space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/8f912fee_20ba70ae PS1, Line 38: {0x03,PCI_DEVFN(0, 0), {{0x61, 0xdcf8}, {0x62, 0xdcf8}, {0x63, 0xdcd8}, {0x60, 0x0dcf8}}, 0xa, 0x0}, space required after that ',' (ctx:VxV)
File src/mainboard/kontron/986lcd-m/irq_tables.c:
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/cbf86aee_1d075449 PS1, Line 21: {0x00,PCI_DEVFN(0x01, 0), {{0x60, 0xdcf8}, {0x61, 0xdcf8}, {0x62, 0xdcf8}, {0x63, 0x0dcd8}}, 0x0, 0x0}, /* PCIe? */ space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/f27542fa_e8134d06 PS1, Line 22: {0x00,PCI_DEVFN(0x02, 0), {{0x60, 0xdcf8}, {0x00, 0x0000}, {0x00, 0x0000}, {0x00, 0x00000}}, 0x0, 0x0}, /* VGA */ space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/defdd7a4_c65f407a PS1, Line 23: {0x00,PCI_DEVFN(0x1e, 0), {{0x61, 0xdcf8}, {0x68, 0xdcf8}, {0x00, 0x0000}, {0x00, 0x00000}}, 0x0, 0x0}, /* PCI bridge */ space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/aef5fa9e_3ba44bc1 PS1, Line 24: {0x00,PCI_DEVFN(0x1f, 0), {{0x62, 0xdcf8}, {0x63, 0xdcd8}, {0x00, 0x0000}, {0x00, 0x00000}}, 0x0, 0x0}, /* LPC */ space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/d9bfe28d_124f80f6 PS1, Line 25: {0x00,PCI_DEVFN(0x1d, 0), {{0x6b, 0xdcf8}, {0x63, 0xdcd8}, {0x62, 0xdcf8}, {0x60, 0x0dcf8}}, 0x0, 0x0}, /* USB#1 */ space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/350ff2c6_2a8878a4 PS1, Line 26: {0x00,PCI_DEVFN(0x1b, 0), {{0x60, 0xdcf8}, {0x00, 0x0000}, {0x00, 0x0000}, {0x00, 0x00000}}, 0x0, 0x0}, /* Audio device */ space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/8382822d_a45f791a PS1, Line 27: {0x00,PCI_DEVFN(0x1c, 0), {{0x60, 0xdcf8}, {0x61, 0xdcf8}, {0x62, 0xdcf8}, {0x63, 0x0dcd8}}, 0x2, 0x0}, /* PCIe bridge */ space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/6824ac51_697f632c PS1, Line 28: {0x04,PCI_DEVFN(0, 0), {{0x60, 0xdcf8}, {0x00, 0x0000}, {0x00, 0x0000}, {0x00, 0x00000}}, 0x0, 0x0}, /* Firewire */ space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/70da4639_045adeb5 PS1, Line 29: {0x04,PCI_DEVFN(0x01, 0), {{0x68, 0xdcf8}, {0x69, 0xdcf8}, {0x6a, 0xdcf8}, {0x6b, 0x0dcf8}}, 0x1, 0x0}, /* PCI Bridge */ space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/13bd2e69_3d9a144e PS1, Line 30: {0x04,PCI_DEVFN(0x02, 0), {{0x69, 0xdcf8}, {0x6a, 0xdcf8}, {0x6b, 0xdcf8}, {0x68, 0x0dcf8}}, 0x2, 0x0}, space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/f1fb8a52_5aa48179 PS1, Line 31: {0x04,PCI_DEVFN(0x03, 0), {{0x6a, 0xdcf8}, {0x6b, 0xdcf8}, {0x68, 0xdcf8}, {0x69, 0x0dcf8}}, 0x3, 0x0}, space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/385f5e9a_c56d2deb PS1, Line 32: {0x04,PCI_DEVFN(0x04, 0), {{0x6b, 0xdcf8}, {0x68, 0xdcf8}, {0x69, 0xdcf8}, {0x6a, 0x0dcf8}}, 0x4, 0x0}, space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/31b96d69_514f5b25 PS1, Line 33: {0x04,PCI_DEVFN(0x05, 0), {{0x63, 0xdcd8}, {0x62, 0xdcf8}, {0x61, 0xdcf8}, {0x60, 0x0dcf8}}, 0x5, 0x0}, space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/a9d718c8_40732147 PS1, Line 34: {0x04,PCI_DEVFN(0x06, 0), {{0x62, 0xdcf8}, {0x61, 0xdcf8}, {0x60, 0xdcf8}, {0x63, 0x0dcd8}}, 0x6, 0x0}, space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/f486e7dd_4f967718 PS1, Line 35: {0x04,PCI_DEVFN(0x09, 0), {{0x69, 0xdcf8}, {0x6a, 0xdcf8}, {0x6b, 0xdcf8}, {0x68, 0x0dcf8}}, 0x9, 0x0}, space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/a34f5410_a0f3e47f PS1, Line 36: {0x01,PCI_DEVFN(0, 0), {{0x60, 0xdcf8}, {0x61, 0xdcf8}, {0x62, 0xdcf8}, {0x63, 0x0dcd8}}, 0x0, 0x0}, /* Ethernet 8168 */ space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/8f747655_7723e0e3 PS1, Line 37: {0x02,PCI_DEVFN(0, 0), {{0x60, 0xdcf8}, {0x61, 0xdcf8}, {0x62, 0xdcf8}, {0x63, 0x0dcd8}}, 0x9, 0x0}, space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/5e769808_df83fe00 PS1, Line 38: {0x03,PCI_DEVFN(0, 0), {{0x61, 0xdcf8}, {0x62, 0xdcf8}, {0x63, 0xdcd8}, {0x60, 0x0dcf8}}, 0xa, 0x0}, space required after that ',' (ctx:VxV)
File src/mainboard/roda/rk886ex/irq_tables.c:
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/59cf91b6_356c4255 PS1, Line 21: {0x00,PCI_DEVFN(0x01, 0), {{0x60, 0xdcf8}, {0x61, 0xdcf8}, {0x62, 0xdcf8}, {0x63, 0x0dcd8}}, 0x0, 0x0}, /* PCIe? */ space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/715eb15f_1a52de86 PS1, Line 22: {0x00,PCI_DEVFN(0x02, 0), {{0x60, 0xdcf8}, {0x00, 0x0000}, {0x00, 0x0000}, {0x00, 0x00000}}, 0x0, 0x0}, /* VGA */ space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/6032d8e1_2a0a32d1 PS1, Line 23: {0x00,PCI_DEVFN(0x1e, 0), {{0x61, 0xdcf8}, {0x68, 0xdcf8}, {0x00, 0x0000}, {0x00, 0x00000}}, 0x0, 0x0}, /* PCI bridge */ space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/e392bcd3_efcf0349 PS1, Line 24: {0x00,PCI_DEVFN(0x1f, 0), {{0x62, 0xdcf8}, {0x63, 0xdcd8}, {0x00, 0x0000}, {0x00, 0x00000}}, 0x0, 0x0}, /* LPC */ space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/da1778a4_72689f2b PS1, Line 25: {0x00,PCI_DEVFN(0x1d, 0), {{0x6b, 0xdcf8}, {0x63, 0xdcd8}, {0x62, 0xdcf8}, {0x60, 0x0dcf8}}, 0x0, 0x0}, /* USB#1 */ space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/c5585b8f_98b90601 PS1, Line 26: {0x00,PCI_DEVFN(0x1b, 0), {{0x60, 0xdcf8}, {0x00, 0x0000}, {0x00, 0x0000}, {0x00, 0x00000}}, 0x0, 0x0}, /* Audio device */ space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/718d8f0c_15d244cf PS1, Line 27: {0x00,PCI_DEVFN(0x1c, 0), {{0x60, 0xdcf8}, {0x61, 0xdcf8}, {0x62, 0xdcf8}, {0x63, 0x0dcd8}}, 0x2, 0x0}, /* PCIe bridge */ space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/f020f87f_2cfe6d9e PS1, Line 28: {0x04,PCI_DEVFN(0, 0), {{0x60, 0xdcf8}, {0x00, 0x0000}, {0x00, 0x0000}, {0x00, 0x00000}}, 0x0, 0x0}, /* Firewire */ space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/0ebb3914_95d4b3d6 PS1, Line 29: {0x04,PCI_DEVFN(0x01, 0), {{0x68, 0xdcf8}, {0x69, 0xdcf8}, {0x6a, 0xdcf8}, {0x6b, 0x0dcf8}}, 0x1, 0x0}, /* PCI Bridge */ space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/8f5fa252_c0f204e2 PS1, Line 30: {0x04,PCI_DEVFN(0x02, 0), {{0x69, 0xdcf8}, {0x6a, 0xdcf8}, {0x6b, 0xdcf8}, {0x68, 0x0dcf8}}, 0x2, 0x0}, space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/90a9753d_210109b1 PS1, Line 31: {0x04,PCI_DEVFN(0x03, 0), {{0x6a, 0xdcf8}, {0x6b, 0xdcf8}, {0x68, 0xdcf8}, {0x69, 0x0dcf8}}, 0x3, 0x0}, space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/928677ea_feeef9ec PS1, Line 32: {0x04,PCI_DEVFN(0x04, 0), {{0x6b, 0xdcf8}, {0x68, 0xdcf8}, {0x69, 0xdcf8}, {0x6a, 0x0dcf8}}, 0x4, 0x0}, space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/31486310_f2e7314d PS1, Line 33: {0x04,PCI_DEVFN(0x05, 0), {{0x63, 0xdcd8}, {0x62, 0xdcf8}, {0x61, 0xdcf8}, {0x60, 0x0dcf8}}, 0x5, 0x0}, space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/10dfa58e_7d5d4b2f PS1, Line 34: {0x04,PCI_DEVFN(0x06, 0), {{0x62, 0xdcf8}, {0x61, 0xdcf8}, {0x60, 0xdcf8}, {0x63, 0x0dcd8}}, 0x6, 0x0}, space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/2756b4ce_7a2abf43 PS1, Line 35: {0x04,PCI_DEVFN(0x09, 0), {{0x69, 0xdcf8}, {0x6a, 0xdcf8}, {0x6b, 0xdcf8}, {0x68, 0x0dcf8}}, 0x9, 0x0}, space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/cdc9d5c1_0b85d6b6 PS1, Line 36: {0x01,PCI_DEVFN(0, 0), {{0x60, 0xdcf8}, {0x61, 0xdcf8}, {0x62, 0xdcf8}, {0x63, 0x0dcd8}}, 0x0, 0x0}, /* Ethernet 8168 */ space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/f1adb1ab_22ce459f PS1, Line 37: {0x02,PCI_DEVFN(0, 0), {{0x60, 0xdcf8}, {0x61, 0xdcf8}, {0x62, 0xdcf8}, {0x63, 0x0dcd8}}, 0x9, 0x0}, space required after that ',' (ctx:VxV)
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-161862): https://review.coreboot.org/c/coreboot/+/69066/comment/fd623774_8070cf3a PS1, Line 38: {0x03,PCI_DEVFN(0, 0), {{0x61, 0xdcf8}, {0x62, 0xdcf8}, {0x63, 0xdcd8}, {0x60, 0x0dcf8}}, 0xa, 0x0}, space required after that ',' (ctx:VxV)