gen_pci_vendors: Add new script

[manu]

Summary

This script takes a pci_vendor file and generate a C header.

MFC after: 3 days
Sponsored by: Beckhoff Automation GmbH & Co. KG

[corvink]

Not sure if it's worth parsing the header file and the pci.ids file. If you want to update the header file, you can simply parse the pci.ids file and diff the output or you simply overwrite the header file and use git clean up the diff.

sys/tools/gen_pci_vendors.awk

42

Should we be more strict and check for 4 digits?

46–51

The order makes no sense. In the first step, you're replacing e.g. ( with _. In a later step, you try to replace \(.*\) with "". It should be:

			gsub(/\[.+\]/, "", key)
			gsub(/\(.+\)/, "", key)
			gsub(/[ \.\)\(-\/\+\',#]/, "_", key)
			gsub(/["\?&]/, "", key)
			gsub(/__/, "_", key)
			gsub(/_$/, "", key)

Btw. removing everything between brackets can lead to duplicate keys. E.g.:

1002  Advanced Micro Devices, Inc. [AMD/ATI]
1022  Advanced Micro Devices, Inc. [AMD]

54

Maybe adding a warning for duplicate keys? Therefore, you should parse pci.ids first and then the header file.

57

Nit! Or even "_" toupper(header_file) "_", would avoid the conversion some lines below.

65–69

Parsing the header file and the pci.ids file is currently broken. Both files are using a different key. So, when the header file already contains some values, you're going to add them a second time. So, just use the same key for header and pci.ids file by prepending PCI_VENDOR_ to the pci.ids key.