i82371eb: Drop support for older PIIX chips

All boards using this code use i82371eb (that shares PCI ID with i82371ab).
Dropping the code lightens compressed ramstage by a few dozen bytes.

Change-Id: Iab1e83b8f5fff44a33619c7925e5448169a2a87c
Signed-off-by: Keith Hui <buurin@gmail.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/38598
Reviewed-by: Patrick Georgi <pgeorgi@google.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>

4 files changed, 2 insertions, 83 deletions

diff --git a/src/southbridge/intel/i82371eb/i82371eb.c b/src/southbridge/intel/i82371eb/i82371eb.c
index 898cdffc25..02812ce40c 100644
--- a/src/southbridge/intel/i82371eb/i82371eb.c
+++ b/src/southbridge/intel/i82371eb/i82371eb.c
@@ -14,22 +14,9 @@
  * GNU General Public License for more details.
  */
 
-/* Note: This code supports the 82371FB/SB/MX/AB/EB/MB and 82437MX. */
+/* Note: This code supports the 82371AB/EB/MB. */
 
 /* Datasheets:
- *   - Name: 82371FB (PIIX) AND 82371SB (PIIX3) PCI ISA IDE XCELERATOR
- *   - URL: http://www.intel.com/design/intarch/datashts/290550.htm
- *   - PDF: http://download.intel.com/design/intarch/datashts/29055002.pdf
- *   - Date: April 1997
- *   - Order Number: 290550-002
- *
- *   - Name: 82371FB (PIIX) and 82371SB (PIIX3) PCI ISA IDE Xcelerator
- *           Specification Update
- *   - URL: http://www.intel.com/design/chipsets/specupdt/297658.htm
- *   - PDF: http://download.intel.com/design/chipsets/specupdt/29765801.pdf
- *   - Date: March 1998
- *   - Order Number: 297658-004
- *
  *   - Name: 82371AB PCI-TO-ISA / IDE XCELERATOR (PIIX4)
  *           (applies to 82371AB/EB/MB, a.k.a. PIIX4/PIIX4E/PIIX4M)
  *   - URL: http://www.intel.com/design/intarch/datashts/290562.htm
@@ -44,10 +31,8 @@
  *   - Order Number: 297738-017
  */
 
-/* TODO: List the other datasheets. */
-
 #include <device/device.h>
 
 const struct chip_operations southbridge_intel_i82371eb_ops = {
-	CHIP_NAME("Intel 82371FB/SB/MX/AB/EB/MB Southbridge")
+	CHIP_NAME("Intel 82371AB/EB/MB Southbridge")
 };
