Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/37548 )
Change subject: New PCI ID for Family 15h (Models 60h-6fh) I/O Memory Management Unit ......................................................................
Patch Set 1:
(2 comments)
Should the macro be renamed?
https://review.coreboot.org/c/coreboot/+/37548/1//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/37548/1//COMMIT_MSG@7 PS1, Line 7: New PCI ID for Family 15h (Models 60h-6fh) I/O Memory Management Unit Please rephrase.
https://review.coreboot.org/c/coreboot/+/37548/1/src/include/device/pci_ids.... File src/include/device/pci_ids.h:
https://review.coreboot.org/c/coreboot/+/37548/1/src/include/device/pci_ids.... PS1, Line 307: #define PCI_DEVICE_ID_AMD_15H_MODEL_707F_NB_IOMMU 0x1577 It’s the same ID.