[ARM] 3821/1: iop3xx: switch iop32x/iop33x over to shared pci code

Switch the iop32x and iop33x code over to the common PCI implementation,
and remove the (nearly identical) iop32x and iop33x PCI implementations.

Signed-off-by: Lennert Buytenhek <buytenh@wantstofly.org>
Signed-off-by: Russell King <rmk+kernel@arm.linux.org.uk>

6 files changed, 8 insertions, 330 deletions

diff --git a/arch/arm/mach-iop33x/Makefile b/arch/arm/mach-iop33x/Makefile
index 7865923b0a60..6c74f13b1139 100644
--- a/arch/arm/mach-iop33x/Makefile
+++ b/arch/arm/mach-iop33x/Makefile
@@ -2,7 +2,7 @@
 # Makefile for the linux kernel.
 #
 
-obj-y                   := common.o setup.o irq.o pci.o time.o
+obj-y                   := setup.o irq.o time.o
 obj-m                   :=
 obj-n                   :=
 obj-                    :=
diff --git a/arch/arm/mach-iop33x/common.c b/arch/arm/mach-iop33x/common.c
deleted file mode 100644
index dd75f2b8e74f..000000000000
--- a/arch/arm/mach-iop33x/common.c
+++ /dev/null
@@ -1,22 +0,0 @@
-/*
- * arch/arm/mach-iop33x/common.c
- *
- * Common routines shared across all IOP3xx implementations
- *
- * Author: Deepak Saxena <dsaxena@mvista.com>
- *
- * Copyright 2003 (c) MontaVista, Software, Inc.
- *
- * This file is licensed under  the terms of the GNU General Public
- * License version 2. This program is licensed "as is" without any
- * warranty of any kind, whether express or implied.
- */
-
-#include <linux/delay.h>
-#include <asm/hardware.h>
-
-/*
- * Shared variables
- */
-unsigned long iop3xx_pcibios_min_io = 0;
-unsigned long iop3xx_pcibios_min_mem = 0;
diff --git a/arch/arm/mach-iop33x/iq80331-pci.c b/arch/arm/mach-iop33x/iq80331-pci.c
index 71adb12a9421..8b0bed5e2f91 100644
--- a/arch/arm/mach-iop33x/iq80331-pci.c
+++ b/arch/arm/mach-iop33x/iq80331-pci.c
@@ -57,51 +57,12 @@ iq80331_map_irq(struct pci_dev *dev, u8 idsel, u8 pin)
         return PCI_IRQ_TABLE_LOOKUP(1, 7);
 }
 
-static int iq80331_setup(int nr, struct pci_sys_data *sys)
-{
-        struct resource *res;
-
-        if(nr != 0)
-                return 0;
-
-        res = kzalloc(sizeof(struct resource) * 2, GFP_KERNEL);
-        if (!res)
-                panic("PCI: unable to alloc resources");
-
-        res[0].start = IOP331_PCI_LOWER_IO_VA;
-        res[0].end   = IOP331_PCI_UPPER_IO_VA;
-        res[0].name  = "IQ80331 PCI I/O Space";
-        res[0].flags = IORESOURCE_IO;
-
-        res[1].start = IOP331_PCI_LOWER_MEM_PA;
-        res[1].end   = IOP331_PCI_UPPER_MEM_PA;
-        res[1].name  = "IQ80331 PCI Memory Space";
-        res[1].flags = IORESOURCE_MEM;
-
-        request_resource(&ioport_resource, &res[0]);
-        request_resource(&iomem_resource, &res[1]);
-
-        sys->mem_offset = IOP331_PCI_MEM_OFFSET;
-        sys->io_offset  = IOP331_PCI_IO_OFFSET;
-
-        sys->resource[0] = &res[0];
-        sys->resource[1] = &res[1];
-        sys->resource[2] = NULL;
-
-        return 1;
-}
-
-static void iq80331_preinit(void)
-{
-        iop331_init();
-}
-
 static struct hw_pci iq80331_pci __initdata = {
         .swizzle        = pci_std_swizzle,
         .nr_controllers = 1,
-        .setup          = iq80331_setup,
-        .scan           = iop331_scan_bus,
-        .preinit        = iq80331_preinit,
+        .setup          = iop3xx_pci_setup,
+        .scan           = iop3xx_pci_scan_bus,
+        .preinit        = iop3xx_pci_preinit,
         .map_irq        = iq80331_map_irq
 };
 
