From: Jan Engelhardt <[EMAIL PROTECTED]>
Date: Tue, 8 May 2007 10:50:56 +0200 (MEST)

> 
> 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?

No, but if later I want to fill in more fields per entry I will indeed
need that comma there.
-
To unsubscribe from this list: send the line "unsubscribe sparclinux" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to