Various Doxygen comment fixes, typo fixes, etc.
[coreboot.git] / src / devices / pci_device.c
index 4a7a2cd9c58322da8cc9f0d23cc3e546b4d39cdc..ac799c5fec459593e0bc735dcda547ecad296290 100644 (file)
@@ -32,8 +32,6 @@
 #include <device/device.h>
 #include <device/pci.h>
 #include <device/pci_ids.h>
-#include <part/hard_reset.h>
-#include <part/fallback_boot.h>
 #include <delay.h>
 #if CONFIG_HYPERTRANSPORT_PLUGIN_SUPPORT == 1
 #include <device/hypertransport.h>
 #if CONFIG_CARDBUS_PLUGIN_SUPPORT == 1
 #include <device/cardbus.h>
 #endif
+#define CONFIG_PC80_SYSTEM 1
+#if CONFIG_PC80_SYSTEM == 1
+#include <pc80/i8259.h>
+#endif
 
 u8 pci_moving_config8(struct device *dev, unsigned int reg)
 {
@@ -104,16 +106,17 @@ u32 pci_moving_config32(struct device * dev, unsigned int reg)
  * matching capability. Always start at the head of the list.
  *
  * @param dev Pointer to the device structure.
- * @param cap_type PCI_CAP_LIST_ID of the PCI capability we're looking for.
+ * @param cap PCI_CAP_LIST_ID of the PCI capability we're looking for.
  * @param last Location of the PCI capability register to start from.
+ * @return The next matching capability.
  */
 unsigned pci_find_next_capability(struct device *dev, unsigned cap,
                                  unsigned last)
 {
-       unsigned pos;
+       unsigned pos = 0;
        unsigned status;
        unsigned reps = 48;
-       pos = 0;
+
        status = pci_read_config16(dev, PCI_STATUS);
        if (!(status & PCI_STATUS_CAP_LIST)) {
                return 0;
@@ -134,7 +137,7 @@ unsigned pci_find_next_capability(struct device *dev, unsigned cap,
                int this_cap;
                pos &= ~3;
                this_cap = pci_read_config8(dev, pos + PCI_CAP_LIST_ID);
-               printk_spew("Capability: type 0x%02x @ 0x%02x\n", this_cap,
+               printk(BIOS_SPEW, "Capability: type 0x%02x @ 0x%02x\n", this_cap,
                            pos);
                if (this_cap == 0xff) {
                        break;
@@ -155,7 +158,8 @@ unsigned pci_find_next_capability(struct device *dev, unsigned cap,
  * capability. Always start at the head of the list.
  *
  * @param dev Pointer to the device structure.
- * @param cap_type PCI_CAP_LIST_ID of the PCI capability we're looking for.
+ * @param cap PCI_CAP_LIST_ID of the PCI capability we're looking for.
+ * @return The next matching capability.
  */
 unsigned pci_find_capability(device_t dev, unsigned cap)
 {
@@ -167,6 +171,7 @@ unsigned pci_find_capability(device_t dev, unsigned cap)
  *
  * @param dev Pointer to the device structure.
  * @param index Address of the PCI configuration register.
+ * @return TODO
  */
 struct resource *pci_get_resource(struct device *dev, unsigned long index)
 {
@@ -224,8 +229,7 @@ struct resource *pci_get_resource(struct device *dev, unsigned long index)
         */
        if (moving == 0) {
                if (value != 0) {
-                       printk_debug
-                           ("%s register %02lx(%08lx), read-only ignoring it\n",
+                       printk(BIOS_DEBUG, "%s register %02lx(%08lx), read-only ignoring it\n",
                             dev_path(dev), index, value);
                }
                resource->flags = 0;
@@ -255,8 +259,8 @@ struct resource *pci_get_resource(struct device *dev, unsigned long index)
                        resource->flags |= IORESOURCE_PCI64;
                } else {
                        /* Invalid value. */
-                       printk_err("Broken BAR with value %lx\n", attr);
-                       printk_err(" on dev %s at index %02lx\n",
+                       printk(BIOS_ERR, "Broken BAR with value %lx\n", attr);
+                       printk(BIOS_ERR, " on dev %s at index %02lx\n",
                               dev_path(dev), index);
                        resource->flags = 0;
                }
@@ -281,11 +285,6 @@ static void pci_get_rom_resource(struct device *dev, unsigned long index)
        unsigned long value;
        resource_t moving;
 
-       if ((dev->on_mainboard) && (dev->rom_address == 0)) {
-               /* Skip it if rom_address is not set in the MB Config.lb. */
-               return;
-       }
-
        /* Initialize the resources to nothing. */
        resource = new_resource(dev, index);
 
@@ -316,22 +315,11 @@ static void pci_get_rom_resource(struct device *dev, unsigned long index)
                resource->flags |= IORESOURCE_MEM | IORESOURCE_READONLY;
        } else {
                if (value != 0) {
-                       printk_debug
-                           ("%s register %02lx(%08lx), read-only ignoring it\n",
+                       printk(BIOS_DEBUG, "%s register %02lx(%08lx), read-only ignoring it\n",
                             dev_path(dev), index, value);
                }
                resource->flags = 0;
        }
-
-       /* For on board device with embedded ROM image, the ROM image is at
-        * fixed address specified in the Config.lb, the dev->rom_address is
-        * inited by driver_pci_onboard_ops::enable_dev() */
-       if ((dev->on_mainboard) && (dev->rom_address != 0)) {
-               resource->base = dev->rom_address;
-               resource->flags |= IORESOURCE_MEM | IORESOURCE_READONLY |
-                   IORESOURCE_ASSIGNED | IORESOURCE_FIXED;
-       }
-
        compact_resources(dev);
 }
 
@@ -388,11 +376,11 @@ static void pci_bridge_read_bases(struct device *dev)
        /* See if the bridge I/O resources are implemented. */
        moving_base = ((u32) pci_moving_config8(dev, PCI_IO_BASE)) << 8;
        moving_base |=
-           ((u32) pci_moving_config16(dev, PCI_IO_BASE_UPPER16)) << 16;
+         ((u32) pci_moving_config16(dev, PCI_IO_BASE_UPPER16)) << 16;
 
        moving_limit = ((u32) pci_moving_config8(dev, PCI_IO_LIMIT)) << 8;
        moving_limit |=
-           ((u32) pci_moving_config16(dev, PCI_IO_LIMIT_UPPER16)) << 16;
+         ((u32) pci_moving_config16(dev, PCI_IO_LIMIT_UPPER16)) << 16;
 
        moving = moving_base & moving_limit;
 
@@ -401,17 +389,14 @@ static void pci_bridge_read_bases(struct device *dev)
 
        /* See if the bridge prefmem resources are implemented. */
        moving_base =
-           ((resource_t) pci_moving_config16(dev, PCI_PREF_MEMORY_BASE)) << 16;
+         ((resource_t) pci_moving_config16(dev, PCI_PREF_MEMORY_BASE)) << 16;
        moving_base |=
-           ((resource_t) pci_moving_config32(dev, PCI_PREF_BASE_UPPER32)) <<
-           32;
+         ((resource_t) pci_moving_config32(dev, PCI_PREF_BASE_UPPER32)) << 32;
 
        moving_limit =
-           ((resource_t) pci_moving_config16(dev, PCI_PREF_MEMORY_LIMIT)) <<
-           16;
+         ((resource_t) pci_moving_config16(dev, PCI_PREF_MEMORY_LIMIT)) << 16;
        moving_limit |=
-           ((resource_t) pci_moving_config32(dev, PCI_PREF_LIMIT_UPPER32)) <<
-           32;
+         ((resource_t) pci_moving_config32(dev, PCI_PREF_LIMIT_UPPER32)) << 32;
 
        moving = moving_base & moving_limit;
        /* Initialize the prefetchable memory constraints on the current bus. */
@@ -467,12 +452,17 @@ static void pci_set_resource(struct device *dev, struct resource *resource)
 
        /* Make certain the resource has actually been assigned a value. */
        if (!(resource->flags & IORESOURCE_ASSIGNED)) {
-               printk_err("ERROR: %s %02lx %s size: 0x%010llx not assigned\n",
+               printk(BIOS_ERR, "ERROR: %s %02lx %s size: 0x%010llx not assigned\n",
                           dev_path(dev), resource->index,
                           resource_type(resource), resource->size);
                return;
        }
 
+       /* If this resource is fixed don't worry about it. */
+       if (resource->flags & IORESOURCE_FIXED) {
+               return;
+       }
+
        /* If I have already stored this resource don't worry about it. */
        if (resource->flags & IORESOURCE_STORED) {
                return;
@@ -551,7 +541,7 @@ static void pci_set_resource(struct device *dev, struct resource *resource)
        } else {
                /* Don't let me think I stored the resource. */
                resource->flags &= ~IORESOURCE_STORED;
-               printk_err("ERROR: invalid resource->index %lx\n",
+               printk(BIOS_ERR, "ERROR: invalid resource->index %lx\n",
                           resource->index);
        }
        report_resource_stored(dev, resource, "");
@@ -560,18 +550,14 @@ static void pci_set_resource(struct device *dev, struct resource *resource)
 
 void pci_dev_set_resources(struct device *dev)
 {
-       struct resource *resource, *last;
-       unsigned link;
+       struct resource *res;
+       struct bus *bus;
        u8 line;
 
-       last = &dev->resource[dev->resources];
-
-       for (resource = &dev->resource[0]; resource < last; resource++) {
-               pci_set_resource(dev, resource);
+       for (res = dev->resource_list; res; res = res->next) {
+               pci_set_resource(dev, res);
        }
-       for (link = 0; link < dev->links; link++) {
-               struct bus *bus;
-               bus = &dev->link[link];
+       for (bus = dev->link_list; bus; bus = bus->next) {
                if (bus->children) {
                        assign_resources(bus);
                }
@@ -602,7 +588,7 @@ void pci_dev_enable_resources(struct device *dev)
        /* Set the subsystem vendor and device id for mainboard devices. */
        ops = ops_pci(dev);
        if (dev->on_mainboard && ops && ops->set_subsystem) {
-               printk_debug("%s subsystem <- %02x/%02x\n",
+               printk(BIOS_DEBUG, "%s subsystem <- %02x/%02x\n",
                             dev_path(dev),
                             CONFIG_MAINBOARD_PCI_SUBSYSTEM_VENDOR_ID,
                             CONFIG_MAINBOARD_PCI_SUBSYSTEM_DEVICE_ID);
@@ -615,7 +601,7 @@ void pci_dev_enable_resources(struct device *dev)
        /* v3 has
         * command |= (PCI_COMMAND_PARITY + PCI_COMMAND_SERR);  // Error check.
         */
-       printk_debug("%s cmd <- %02x\n", dev_path(dev), command);
+       printk(BIOS_DEBUG, "%s cmd <- %02x\n", dev_path(dev), command);
        pci_write_config16(dev, PCI_COMMAND, command);
 }
 
@@ -626,16 +612,15 @@ void pci_bus_enable_resources(struct device *dev)
        /* Enable I/O in command register if there is VGA card
         * connected with (even it does not claim I/O resource).
         */
-       if (dev->link[0].bridge_ctrl & PCI_BRIDGE_CTL_VGA)
+       if (dev->link_list->bridge_ctrl & PCI_BRIDGE_CTL_VGA)
                dev->command |= PCI_COMMAND_IO;
        ctrl = pci_read_config16(dev, PCI_BRIDGE_CONTROL);
-       ctrl |= dev->link[0].bridge_ctrl;
+       ctrl |= dev->link_list->bridge_ctrl;
        ctrl |= (PCI_BRIDGE_CTL_PARITY + PCI_BRIDGE_CTL_SERR);  /* Error check. */
-       printk_debug("%s bridge ctrl <- %04x\n", dev_path(dev), ctrl);
+       printk(BIOS_DEBUG, "%s bridge ctrl <- %04x\n", dev_path(dev), ctrl);
        pci_write_config16(dev, PCI_BRIDGE_CONTROL, ctrl);
 
        pci_dev_enable_resources(dev);
-       enable_childrens_resources(dev);
 }
 
 void pci_bus_reset(struct bus *bus)
@@ -660,17 +645,15 @@ void pci_dev_set_subsystem(struct device *dev, unsigned vendor, unsigned device)
 void pci_dev_init(struct device *dev)
 {
 #if CONFIG_PCI_ROM_RUN == 1 || CONFIG_VGA_ROM_RUN == 1
-       void run_bios(struct device *dev, unsigned long addr);
        struct rom_header *rom, *ram;
 
-#if CONFIG_PCI_ROM_RUN != 1
-       /* We want to execute VGA option ROMs when CONFIG_VGA_ROM_RUN
-        * is set but CONFIG_PCI_ROM_RUN is not. In this case we skip
-        * all other option ROM types.
-        */
-       if ((dev->class >> 8) != PCI_CLASS_DISPLAY_VGA)
+       if (CONFIG_PCI_ROM_RUN != 1 && /* Only execute VGA ROMs. */
+           ((dev->class >> 8) != PCI_CLASS_DISPLAY_VGA))
+               return;
+
+       if (CONFIG_VGA_ROM_RUN != 1 && /* Only execute non-VGA ROMs. */
+           ((dev->class >> 8) == PCI_CLASS_DISPLAY_VGA))
                return;
-#endif
 
        rom = pci_rom_probe(dev);
        if (rom == NULL)
@@ -721,7 +704,7 @@ struct device_operations default_pci_ops_bus = {
 };
 
 /**
- * @brief Detect the type of downstream bridge
+ * Detect the type of downstream bridge.
  *
  * This function is a heuristic to detect which type of bus is downstream
  * of a PCI-to-PCI bridge. This functions by looking for various capability
@@ -741,7 +724,7 @@ static struct device_operations *get_pci_bridge_ops(device_t dev)
 #if CONFIG_PCIX_PLUGIN_SUPPORT == 1
        pos = pci_find_capability(dev, PCI_CAP_ID_PCIX);
        if (pos) {
-               printk_debug("%s subbordinate bus PCI-X\n", dev_path(dev));
+               printk(BIOS_DEBUG, "%s subordinate bus PCI-X\n", dev_path(dev));
                return &default_pcix_ops_bus;
        }
 #endif
@@ -755,7 +738,7 @@ static struct device_operations *get_pci_bridge_ops(device_t dev)
                flags = pci_read_config16(dev, pos + PCI_CAP_FLAGS);
                if ((flags >> 13) == 1) {
                        /* Host or Secondary Interface */
-                       printk_debug("%s subbordinate bus Hypertransport\n",
+                       printk(BIOS_DEBUG, "%s subordinate bus Hypertransport\n",
                                     dev_path(dev));
                        return &default_ht_ops_bus;
                }
@@ -770,11 +753,11 @@ static struct device_operations *get_pci_bridge_ops(device_t dev)
                case PCI_EXP_TYPE_ROOT_PORT:
                case PCI_EXP_TYPE_UPSTREAM:
                case PCI_EXP_TYPE_DOWNSTREAM:
-                       printk_debug("%s subbordinate bus PCI Express\n",
+                       printk(BIOS_DEBUG, "%s subordinate bus PCI Express\n",
                                     dev_path(dev));
                        return &default_pciexp_ops_bus;
                case PCI_EXP_TYPE_PCI_BRIDGE:
-                       printk_debug("%s subbordinate PCI\n", dev_path(dev));
+                       printk(BIOS_DEBUG, "%s subordinate PCI\n", dev_path(dev));
                        return &default_pci_ops_bus;
                default:
                        break;
@@ -785,8 +768,10 @@ static struct device_operations *get_pci_bridge_ops(device_t dev)
 }
 
 /**
- * Set up PCI device operation.  Check if it already has a driver.  If not, use
- * find_device_operations, or set to a default based on type.
+ * Set up PCI device operation.
+ *
+ * Check if it already has a driver. If not, use find_device_operations(),
+ * or set to a default based on type.
  *
  * @param dev Pointer to the device whose pci_ops you want to set.
  * @see pci_drivers
@@ -804,8 +789,8 @@ static void set_pci_ops(struct device *dev)
        for (driver = &pci_drivers[0]; driver != &epci_drivers[0]; driver++) {
                if ((driver->vendor == dev->vendor) &&
                    (driver->device == dev->device)) {
-                       dev->ops = driver->ops;
-                       printk_spew("%s [%04x/%04x] %sops\n",
+                       dev->ops = (struct device_operations *)driver->ops;
+                       printk(BIOS_SPEW, "%s [%04x/%04x] %sops\n",
                                    dev_path(dev),
                                    driver->vendor, driver->device,
                                    (driver->ops->scan_bus ? "bus " : ""));
@@ -833,7 +818,7 @@ static void set_pci_ops(struct device *dev)
        default:
              bad:
                if (dev->enabled) {
-                       printk_err("%s [%04x/%04x/%06x] has unknown header "
+                       printk(BIOS_ERR, "%s [%04x/%04x/%06x] has unknown header "
                                   "type %02x, ignoring.\n",
                                   dev_path(dev),
                                   dev->vendor, dev->device,
@@ -844,7 +829,7 @@ static void set_pci_ops(struct device *dev)
 }
 
 /**
- * @brief See if we have already allocated a device structure for a given devfn.
+ * See if we have already allocated a device structure for a given devfn.
  *
  * Given a linked list of PCI device structures and a devfn number, find the
  * device structure correspond to the devfn, if present. This function also
@@ -852,7 +837,6 @@ static void set_pci_ops(struct device *dev)
  *
  * @param list The device structure list.
  * @param devfn A device/function number.
- *
  * @return Pointer to the device structure found or NULL if we have not
  *        allocated a device for this devfn yet.
  */
@@ -862,7 +846,7 @@ static struct device *pci_scan_get_dev(struct device **list, unsigned int devfn)
        dev = 0;
        for (; *list; list = &(*list)->sibling) {
                if ((*list)->path.type != DEVICE_PATH_PCI) {
-                       printk_err("child %s not a pci device\n",
+                       printk(BIOS_ERR, "child %s not a pci device\n",
                                   dev_path(*list));
                        continue;
                }
@@ -898,18 +882,17 @@ static struct device *pci_scan_get_dev(struct device **list, unsigned int devfn)
 }
 
 /**
- * @brief Scan a PCI bus.
+ * Scan a PCI bus.
  *
  * Determine the existence of a given PCI device. Allocate a new struct device
  * if dev==NULL was passed in and the device exists in hardware.
  *
- * @param bus pointer to the bus structure
- * @param devfn to look at
- *
- * @return The device structure for hte device (if found)
- *         or the NULL if no device is found.
+ * @param dev Pointer to the dev structure.
+ * @param bus Pointer to the bus structure.
+ * @param devfn A device/function number to look at.
+ * @return The device structure for the device (if found), NULL otherwise.
  */
-device_t pci_probe_dev(device_t dev, struct bus * bus, unsigned devfn)
+device_t pci_probe_dev(device_t dev, struct bus *bus, unsigned devfn)
 {
        u32 id, class;
        u8 hdr_type;
@@ -922,11 +905,15 @@ device_t pci_probe_dev(device_t dev, struct bus * bus, unsigned devfn)
                dummy.path.pci.devfn = devfn;
                id = pci_read_config32(&dummy, PCI_VENDOR_ID);
                /* Have we found something?
-                * Some broken boards return 0 if a slot is empty.
+                * Some broken boards return 0 if a slot is empty, but
+                * the expected answer is 0xffffffff
                 */
-               if ((id == 0xffffffff) || (id == 0x00000000) ||
-                   (id == 0x0000ffff) || (id == 0xffff0000)) {
-                       printk_spew("%s, bad id 0x%x\n", dev_path(&dummy), id);
+               if (id == 0xffffffff) {
+                       return NULL;
+               }
+               if ((id == 0x00000000) || (id == 0x0000ffff) ||
+                   (id == 0xffff0000)) {
+                       printk(BIOS_SPEW, "%s, bad id 0x%x\n", dev_path(&dummy), id);
                        return NULL;
                }
                dev = alloc_dev(bus, &dummy.path);
@@ -957,7 +944,7 @@ device_t pci_probe_dev(device_t dev, struct bus * bus, unsigned devfn)
                if ((id == 0xffffffff) || (id == 0x00000000) ||
                    (id == 0x0000ffff) || (id == 0xffff0000)) {
                        if (dev->enabled) {
-                               printk_info("Disabling static device: %s\n",
+                               printk(BIOS_INFO, "PCI: Static device %s not found, disabling it.\n",
                                            dev_path(dev));
                                dev->enabled = 0;
                        }
@@ -992,7 +979,7 @@ device_t pci_probe_dev(device_t dev, struct bus * bus, unsigned devfn)
        }
 
        /* Display the device. */
-       printk_debug("%s [%04x/%04x] %s%s\n",
+       printk(BIOS_DEBUG, "%s [%04x/%04x] %s%s\n",
                     dev_path(dev),
                     dev->vendor, dev->device,
                     dev->enabled ? "enabled" : "disabled",
@@ -1002,7 +989,7 @@ device_t pci_probe_dev(device_t dev, struct bus * bus, unsigned devfn)
 }
 
 /**
- * @brief Scan a PCI bus.
+ * Scan a PCI bus.
  *
  * Determine the existence of devices and bridges on a PCI bus. If there are
  * bridges on the bus, recursively scan the buses behind the bridges.
@@ -1010,12 +997,11 @@ device_t pci_probe_dev(device_t dev, struct bus * bus, unsigned devfn)
  * This function is the default scan_bus() method for the root device
  * 'dev_root'.
  *
- * @param bus pointer to the bus structure
- * @param min_devfn minimum devfn to look at in the scan usually 0x00
- * @param max_devfn maximum devfn to look at in the scan usually 0xff
- * @param max current bus number
- *
- * @return The maximum bus number found, after scanning all subordinate busses
+ * @param bus Pointer to the bus structure.
+ * @param min_devfn Minimum devfn to look at in the scan, usually 0x00.
+ * @param max_devfn Maximum devfn to look at in the scan, usually 0xff.
+ * @param max Current bus number.
+ * @return The maximum bus number found, after scanning all subordinate busses.
  */
 unsigned int pci_scan_bus(struct bus *bus,
                          unsigned min_devfn, unsigned max_devfn,
@@ -1026,12 +1012,20 @@ unsigned int pci_scan_bus(struct bus *bus,
        struct device *child;
 
 #if CONFIG_PCI_BUS_SEGN_BITS
-       printk_debug("PCI: pci_scan_bus for bus %04x:%02x\n",
+       printk(BIOS_DEBUG, "PCI: pci_scan_bus for bus %04x:%02x\n",
                     bus->secondary >> 8, bus->secondary & 0xff);
 #else
-       printk_debug("PCI: pci_scan_bus for bus %02x\n", bus->secondary);
+       printk(BIOS_DEBUG, "PCI: pci_scan_bus for bus %02x\n", bus->secondary);
 #endif
 
+       // Maximum sane devfn is 0xFF
+       if (max_devfn > 0xff) {
+               printk(BIOS_ERR, "PCI: pci_scan_bus limits devfn %x - devfn %x\n",
+                          min_devfn, max_devfn );
+               printk(BIOS_ERR, "PCI: pci_scan_bus upper limit too big. Using 0xff.\n");
+               max_devfn=0xff;
+       }
+
        old_devices = bus->children;
        bus->children = NULL;
 
@@ -1065,11 +1059,11 @@ unsigned int pci_scan_bus(struct bus *bus,
         */
        if (old_devices) {
                device_t left;
-               printk_warning("PCI: Left over static devices:\n");
+               printk(BIOS_WARNING, "PCI: Left over static devices:\n");
                for (left = old_devices; left; left = left->sibling) {
-                       printk_warning("%s\n", dev_path(left));
+                       printk(BIOS_WARNING, "%s\n", dev_path(left));
                }
-               printk_warning("PCI: Check your mainboard Config.lb.\n");
+               printk(BIOS_WARNING, "PCI: Check your mainboard Config.lb.\n");
        }
 
        /* For all children that implement scan_bus() (i.e. bridges)
@@ -1083,13 +1077,13 @@ unsigned int pci_scan_bus(struct bus *bus,
         * side of any bridges that may be on this bus plus any devices.
         * Return how far we've got finding sub-buses.
         */
-       printk_debug("PCI: pci_scan_bus returning with max=%03x\n", max);
+       printk(BIOS_DEBUG, "PCI: pci_scan_bus returning with max=%03x\n", max);
        post_code(0x55);
        return max;
 }
 
 /**
- * @brief Scan a PCI bridge and the buses behind the bridge.
+ * Scan a PCI bridge and the buses behind the bridge.
  *
  * Determine the existence of buses behind the bridge. Set up the bridge
  * according to the result of the scan.
@@ -1098,6 +1092,7 @@ unsigned int pci_scan_bus(struct bus *bus,
  *
  * @param dev Pointer to the bridge device.
  * @param max The highest bus number assigned up to now.
+ * @param do_scan_bus TODO
  * @return The maximum bus number found, after scanning all subordinate buses.
  */
 unsigned int do_pci_scan_bridge(struct device *dev, unsigned int max,
@@ -1110,11 +1105,19 @@ unsigned int do_pci_scan_bridge(struct device *dev, unsigned int max,
        u32 buses;
        u16 cr;
 
-       printk_spew("%s for %s\n", __func__, dev_path(dev));
+       printk(BIOS_SPEW, "%s for %s\n", __func__, dev_path(dev));
+
+       if (dev->link_list == NULL) {
+               struct bus *link;
+               link = malloc(sizeof(*link));
+               if (link == NULL)
+                       die("Couldn't allocate a link!\n");
+               memset(link, 0, sizeof(*link));
+               link->dev = dev;
+               dev->link_list = link;
+       }
 
-       bus = &dev->link[0];
-       bus->dev = dev;
-       dev->links = 1;
+       bus = dev->link_list;
 
        /* Set up the primary, secondary and subordinate bus numbers. We have
         * no idea how many buses are behind this bridge yet, so we set the
@@ -1156,12 +1159,12 @@ unsigned int do_pci_scan_bridge(struct device *dev, unsigned int max,
        pci_write_config32(dev, PCI_PRIMARY_BUS, buses);
        pci_write_config16(dev, PCI_COMMAND, cr);
 
-       printk_spew("%s returns max %d\n", __func__, max);
+       printk(BIOS_SPEW, "%s returns max %d\n", __func__, max);
        return max;
 }
 
 /**
- * @brief Scan a PCI bridge and the buses behind the bridge.
+ * Scan a PCI bridge and the buses behind the bridge.
  *
  * Determine the existence of buses behind the bridge. Set up the bridge
  * according to the result of the scan.
@@ -1178,101 +1181,73 @@ unsigned int pci_scan_bridge(struct device *dev, unsigned int max)
 }
 
 /**
- * @brief Scan a PCI domain.
+ * Scan a PCI domain.
  *
  * This function is the default scan_bus() method for PCI domains.
  *
- * @param dev pointer to the domain
- * @param max the highest bus number assgined up to now
- *
- * @return The maximum bus number found, after scanning all subordinate busses
+ * @param dev Pointer to the domain.
+ * @param max The highest bus number assigned up to now.
+ * @return The maximum bus number found, after scanning all subordinate busses.
  */
 unsigned int pci_domain_scan_bus(device_t dev, unsigned int max)
 {
-       max = pci_scan_bus(&dev->link[0], PCI_DEVFN(0, 0), 0xff, max);
+       max = pci_scan_bus(dev->link_list, PCI_DEVFN(0, 0), 0xff, max);
        return max;
 }
 
+#if CONFIG_PC80_SYSTEM == 1
 /**
- * Tell the EISA int controller this int must be level triggered.
+ * Assign IRQ numbers.
  *
- * THIS IS A KLUDGE -- sorry, this needs to get cleaned up.
- */
-void pci_level_irq(unsigned char intNum)
-{
-       unsigned short intBits = inb(0x4d0) | (((unsigned)inb(0x4d1)) << 8);
-
-       printk_spew("%s: current ints are 0x%x\n", __func__, intBits);
-       intBits |= (1 << intNum);
-
-       printk_spew("%s: try to set ints 0x%x\n", __func__, intBits);
-
-       /* Write new values. */
-       outb((unsigned char)intBits, 0x4d0);
-       outb((unsigned char)(intBits >> 8), 0x4d1);
-
-       /* This seems like an error but is not. */
-       if (inb(0x4d0) != (intBits & 0xff)) {
-               printk_err(
-                          "%s: lower order bits are wrong: want 0x%x, got 0x%x\n",
-                          __func__, intBits & 0xff, inb(0x4d0));
-       }
-       if (inb(0x4d1) != ((intBits >> 8) & 0xff)) {
-               printk_err(
-                          "%s: lower order bits are wrong: want 0x%x, got 0x%x\n",
-                          __func__, (intBits >> 8) & 0xff, inb(0x4d1));
-       }
-}
-
-/**
- * This function assigns IRQs for all functions contained within the
- * indicated device address. If the device does not exist or does not
- * require interrupts then this function has no effect.
+ * This function assigns IRQs for all functions contained within the indicated
+ * device address.  If the device does not exist or does not require interrupts
+ * then this function has no effect.
  *
  * This function should be called for each PCI slot in your system.
  *
- * pIntAtoD is an array of IRQ #s that are assigned to PINTA through PINTD of
- * this slot.
- *
- * The particular irq #s that are passed in depend on the routing inside
- * your southbridge and on your motherboard.
- *
- * -kevinh@ispiri.com
- *
-*/
+ * @param bus Pointer to the bus structure.
+ * @param slot TODO
+ * @param pIntAtoD An array of IRQ #s that are assigned to PINTA through PINTD
+ *        of this slot. The particular IRQ #s that are passed in depend on the
+ *        routing inside your southbridge and on your board.
+ */
 void pci_assign_irqs(unsigned bus, unsigned slot,
-                    const unsigned char pIntAtoD[4])
+       const unsigned char pIntAtoD[4])
 {
-       unsigned functNum;
-       struct device *pdev;
-       unsigned char line;
-       unsigned char irq;
-       unsigned char readback;
+       unsigned int funct;
+       device_t pdev;
+       u8 line;
+       u8 irq;
 
-       /* Each slot may contain up to eight functions. */
-       for (functNum = 0; functNum < 8; functNum++) {
-               pdev = dev_find_slot(bus, (slot << 3) + functNum);
+       /* Each slot may contain up to eight functions */
+       for (funct = 0; funct < 8; funct++) {
+               pdev = dev_find_slot(bus, (slot << 3) + funct);
 
-               if (pdev) {
-                       line = pci_read_config8(pdev, PCI_INTERRUPT_PIN);
+               if (!pdev)
+                       continue;
 
-                       /* PCI spec says all other values are reserved. */
-                       if ((line >= 1) && (line <= 4)) {
-                               irq = pIntAtoD[line - 1];
+               line = pci_read_config8(pdev, PCI_INTERRUPT_PIN);
 
-                               printk_debug("Assigning IRQ %d to %d:%x.%d\n",
-                                            irq, bus, slot, functNum);
+               // PCI spec says all values except 1..4 are reserved.
+               if ((line < 1) || (line > 4))
+                       continue;
 
-                               pci_write_config8(pdev, PCI_INTERRUPT_LINE,
-                                                 pIntAtoD[line - 1]);
+               irq = pIntAtoD[line - 1];
 
-                               readback =
-                                   pci_read_config8(pdev, PCI_INTERRUPT_LINE);
-                               printk_debug("  Readback = %d\n", readback);
+               printk(BIOS_DEBUG, "Assigning IRQ %d to %d:%x.%d\n",
+                       irq, bus, slot, funct);
 
-                               // Change to level triggered.
-                               pci_level_irq(pIntAtoD[line - 1]);
-                       }
-               }
+               pci_write_config8(pdev, PCI_INTERRUPT_LINE,
+                       pIntAtoD[line - 1]);
+
+#ifdef PARANOID_IRQ_ASSIGNMENTS
+               irq = pci_read_config8(pdev, PCI_INTERRUPT_LINE);
+               printk(BIOS_DEBUG, "  Readback = %d\n", irq);
+#endif
+
+               // Change to level triggered
+               i8259_configure_irq_trigger(pIntAtoD[line - 1], IRQ_LEVEL_TRIGGERED);
        }
 }
+#endif
+