diff --git a/src/southbridge/intel/i82371eb/ide.c b/src/southbridge/intel/i82371eb/ide.c
index 7a72a6552d..1b8136a9ca 100644
--- a/src/southbridge/intel/i82371eb/ide.c
+++ b/src/southbridge/intel/i82371eb/ide.c
@@ -119,18 +119,6 @@ static void ide_init_udma33(struct device *dev)
 }
 
 /**
- * IDE init for the Intel 82371FB/SB IDE controller.
- *
- * These devices do not support UDMA/33, so don't attempt to enable it.
- *
- * @param dev The device to use.
- */
-static void ide_init_i82371fb_sb(struct device *dev)
-{
-	ide_init_enable(dev);
-}
-
-/**
  * IDE init for the Intel 82371AB/EB/MB IDE controller.
  *
  * @param dev The device to use.
@@ -141,17 +129,6 @@ static void ide_init_i82371ab_eb_mb(struct device *dev)
 	ide_init_udma33(dev);
 }
 
-/* Intel 82371FB/SB */
-static const struct device_operations ide_ops_fb_sb = {
-	.read_resources		= pci_dev_read_resources,
-	.set_resources		= pci_dev_set_resources,
-	.enable_resources	= pci_dev_enable_resources,
-	.init			= ide_init_i82371fb_sb,
-	.scan_bus		= 0,
-	.enable			= 0,
-	.ops_pci		= 0, /* No subsystem IDs on 82371XX! */
-};
-
 /* Intel 82371AB/EB/MB */
 static const struct device_operations ide_ops_ab_eb_mb = {
 	.read_resources		= pci_dev_read_resources,
@@ -163,34 +140,6 @@ static const struct device_operations ide_ops_ab_eb_mb = {
 	.ops_pci		= 0, /* No subsystem IDs on 82371XX! */
 };
 
-/* Intel 82371FB (PIIX) */
-static const struct pci_driver ide_driver_fb __pci_driver = {
-	.ops	= &ide_ops_fb_sb,
-	.vendor	= PCI_VENDOR_ID_INTEL,
-	.device	= PCI_DEVICE_ID_INTEL_82371FB_IDE,
-};
-
-/* Intel 82371SB (PIIX3) */
-static const struct pci_driver ide_driver_sb __pci_driver = {
-	.ops	= &ide_ops_fb_sb,
-	.vendor	= PCI_VENDOR_ID_INTEL,
-	.device	= PCI_DEVICE_ID_INTEL_82371SB_IDE,
-};
-
-/* Intel 82371MX (MPIIX) */
-static const struct pci_driver ide_driver_mx __pci_driver = {
-	.ops	= &ide_ops_fb_sb,
-	.vendor	= PCI_VENDOR_ID_INTEL,
-	.device	= PCI_DEVICE_ID_INTEL_82371MX_ISA_IDE,
-};
-
-/* Intel 82437MX (part of the 430MX chipset) */
-static const struct pci_driver ide_driver_82437mx __pci_driver = {
-	.ops	= &ide_ops_fb_sb,
-	.vendor	= PCI_VENDOR_ID_INTEL,
-	.device	= PCI_DEVICE_ID_INTEL_82437MX_ISA_IDE,
-};
-
 /* Intel 82371AB/EB/MB */
 static const struct pci_driver ide_driver_ab_eb_mb __pci_driver = {
 	.ops	= &ide_ops_ab_eb_mb,
diff --git a/src/southbridge/intel/i82371eb/isa.c b/src/southbridge/intel/i82371eb/isa.c
index a57d61ae03..fefead06e4 100644
--- a/src/southbridge/intel/i82371eb/isa.c
+++ b/src/southbridge/intel/i82371eb/isa.c
@@ -157,9 +157,3 @@ static const struct pci_driver isa_driver __pci_driver = {
 	.vendor	= PCI_VENDOR_ID_INTEL,
 	.device	= PCI_DEVICE_ID_INTEL_82371AB_ISA,
 };
-
-static const struct pci_driver isa_SB_driver __pci_driver = {
-	.ops	= &isa_ops,
-	.vendor	= PCI_VENDOR_ID_INTEL,
-	.device	= PCI_DEVICE_ID_INTEL_82371SB_ISA,
-};
diff --git a/src/southbridge/intel/i82371eb/usb.c b/src/southbridge/intel/i82371eb/usb.c
index 80b19a187e..38ab167733 100644
--- a/src/southbridge/intel/i82371eb/usb.c
+++ b/src/southbridge/intel/i82371eb/usb.c
@@ -43,15 +43,6 @@ static const struct device_operations usb_ops = {
 	.ops_pci		= 0, /* No subsystem IDs on 82371EB! */
 };
 
-/* Note: No USB on 82371FB/MX (PIIX/MPIIX) and 82437MX. */
-
-/* Intel 82371SB (PIIX3) */
-static const struct pci_driver usb_driver_sb __pci_driver = {
-	.ops	= &usb_ops,
-	.vendor	= PCI_VENDOR_ID_INTEL,
-	.device	= PCI_DEVICE_ID_INTEL_82371SB_USB,
-};
-
 /* Intel 82371AB/EB/MB (PIIX4/PIIX4E/PIIX4M) */
 /* The 440MX (82443MX) consists of 82443BX + 82371EB (uses same PCI IDs). */
 static const struct pci_driver usb_driver_ab_eb_mb __pci_driver = {