pci_ids.h - duplicate entry and many out of order definitions

From: David Wilson
Date: Tue Jul 29 2008 - 19:00:27 EST


I was browsing the pci_ids.h file and came across a duplicate entry
for PCI_VENDOR_ID_RDC. This prompted me to look further and I found a
number of out-of-order definitions (based on the comment at the head
of the file that indicates that they should be sorted). I decided to
write a script to check this and it found 74 "errors".

1) Is it worth my time generating a patch to fix all these mis-orderings?
2) Should it be one large patch or many small ones (eg one per PCI_VENDOR)?
3) Is there a maintainer for this file?

Here is an example of the output my script generates (on the
2.6.27-rc1 pci_ids.h file):

line 153: PCI_DEVICE_ID_COMPAQ_NETEL10 0xae34
line 154: PCI_DEVICE_ID_COMPAQ_TRIFLEX_IDE 0xae33

line 160: PCI_DEVICE_ID_COMPAQ_CISSB 0xb178
line 161: PCI_DEVICE_ID_COMPAQ_CISSC 0x46

line 193: PCI_DEVICE_ID_LSI_FC939X 0x0642
line 194: PCI_DEVICE_ID_LSI_FC949X 0x0640

line 195: PCI_DEVICE_ID_LSI_FC949ES 0x0646
line 196: PCI_DEVICE_ID_LSI_FC919X 0x0628

line 199: PCI_DEVICE_ID_LSI_63C815 0x1000
line 200: PCI_DEVICE_ID_LSI_SAS1064 0x0050

line 201: PCI_DEVICE_ID_LSI_SAS1064R 0x0411
line 202: PCI_DEVICE_ID_LSI_SAS1066 0x005E
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/