Re: [PATCH v3 07/17] x86/pci: add Hygon PCI vendor and northbridge support

From: Pu Wen
Date: Tue Aug 14 2018 - 09:27:16 EST


On 2018/8/14 6:14, Bjorn Helgaas wrote:
On Sat, Aug 11, 2018 at 09:27:42PM +0800, Pu Wen wrote:
diff --git a/include/linux/pci_ids.h b/include/linux/pci_ids.h
index 2950223..d0e98a9 100644
--- a/include/linux/pci_ids.h
+++ b/include/linux/pci_ids.h
@@ -511,6 +511,8 @@
#define PCI_DEVICE_ID_AMI_MEGARAID 0x9010
#define PCI_DEVICE_ID_AMI_MEGARAID2 0x9060
+#define PCI_VENDOR_ID_HYGON 0x1d94

Please add this entry so pci_ids.h remains sorted by Vendor ID, then Device
ID, as the comment at the top suggests.

Thanks for the suggestion. Will place PCI_VENDOR_ID_HYGON between PCI_VENDOR_ID_AMAZON(0x1d0f) and PCI_VENDOR_ID_TEKRAM(0x1de1) in next
patch set.

Pu Wen


With that changed,

Acked-by: Bjorn Helgaas <bhelgaas@xxxxxxxxxx> # pci_ids.h

#define PCI_VENDOR_ID_AMD 0x1022
#define PCI_DEVICE_ID_AMD_K8_NB 0x1100
#define PCI_DEVICE_ID_AMD_K8_NB_ADDRMAP 0x1101
--
2.7.4