diff --git a/arch/arm/mach-iop33x/iq80332-pci.c b/arch/arm/mach-iop33x/iq80332-pci.c
index f3010f85b1a2..0de8aa748dd8 100644
--- a/arch/arm/mach-iop33x/iq80332-pci.c
+++ b/arch/arm/mach-iop33x/iq80332-pci.c
@@ -63,51 +63,12 @@ iq80332_map_irq(struct pci_dev *dev, u8 idsel, u8 pin)
         return PCI_IRQ_TABLE_LOOKUP(1, 7);
 }
 
-static int iq80332_setup(int nr, struct pci_sys_data *sys)
-{
-        struct resource *res;
-
-        if(nr != 0)
-                return 0;
-
-        res = kzalloc(sizeof(struct resource) * 2, GFP_KERNEL);
-        if (!res)
-                panic("PCI: unable to alloc resources");
-
-        res[0].start = IOP331_PCI_LOWER_IO_VA;
-        res[0].end   = IOP331_PCI_UPPER_IO_VA;
-        res[0].name  = "IQ80332 PCI I/O Space";
-        res[0].flags = IORESOURCE_IO;
-
-        res[1].start = IOP331_PCI_LOWER_MEM_PA;
-        res[1].end   = IOP331_PCI_UPPER_MEM_PA;
-        res[1].name  = "IQ80332 PCI Memory Space";
-        res[1].flags = IORESOURCE_MEM;
-
-        request_resource(&ioport_resource, &res[0]);
-        request_resource(&iomem_resource, &res[1]);
-
-        sys->mem_offset = IOP331_PCI_MEM_OFFSET;
-        sys->io_offset  = IOP331_PCI_IO_OFFSET;
-
-        sys->resource[0] = &res[0];
-        sys->resource[1] = &res[1];
-        sys->resource[2] = NULL;
-
-        return 1;
-}
-
-static void iq80332_preinit(void)
-{
-        iop331_init();
-}
-
 static struct hw_pci iq80332_pci __initdata = {
         .swizzle        = pci_std_swizzle,
         .nr_controllers = 1,
-        .setup          = iq80332_setup,
-        .scan           = iop331_scan_bus,
-        .preinit        = iq80332_preinit,
+        .setup          = iop3xx_pci_setup,
+        .scan           = iop3xx_pci_scan_bus,
+        .preinit        = iop3xx_pci_preinit,
         .map_irq        = iq80332_map_irq
 };
 
diff --git a/arch/arm/mach-iop33x/irq.c b/arch/arm/mach-iop33x/irq.c
index 029fa78876a5..bcffc33a5be8 100644
--- a/arch/arm/mach-iop33x/irq.c
+++ b/arch/arm/mach-iop33x/irq.c
@@ -117,7 +117,7 @@ void __init iop331_init_irq(void)
         intstr_write0(0);               // treat all as IRQ
         intstr_write1(0);
         if(machine_is_iq80331())        // all interrupts are inputs to chip
