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 2:
(122 comments)
File src/mainboard/asus/am1i-a/irq_tables.c:
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/61949a30_6943916e PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/8045287a_e03051c7 PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/8d947e9a_c55e3de1 PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/ef4726c9_f4e6386c PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/15ce5c5c_88ab4564 PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/be042625_3dcb6088 PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/ac8d2b6b_9d0fab25 PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/626f891e_02289e6e PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/8e4e97df_1bec47d1 PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/4dde7306_afc1a510 PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/918764f8_88587f8f PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/0fa7bd81_12b54729 PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/0edbb5c5_d0e460f8 PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/b67e3356_8bce2925 PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/108ca86a_7986c6f4 PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/8bf90a0f_c07ef32a PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/8870758b_9cf5624c PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/dcf7d782_656259e1 PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/e789d54b_4a6d139d PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/19d91174_08d5d02d PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/e20f6eaf_d0056d5e PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/417dbe7c_d4107bcc PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/c2ba5efe_a552cca5 PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/ecf8d95e_f8ba9751 PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/9df9d635_4c2ac603 PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/5979e9e3_16163dab PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/eba0252b_57abe75d PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/82707201_a6c51e6c PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/7bfc714c_8741bfac PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/a41942d9_a0d163cd PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/10df67b4_782bd8b9 PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/6477bbac_d67275e0 PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/bfd9afb7_e1c36e22 PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/31cdc83e_79369f51 PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/20880518_359d732c PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/0f98992f_4655a46e PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/3b840901_4b247a8a PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/a2423c04_e01678ba PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/2410b4fd_12175e2e PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/8d7a5468_85d0aa8f PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/b7e654f8_5e4d8f8e PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/7b2e17bf_8d9f3207 PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/12426b1f_1073cd0e PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/6e419a0a_9377d140 PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/be0f1354_d18646b7 PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/51a088c6_b8e8a482 PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/61dd7976_583dd44c PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/b6ac7a32_a1253efc PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/2f05be52_85e9bbad PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/ec0bddc8_4b7e0751 PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/611cf295_b080d638 PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/96f12868_1a63abba PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/2c24a413_b73fa446 PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/04fd6f03_ab8cdd8e PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/224f6c65_e7e0f2f8 PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/7225a53c_ce66521c PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/7a7bfe00_1d46b347 PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/3f298b87_7d9a606f PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/13b97fe5_4fdf90c8 PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/d943fe1c_70eeebb4 PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/435a3663_8b354800 PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/7d225a22_d7a022e0 PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/c35b8638_a78ad99e PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/80fae7c7_71d8cee6 PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/27898aab_0b51772b PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/9469a9a3_8d967d86 PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/d851d22e_0cc51a8a PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/ea9af23a_8eb6a14f PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/3b183240_041b7706 PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/d4f76575_9b98b9b2 PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/a4357fe6_38736ac7 PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/fbeb5341_82903e6c PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/ccc5e06e_48385f91 PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/c031de03_ba92f5f3 PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/809053d8_35113883 PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/8bdafcd4_2cb656e1 PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/ca5ec409_2d5ea10b PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/9ec50ebc_d8a2997b PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/127865b0_fa8967d9 PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/3e69a591_6167354a PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/90ab2a96_7bceaf1b PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/90943647_3ea249d3 PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/2489ebc3_073fbea9 PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/b3919938_4dc8a531 PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/8e79ef10_7dbeb44b PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/9cbb2f86_9fca4456 PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/704fec02_0cd2d233 PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/ab81424b_63b2e4ed PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/bbe6017f_cc8b4e48 PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/d5d2caae_6bb67ab7 PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/5d9b56d7_9d60998c PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/ed71eadb_06c96267 PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/610365ee_ea1fcd9c PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/62ead0a8_1c1f41d5 PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/0babbd0f_8dec4b03 PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/b6e73b63_8a341c82 PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/cbe5ab5a_29555815 PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/2d5f590e_d5776a4d PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/5a4608ff_7a2cf7e5 PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/68805ac1_656da2ed PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/8aa14d3c_62eeb5aa PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/450e5a58_07847c99 PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/cfebb01f_5478358b PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/38c4e850_02b55eaa PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/4bd49157_b5288701 PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/d4323ed9_543c7937 PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/c8467c69_1579e3d9 PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/1220d2e9_b2a1520c PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/511648ed_41ea14cb PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/42981bf3_92afe896 PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/a31382f8_62f48ca5 PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/a53e140e_456c679d PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/913f6f91_a56d6855 PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/2b18caa6_3d2980fe PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/2936c557_b5810c37 PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/781fbfe5_d8ae868d PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/7b589f63_fe511a5b PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/b751a6ea_5636e332 PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/570cea34_faf246c5 PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/a3c59434_29961502 PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/676849fa_081afc19 PS2, 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-162971): https://review.coreboot.org/c/coreboot/+/69066/comment/26334478_60136365 PS2, Line 38: {0x03,PCI_DEVFN(0, 0), {{0x61, 0xdcf8}, {0x62, 0xdcf8}, {0x63, 0xdcd8}, {0x60, 0x0dcf8}}, 0xa, 0x0}, space required after that ',' (ctx:VxV)
-
build bot (Jenkins) (Code Review)