> >> static struct pci_device_id serial_pci_tbl[] = { > >> + { PCI_VENDOR_ID_ADVANTECH, PCI_DEVICE_ID_ADVANTECH_PCI3620, > >> + 0x3620, PCI_ANY_ID, 0, 0, > > Why not use PCI_VENDOR_ID_ADVANTECH as PCI_SUBVENDOR_ID too? The Advantech vendor id is not 0x3620. This confused me as well which is why I asked for an lspci. Advantech has stuck the device id in the subvendor bits and '1' in the subdevice (so it should be 1 not PCI_ANY_ID). Alan -- To unsubscribe from this list: send the line "unsubscribe linux-serial" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html