ide: remove leftover support for 82371FB (Step A1)

PATCH New Carlo Marcelo Arenas Belon 1 participant 1 message Nov 26, 2007

Share & tools

Carlo Marcelo Arenas Belon

author

2007-11-26 09:44 UTC #0 lore raw

PATCH 1/1 ide: remove leftover support for 82371FB (Step A1)

The following patch removes the remaining support for the 82371FB (Step A1) IDE chip which was added to ide.c in revision 1.53 and then removed (as it was never used in any profile and was considered dead code) in 1.57.

This code was added to piix_pci.c in revision 1.9 and after the corresponding code was removed from ide.c was generating the following warning :

qemu/hw/piix_pci.c:318: warning: 'piix_init' defined but not used

Carlo

---
Index: hw/piix_pci.c
===================================================================
RCS file: /sources/qemu/qemu/hw/piix_pci.c,v
retrieving revision 1.14

piix_pci.c +0 ~~-25~~

diff -u -p -r1.14 piix_pci.c
@@ -314,31 +314,6 @@ static int piix_load(QEMUFile* f, void *
     return pci_device_load(d, f);
 }
 
-static int piix_init(PCIBus *bus, int devfn)
-{
-    PCIDevice *d;
-    uint8_t *pci_conf;
-
-    d = pci_register_device(bus, "PIIX", sizeof(PCIDevice),
-                                    devfn, NULL, NULL);
-    register_savevm("PIIX", 0, 2, piix_save, piix_load, d);
-
-    piix3_dev = d;
-    pci_conf = d->config;
-
-    pci_conf[0x00] = 0x86; // Intel
-    pci_conf[0x01] = 0x80;
-    pci_conf[0x02] = 0x2E; // 82371FB PIIX PCI-to-ISA bridge
-    pci_conf[0x03] = 0x12;
-    pci_conf[0x08] = 0x02; // Step A1
-    pci_conf[0x0a] = 0x01; // class_sub = PCI_ISA
-    pci_conf[0x0b] = 0x06; // class_base = PCI_bridge
-    pci_conf[0x0e] = 0x80; // header_type = PCI_multifunction, generic
-
-    piix3_reset(d);
-    return d->devfn;
-}
-
 int piix3_init(PCIBus *bus, int devfn)
 {
     PCIDevice *d;