On May 7 2007 14:03, David Miller wrote: > >diff --git a/drivers/video/sunxvr2500.c b/drivers/video/sunxvr2500.c >index 4010492..4316c7f 100644 >--- a/drivers/video/sunxvr2500.c >+++ b/drivers/video/sunxvr2500.c >@@ -237,14 +237,14 @@ static void __devexit s3d_pci_unregister(struct pci_dev *pdev) > } > > static struct pci_device_id s3d_pci_table[] = { >- { /* XVR-2500 */ >- PCI_DEVICE(PCI_VENDOR_ID_3DLABS, 0x0032), >- .driver_data = 1, >- }, >- { /* XVR-500 */ >- PCI_DEVICE(PCI_VENDOR_ID_3DLABS, 0x07a2), >- .driver_data = 0, >- }, >+ { PCI_DEVICE(PCI_VENDOR_ID_3DLABS, 0x002c), }, >+ { PCI_DEVICE(PCI_VENDOR_ID_3DLABS, 0x002d), }, >+ { PCI_DEVICE(PCI_VENDOR_ID_3DLABS, 0x002e), }, >+ { PCI_DEVICE(PCI_VENDOR_ID_3DLABS, 0x002f), }, >+ { PCI_DEVICE(PCI_VENDOR_ID_3DLABS, 0x0030), }, >+ { PCI_DEVICE(PCI_VENDOR_ID_3DLABS, 0x0031), }, >+ { PCI_DEVICE(PCI_VENDOR_ID_3DLABS, 0x0032), }, >+ { PCI_DEVICE(PCI_VENDOR_ID_3DLABS, 0x0033), }, The extra comma inside the { } is not needed, is it? Jan -- - To unsubscribe from this list: send the line "unsubscribe sparclinux" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html