pata_sc1200: Fix crash on boot
authorAlan Cox <alan@linux.intel.com>
Tue, 6 Oct 2009 15:07:51 +0000 (16:07 +0100)
committerJeff Garzik <jgarzik@redhat.com>
Fri, 16 Oct 2009 10:22:40 +0000 (06:22 -0400)
The SC1200 needs a NULL terminator or it may cause a crash on boot.

Bug #14227

Also correct a bogus comment as the driver had serializing added so can run
dual port.

Signed-off-by: Alan Cox <alan@linux.intel.com>
Signed-off-by: Jeff Garzik <jgarzik@redhat.com>
drivers/ata/pata_sc1200.c

index f49814d6fd2eaf8de34566542fb40e15be8788fa..3bbed8322ecf0cb72bf57d23e0c143cd22aeca49 100644 (file)
@@ -235,8 +235,7 @@ static int sc1200_init_one(struct pci_dev *dev, const struct pci_device_id *id)
                .udma_mask = ATA_UDMA2,
                .port_ops = &sc1200_port_ops
        };
-       /* Can't enable port 2 yet, see top comments */
-       const struct ata_port_info *ppi[] = { &info, };
+       const struct ata_port_info *ppi[] = { &info, NULL };
 
        return ata_pci_sff_init_one(dev, ppi, &sc1200_sht, NULL);
 }