Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/39963 )
Change subject: nicintel: add I350 support ......................................................................
Patch Set 6: Code-Review+1
(2 comments)
https://review.coreboot.org/c/flashrom/+/39963/6//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/flashrom/+/39963/6//COMMIT_MSG@10 PS6, Line 10: some things Mention it was necessary to change the type of the base address variables?
https://review.coreboot.org/c/flashrom/+/39963/6/nicintel_spi.c File nicintel_spi.c:
https://review.coreboot.org/c/flashrom/+/39963/6/nicintel_spi.c@110 PS6, Line 110: {PCI_VENDOR_ID_INTEL, 0x151f, NT, "Intel", "I350 Gigabit Network Connection Unprogrammed"}, : {PCI_VENDOR_ID_INTEL, 0x1521, NT, "Intel", "I350 Gigabit Network Connection Copper"}, : {PCI_VENDOR_ID_INTEL, 0x1522, NT, "Intel", "I350 Gigabit Network Connection Fiber"}, : {PCI_VENDOR_ID_INTEL, 0x1523, NT, "Intel", "I350 Gigabit Network Connection 1000BASE-KX / 1000BASE-BX backplane"}, : {PCI_VENDOR_ID_INTEL, 0x1524, NT, "Intel", "I350 Gigabit Network Connection External SGMII PHY"}, None of these are tested?