vserver 2.0 rc7
[linux-2.6.git] / arch / ia64 / pci / pci.c
index 2a0dcf6..88641e5 100644 (file)
@@ -3,9 +3,9 @@
  *
  * Derived from bios32.c of i386 tree.
  *
- * Copyright (C) 2002 Hewlett-Packard Co
+ * (c) Copyright 2002, 2005 Hewlett-Packard Development Company, L.P.
  *     David Mosberger-Tang <davidm@hpl.hp.com>
- *     Bjorn Helgaas <bjorn_helgaas@hp.com>
+ *     Bjorn Helgaas <bjorn.helgaas@hp.com>
  * Copyright (C) 2004 Silicon Graphics, Inc.
  *
  * Note: Above list of copyright holders is incomplete...
 #include <asm/segment.h>
 #include <asm/system.h>
 #include <asm/io.h>
-
 #include <asm/sal.h>
-
-
-#ifdef CONFIG_SMP
-# include <asm/smp.h>
-#endif
+#include <asm/smp.h>
 #include <asm/irq.h>
 #include <asm/hw_irq.h>
 
 
-#undef DEBUG
-#define DEBUG
-
-#ifdef DEBUG
-#define DBG(x...) printk(x)
-#else
-#define DBG(x...)
-#endif
-
 static int pci_routeirq;
 
 /*
@@ -55,23 +41,22 @@ static int pci_routeirq;
  * synchronization mechanism here.
  */
 
-#define PCI_SAL_ADDRESS(seg, bus, devfn, reg)  \
-       ((u64)(seg << 24) | (u64)(bus << 16) |  \
-        (u64)(devfn << 8) | (u64)(reg))
+#define PCI_SAL_ADDRESS(seg, bus, devfn, reg)          \
+       (((u64) seg << 24) | (bus << 16) | (devfn << 8) | (reg))
 
 /* SAL 3.2 adds support for extended config space. */
 
 #define PCI_SAL_EXT_ADDRESS(seg, bus, devfn, reg)      \
-       ((u64)(seg << 28) | (u64)(bus << 20) |          \
-        (u64)(devfn << 12) | (u64)(reg))
+       (((u64) seg << 28) | (bus << 20) | (devfn << 12) | (reg))
 
 static int
-pci_sal_read (int seg, int bus, int devfn, int reg, int len, u32 *value)
+pci_sal_read (unsigned int seg, unsigned int bus, unsigned int devfn,
+             int reg, int len, u32 *value)
 {
-       u64 addr, mode, data = 0;
-       int result = 0;
+       u64 addr, data = 0;
+       int mode, result;
 
-       if ((seg > 65535) || (bus > 255) || (devfn > 255) || (reg > 4095))
+       if (!value || (seg > 65535) || (bus > 255) || (devfn > 255) || (reg > 4095))
                return -EINVAL;
 
        if ((seg | reg) <= 255) {
@@ -82,16 +67,19 @@ pci_sal_read (int seg, int bus, int devfn, int reg, int len, u32 *value)
                mode = 1;
        }
        result = ia64_sal_pci_config_read(addr, mode, len, &data);
+       if (result != 0)
+               return -EINVAL;
 
        *value = (u32) data;
-
-       return result;
+       return 0;
 }
 
 static int
-pci_sal_write (int seg, int bus, int devfn, int reg, int len, u32 value)
+pci_sal_write (unsigned int seg, unsigned int bus, unsigned int devfn,
+              int reg, int len, u32 value)
 {
-       u64 addr, mode;
+       u64 addr;
+       int mode, result;
 
        if ((seg > 65535) || (bus > 255) || (devfn > 255) || (reg > 4095))
                return -EINVAL;
@@ -103,7 +91,10 @@ pci_sal_write (int seg, int bus, int devfn, int reg, int len, u32 value)
                addr = PCI_SAL_EXT_ADDRESS(seg, bus, devfn, reg);
                mode = 1;
        }
