[POWERPC] Added indirect_type to handle variants of PCI ops
authorKumar Gala <galak@kernel.crashing.org>
Mon, 25 Jun 2007 20:19:48 +0000 (15:19 -0500)
committerKumar Gala <galak@kernel.crashing.org>
Fri, 29 Jun 2007 06:58:20 +0000 (01:58 -0500)
The generic PCI config ops indirect support for ppc32 covers only two
cases (implicit vs explicit) type 0/1 config cycles via set_cfg_type.
Added a indirect_type bit mask to handle other variants.

Added support for PCI-e extended registers and moved the cfg_type
handling into the bit mask for ARCH=powerpc.  We can also use this to
handle indirect quirks.

Signed-off-by: Kumar Gala <galak@kernel.crashing.org>
arch/powerpc/sysdev/indirect_pci.c
include/asm-ppc/pci-bridge.h

index efe3cff8dcd153284b318d189e08384493c097c4..3a1612253c600b33d2aee10daf4f83d7b360121f 100644 (file)
@@ -33,22 +33,27 @@ indirect_read_config(struct pci_bus *bus, unsigned int devfn, int offset,
        struct pci_controller *hose = bus->sysdata;
        volatile void __iomem *cfg_data;
        u8 cfg_type = 0;
-       u32 bus_no;
+       u32 bus_no, reg;
 
        if (ppc_md.pci_exclude_device)
                if (ppc_md.pci_exclude_device(hose, bus->number, devfn))
                        return PCIBIOS_DEVICE_NOT_FOUND;
        
-       if (hose->set_cfg_type)
+       if (hose->indirect_type & PPC_INDIRECT_TYPE_SET_CFG_TYPE)
                if (bus->number != hose->first_busno)
                        cfg_type = 1;
 
        bus_no = (bus->number == hose->first_busno) ?
                        hose->self_busno : bus->number;
 
+       if (hose->indirect_type & PPC_INDIRECT_TYPE_EXT_REG)
+               reg = ((offset & 0xf00) << 16) | (offset & 0xfc);
+       else
+               reg = offset & 0xfc;
+
        PCI_CFG_OUT(hose->cfg_addr,
                 (0x80000000 | (bus_no << 16)
-                 | (devfn << 8) | ((offset & 0xfc) | cfg_type)));
+                 | (devfn << 8) | reg | cfg_type));
 
        /*
         * Note: the caller has already checked that offset is
@@ -76,22 +81,27 @@ indirect_write_config(struct pci_bus *bus, unsigned int devfn, int offset,
        struct pci_controller *hose = bus->sysdata;
        volatile void __iomem *cfg_data;
        u8 cfg_type = 0;
-       u32 bus_no;
+       u32 bus_no, reg;
 
        if (ppc_md.pci_exclude_device)
                if (ppc_md.pci_exclude_device(hose, bus->number, devfn))
                        return PCIBIOS_DEVICE_NOT_FOUND;
 
-       if (hose->set_cfg_type)
+       if (hose->indirect_type & PPC_INDIRECT_TYPE_SET_CFG_TYPE)
                if (bus->number != hose->first_busno)
                        cfg_type = 1;
 
        bus_no = (bus->number == hose->first_busno) ?
                        hose->self_busno : bus->number;
 
+       if (hose->indirect_type & PPC_INDIRECT_TYPE_EXT_REG)
+               reg = ((offset & 0xf00) << 16) | (offset & 0xfc);
+       else
+               reg = offset & 0xfc;
+
        PCI_CFG_OUT(hose->cfg_addr,
                 (0x80000000 | (bus_no << 16)
-                 | (devfn << 8) | ((offset & 0xfc) | cfg_type)));
+                 | (devfn << 8) | reg | cfg_type));
 
        /*
         * Note: the caller has already checked that offset is
index 70aab6691043e1d38f9fcf6dd48906a8e950e405..0dee56b839a44d61c8a510686d3cfd8b7643ea3d 100644 (file)
@@ -65,9 +65,19 @@ struct pci_controller {
        /*
         * If set, indirect method will set the cfg_type bit as
         * needed to generate type 1 configuration transactions.
+        * use only on ARCH=ppc
         */
        int set_cfg_type;
 
+       /*
+        * Used for variants of PCI indirect handling and possible quirks:
+        *  SET_CFG_TYPE - used on 4xx or any PHB that does explicit type0/1
+        *  EXT_REG - provides access to PCI-e extended registers
+        */
+#define PPC_INDIRECT_TYPE_SET_CFG_TYPE         (0x00000001)
+#define PPC_INDIRECT_TYPE_EXT_REG              (0x00000002)
+       u32 indirect_type;
+
        /* Currently, we limit ourselves to 1 IO range and 3 mem
         * ranges since the common pci_bus structure can't handle more
         */