-                *IOP331_PCIIRSR = 0x0f;
+                *IOP3XX_PCIIRSR = 0x0f;
 
         for(i = IOP331_IRQ_OFS; i < NR_IRQS; i++)
         {
diff --git a/arch/arm/mach-iop33x/pci.c b/arch/arm/mach-iop33x/pci.c
deleted file mode 100644
index bf6aba91b47d..000000000000
--- a/arch/arm/mach-iop33x/pci.c
+++ /dev/null
@@ -1,222 +0,0 @@
-/*
- * arch/arm/mach-iop33x/pci.c
- *
- * PCI support for the Intel IOP331 chipset
- *
- * Author: Dave Jiang (dave.jiang@intel.com)
- * Copyright (C) 2003, 2004 Intel Corp.
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License version 2 as
- * published by the Free Software Foundation.
- *
- */
-#include <linux/kernel.h>
-#include <linux/pci.h>
-#include <linux/slab.h>
-#include <linux/mm.h>
-#include <linux/init.h>
-#include <linux/ioport.h>
-
-#include <asm/io.h>
-#include <asm/irq.h>
-#include <asm/system.h>
-#include <asm/hardware.h>
-#include <asm/mach/pci.h>
-
-#include <asm/arch/iop331.h>
-
-#undef DEBUG
-#undef DEBUG1
-
-#ifdef DEBUG
-#define  DBG(x...) printk(x)
-#else
-#define  DBG(x...) do { } while (0)
-#endif
-
-#ifdef DEBUG1
-#define  DBG1(x...) printk(x)
-#else
-#define  DBG1(x...) do { } while (0)
-#endif
-
-/*
- * This routine builds either a type0 or type1 configuration command.  If the
- * bus is on the 80331 then a type0 made, else a type1 is created.
- */
-static u32 iop331_cfg_address(struct pci_bus *bus, int devfn, int where)
-{
-        struct pci_sys_data *sys = bus->sysdata;
-        u32 addr;
-
-        if (sys->busnr == bus->number)
-                addr = 1 << (PCI_SLOT(devfn) + 16) | (PCI_SLOT(devfn) << 11);
-        else
-                addr = bus->number << 16 | PCI_SLOT(devfn) << 11 | 1;
-
-        addr |= PCI_FUNC(devfn) << 8 | (where & ~3);
-
-        return addr;
-}
-
-/*
- * This routine checks the status of the last configuration cycle.  If an error
- * was detected it returns a 1, else it returns a 0.  The errors being checked
- * are parity, master abort, target abort (master and target).  These types of
- * errors occure during a config cycle where there is no device, like during
- * the discovery stage.
- */
-static int iop331_pci_status(void)
-{
-        unsigned int status;
-        int ret = 0;
-
-        /*
-         * Check the status registers.
-         */
-        status = *IOP331_ATUSR;
-        if (status & 0xf900)
-        {
-                DBG("\t\t\tPCI: P0 - status = 0x%08x\n", status);
-                *IOP331_ATUSR = status & 0xf900;
-                ret = 1;
-        }
-        status = *IOP331_ATUISR;
-        if (status & 0x679f)
-        {
-                DBG("\t\t\tPCI: P1 - status = 0x%08x\n", status);
-                *IOP331_ATUISR = status & 0x679f;
-                ret = 1;
-        }
-        return ret;
-}
-
-/*
- * Simply write the address register and read the configuration
- * data.  Note that the 4 nop's ensure that we are able to handle
- * a delayed abort (in theory.)
- */
-static inline u32 iop331_read(unsigned long addr)
-{
-        u32 val;
-
-        __asm__ __volatile__(
-                "str    %1, [%2]\n\t"
-                "ldr    %0, [%3]\n\t"
-                "nop\n\t"
-                "nop\n\t"
-                "nop\n\t"
-                "nop\n\t"
-                : "=r" (val)
-                : "r" (addr), "r" (IOP331_OCCAR), "r" (IOP331_OCCDR));
-
-        return val;
-}
-
-/*
- * The read routines must check the error status of the last configuration
- * cycle.  If there was an error, the routine returns all hex f's.
- */
-static int
-iop331_read_config(struct pci_bus *bus, unsigned int devfn, int where,
-                int size, u32 *value)
-{
-        unsigned long addr = iop331_cfg_address(bus, devfn, where);
-        u32 val = iop331_read(addr) >> ((where & 3) * 8);
-
-        if( iop331_pci_status() )
-                val = 0xffffffff;
-
-        *value = val;
-
-        return PCIBIOS_SUCCESSFUL;
-}
-
-static int
-iop331_write_config(struct pci_bus *bus, unsigned int devfn, int where,
-                int size, u32 value)
-{
-        unsigned long addr = iop331_cfg_address(bus, devfn, where);
-        u32 val;
-
-        if (size != 4) {
-                val = iop331_read(addr);
-                if (!iop331_pci_status() == 0)
-                        return PCIBIOS_SUCCESSFUL;
-
-                where = (where & 3) * 8;
-
-                if (size == 1)
-                        val &= ~(0xff << where);
-                else
-                        val &= ~(0xffff << where);
-
-                *IOP331_OCCDR = val | value << where;
-        } else {
-                asm volatile(
-                        "str    %1, [%2]\n\t"
-                        "str    %0, [%3]\n\t"
-                        "nop\n\t"
-                        "nop\n\t"
-                        "nop\n\t"
-                        "nop\n\t"
-                        :
-                        : "r" (value), "r" (addr),
-                          "r" (IOP331_OCCAR), "r" (IOP331_OCCDR));
-        }
-
-        return PCIBIOS_SUCCESSFUL;
-}
-
-static struct pci_ops iop331_ops = {
-        .read   = iop331_read_config,
-        .write  = iop331_write_config,
-};
-
-/*
- * When a PCI device does not exist during config cycles, the XScale gets a
- * bus error instead of returning 0xffffffff. This handler simply returns.
- */
-int
-iop331_pci_abort(unsigned long addr, unsigned int fsr, struct pt_regs *regs)
-{
-        DBG("PCI abort: address = 0x%08lx fsr = 0x%03x PC = 0x%08lx LR = 0x%08lx\n",
-                addr, fsr, regs->ARM_pc, regs->ARM_lr);
-
-        /*
-         * If it was an imprecise abort, then we need to correct the
-         * return address to be _after_ the instruction.
-         */
-        if (fsr & (1 << 10))
-                regs->ARM_pc += 4;
-
-        return 0;
-}
-
-/*
- * Scan an IOP331 PCI bus.  sys->bus defines which bus we scan.
- */
-struct pci_bus *iop331_scan_bus(int nr, struct pci_sys_data *sys)
-{
-        return pci_scan_bus(sys->busnr, &iop331_ops, sys);
-}
-
-void iop331_init(void)
-{
-        DBG1("PCI:  Intel 80331 PCI init code.\n");
-        DBG1("\tATU: IOP331_ATUCMD=0x%04x\n", *IOP331_ATUCMD);
-        DBG1("\tATU: IOP331_OMWTVR0=0x%04x, IOP331_OIOWTVR=0x%04x\n",
-                        *IOP331_OMWTVR0,
-                        *IOP331_OIOWTVR);
-        DBG1("\tATU: IOP331_OMWTVR1=0x%04x\n", *IOP331_OMWTVR1);
-        DBG1("\tATU: IOP331_ATUCR=0x%08x\n", *IOP331_ATUCR);
-        DBG1("\tATU: IOP331_IABAR0=0x%08x IOP331_IALR0=0x%08x IOP331_IATVR0=%08x\n", *IOP331_IABAR0, *IOP331_IALR0, *IOP331_IATVR0);
-        DBG1("\tATU: IOP31_IABAR1=0x%08x IOP331_IALR1=0x%08x\n", *IOP331_IABAR1, *IOP331_IALR1);
-        DBG1("\tATU: IOP331_ERBAR=0x%08x IOP331_ERLR=0x%08x IOP331_ERTVR=%08x\n", *IOP331_ERBAR, *IOP331_ERLR, *IOP331_ERTVR);
-        DBG1("\tATU: IOP331_IABAR2=0x%08x IOP331_IALR2=0x%08x IOP331_IATVR2=%08x\n", *IOP331_IABAR2, *IOP331_IALR2, *IOP331_IATVR2);
-        DBG1("\tATU: IOP331_IABAR3=0x%08x IOP331_IALR3=0x%08x IOP331_IATVR3=%08x\n", *IOP331_IABAR3, *IOP331_IALR3, *IOP331_IATVR3);
-
-        hook_fault_code(16+6, iop331_pci_abort, SIGBUS, "imprecise external abort");
-}
-