-       return ia64_sal_pci_config_write(addr, mode, len, value);
+       result = ia64_sal_pci_config_write(addr, mode, len, value);
+       if (result != 0)
+               return -EINVAL;
+       return 0;
 }
 
 static struct pci_raw_ops pci_sal_ops = {
@@ -160,21 +151,11 @@ pci_acpi_init (void)
                 * also do it here in case there are still broken drivers that
                 * don't use pci_enable_device().
                 */
-               printk(KERN_INFO "** Routing PCI interrupts for all devices because \"pci=routeirq\"\n");
-               printk(KERN_INFO "** was specified.  If this was required to make a driver work,\n");
-               printk(KERN_INFO "** please email the output of \"lspci\" to bjorn.helgaas@hp.com\n");
-               printk(KERN_INFO "** so I can fix the driver.\n");
+               printk(KERN_INFO "PCI: Routing interrupts for all devices because \"pci=routeirq\" specified\n");
                for_each_pci_dev(dev)
                        acpi_pci_irq_enable(dev);
-       } else {
-               printk(KERN_INFO "** PCI interrupts are no longer routed automatically.  If this\n");
-               printk(KERN_INFO "** causes a device to stop working, it is probably because the\n");
-               printk(KERN_INFO "** driver failed to call pci_enable_device().  As a temporary\n");
-               printk(KERN_INFO "** workaround, the \"pci=routeirq\" argument restores the old\n");
-               printk(KERN_INFO "** behavior.  If this argument makes the device work again,\n");
-               printk(KERN_INFO "** please email the output of \"lspci\" to bjorn.helgaas@hp.com\n");
-               printk(KERN_INFO "** so I can fix the driver.\n");
-       }
+       } else
+               printk(KERN_INFO "PCI: If a device doesn't work, try \"pci=routeirq\".  If it helps, post a report\n");
 
        return 0;
 }
@@ -197,30 +178,6 @@ alloc_pci_controller (int seg)
        return controller;
 }
 
-static int __devinit
-alloc_resource (char *name, struct resource *root, unsigned long start, unsigned long end,
-               unsigned long flags)
-{
-       struct resource *res;
-
-       res = kmalloc(sizeof(*res), GFP_KERNEL);
-       if (!res)
-               return -ENOMEM;
-
-       memset(res, 0, sizeof(*res));
-       res->name = name;
-       res->start = start;
-       res->end = end;
-       res->flags = flags;
-
-       if (insert_resource(root, res)) {
-               kfree(res);
-               return -EBUSY;
-       }
-
-       return 0;
-}
-
 static u64 __devinit
 add_io_space (struct acpi_resource_address64 *addr)
 {
@@ -273,10 +230,9 @@ struct pci_root_info {
        char *name;
 };
 
-static acpi_status __devinit
-add_window (struct acpi_resource *res, void *data)
+static __devinit acpi_status add_window(struct acpi_resource *res, void *data)
 {
-       struct pci_root_info *info = (struct pci_root_info *) data;
+       struct pci_root_info *info = data;
        struct pci_window *window;
        struct acpi_resource_address64 addr;
        acpi_status status;
@@ -284,45 +240,71 @@ add_window (struct acpi_resource *res, void *data)
        struct resource *root;
 
        status = acpi_resource_to_address64(res, &addr);
-       if (ACPI_SUCCESS(status)) {
-               if (!addr.address_length)
-                       return AE_OK;
-
-               if (addr.resource_type == ACPI_MEMORY_RANGE) {
-                       flags = IORESOURCE_MEM;
-                       root = &iomem_resource;
-                       offset = addr.address_translation_offset;
-               } else if (addr.resource_type == ACPI_IO_RANGE) {
-                       flags = IORESOURCE_IO;
-                       root = &ioport_resource;
-                       offset = add_io_space(&addr);
-                       if (offset == ~0)
-                               return AE_OK;
-               } else
+       if (!ACPI_SUCCESS(status))
+               return AE_OK;
+
+       if (!addr.address_length)
+               return AE_OK;
+
+       if (addr.resource_type == ACPI_MEMORY_RANGE) {
+               flags = IORESOURCE_MEM;
+               root = &iomem_resource;
+               offset = addr.address_translation_offset;
+       } else if (addr.resource_type == ACPI_IO_RANGE) {
+               flags = IORESOURCE_IO;
+               root = &ioport_resource;
+               offset = add_io_space(&addr);
+               if (offset == ~0)
                        return AE_OK;
-
-               window = &info->controller->window[info->controller->windows++];
-               window->resource.flags  = flags;
-               window->resource.start  = addr.min_address_range;
-               window->resource.end    = addr.max_address_range;
-               window->offset          = offset;
-
-               if (alloc_resource(info->name, root, addr.min_address_range + offset,
-                       addr.max_address_range + offset, flags))
-                       printk(KERN_ERR "alloc 0x%lx-0x%lx from %s for %s failed\n",
-                               addr.min_address_range + offset, addr.max_address_range + offset,
-                               root->name, info->name);
+       } else
+               return AE_OK;
+
+       window = &info->controller->window[info->controller->windows++];
+       window->resource.name = info->name;
+       window->resource.flags = flags;
+       window->resource.start = addr.min_address_range + offset;
+       window->resource.end = addr.max_address_range + offset;
+       window->resource.child = NULL;
+       window->offset = offset;
+
+       if (insert_resource(root, &window->resource)) {
+               printk(KERN_ERR "alloc 0x%lx-0x%lx from %s for %s failed\n",
+                       window->resource.start, window->resource.end,
+                       root->name, info->name);
        }
 
        return AE_OK;
 }
 
+static void __devinit
+pcibios_setup_root_windows(struct pci_bus *bus, struct pci_controller *ctrl)
+{
+       int i, j;
+
+       j = 0;
+       for (i = 0; i < ctrl->windows; i++) {
+               struct resource *res = &ctrl->window[i].resource;
+               /* HP's firmware has a hack to work around a Windows bug.
+                * Ignore these tiny memory ranges */
+               if ((res->flags & IORESOURCE_MEM) &&
+                   (res->end - res->start < 16))
+                       continue;
+               if (j >= PCI_BUS_NUM_RESOURCES) {
+                       printk("Ignoring range [%lx-%lx] (%lx)\n", res->start,
+                                       res->end, res->flags);
+                       continue;
+               }
+               bus->resource[j++] = res;
+       }
+}
+
 struct pci_bus * __devinit
-pci_acpi_scan_root (struct acpi_device *device, int domain, int bus)
+pci_acpi_scan_root(struct acpi_device *device, int domain, int bus)
 {
        struct pci_root_info info;
        struct pci_controller *controller;
        unsigned int windows = 0;
+       struct pci_bus *pbus;
        char *name;
 
        controller = alloc_pci_controller(domain);
@@ -331,8 +313,10 @@ pci_acpi_scan_root (struct acpi_device *device, int domain, int bus)
 
        controller->acpi_handle = device->handle;
 
-       acpi_walk_resources(device->handle, METHOD_NAME__CRS, count_window, &windows);
-       controller->window = kmalloc(sizeof(*controller->window) * windows, GFP_KERNEL);
+       acpi_walk_resources(device->handle, METHOD_NAME__CRS, count_window,
+                       &windows);
+       controller->window = kmalloc(sizeof(*controller->window) * windows,
+                       GFP_KERNEL);
        if (!controller->window)
                goto out2;
 
@@ -343,9 +327,14 @@ pci_acpi_scan_root (struct acpi_device *device, int domain, int bus)
        sprintf(name, "PCI Bus %04x:%02x", domain, bus);
        info.controller = controller;
        info.name = name;
-       acpi_walk_resources(device->handle, METHOD_NAME__CRS, add_window, &info);
+       acpi_walk_resources(device->handle, METHOD_NAME__CRS, add_window,
+                       &info);
+
+       pbus = pci_scan_bus(bus, &pci_root_ops, controller);
+       if (pbus)
+               pcibios_setup_root_windows(pbus, controller);
 
-       return pci_scan_bus(bus, &pci_root_ops, controller);
+       return pbus;
 
 out3:
        kfree(controller->window);
@@ -366,9 +355,9 @@ void pcibios_resource_to_bus(struct pci_dev *dev,
                struct pci_window *window = &controller->window[i];
                if (!(window->resource.flags & res->flags))
                        continue;
-               if (window->resource.start > res->start - window->offset)
+               if (window->resource.start > res->start)
                        continue;
-               if (window->resource.end < res->end - window->offset)
+               if (window->resource.end < res->end)
                        continue;
                offset = window->offset;
                break;
@@ -390,9 +379,9 @@ void pcibios_bus_to_resource(struct pci_dev *dev,
                struct pci_window *window = &controller->window[i];
                if (!(window->resource.flags & res->flags))
                        continue;
-               if (window->resource.start > region->start)
+               if (window->resource.start - window->offset > region->start)
                        continue;
-               if (window->resource.end < region->end)
+               if (window->resource.end - window->offset < region->end)
                        continue;
                offset = window->offset;
                break;