Commit 8b8da358 authored by Benjamin Herrenschmidt's avatar Benjamin Herrenschmidt Committed by Paul Mackerras

powerpc/pci: Split pcibios_fixup_bus() into bus setup and device setup

Currently, our PCI code uses the pcibios_fixup_bus() callback, which
is called by the generic code when probing PCI buses, for two
different things.

One is to set up things related to the bus itself, such as reading
bridge resources for P2P bridges, fixing them up, or setting up the
iommu's associated with bridges on some platforms.

The other is some setup for each individual device under that bridge,
mostly setting up DMA mappings and interrupts.

The problem is that this approach doesn't work well with PCI hotplug
when an existing bus is re-probed for new children.  We fix this
problem by splitting pcibios_fixup_bus into two routines:

	pcibios_setup_bus_self() is now called to setup the bus itself

	pcibios_setup_bus_devices() is now called to setup devices

pcibios_fixup_bus() is then modified to call these two after reading the
bridge bases, and the OF based PCI probe is modified to avoid calling
into the first one when rescanning an existing bridge.

[paulus@samba.org - fixed eeh.h for 32-bit compile now that pci-common.c
is including it unconditionally.]
Signed-off-by: default avatarBenjamin Herrenschmidt <benh@kernel.crashing.org>
Signed-off-by: default avatarPaul Mackerras <paulus@samba.org>
parent ab56ced9
...@@ -17,8 +17,8 @@ ...@@ -17,8 +17,8 @@
* Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA
*/ */
#ifndef _PPC64_EEH_H #ifndef _POWERPC_EEH_H
#define _PPC64_EEH_H #define _POWERPC_EEH_H
#ifdef __KERNEL__ #ifdef __KERNEL__
#include <linux/init.h> #include <linux/init.h>
...@@ -110,6 +110,7 @@ static inline void eeh_remove_bus_device(struct pci_dev *dev) { } ...@@ -110,6 +110,7 @@ static inline void eeh_remove_bus_device(struct pci_dev *dev) { }
#define EEH_IO_ERROR_VALUE(size) (-1UL) #define EEH_IO_ERROR_VALUE(size) (-1UL)
#endif /* CONFIG_EEH */ #endif /* CONFIG_EEH */
#ifdef CONFIG_PPC64
/* /*
* MMIO read/write operations with EEH support. * MMIO read/write operations with EEH support.
*/ */
...@@ -207,5 +208,6 @@ static inline void eeh_readsl(const volatile void __iomem *addr, void * buf, ...@@ -207,5 +208,6 @@ static inline void eeh_readsl(const volatile void __iomem *addr, void * buf,
eeh_check_failure(addr, *(u32*)buf); eeh_check_failure(addr, *(u32*)buf);
} }
#endif /* CONFIG_PPC64 */
#endif /* __KERNEL__ */ #endif /* __KERNEL__ */
#endif /* _PPC64_EEH_H */ #endif /* _POWERPC_EEH_H */
...@@ -221,6 +221,7 @@ extern void of_scan_pci_bridge(struct device_node *node, ...@@ -221,6 +221,7 @@ extern void of_scan_pci_bridge(struct device_node *node,
struct pci_dev *dev); struct pci_dev *dev);
extern void of_scan_bus(struct device_node *node, struct pci_bus *bus); extern void of_scan_bus(struct device_node *node, struct pci_bus *bus);
extern void of_rescan_bus(struct device_node *node, struct pci_bus *bus);
extern int pci_read_irq_line(struct pci_dev *dev); extern int pci_read_irq_line(struct pci_dev *dev);
...@@ -235,8 +236,8 @@ extern void pci_resource_to_user(const struct pci_dev *dev, int bar, ...@@ -235,8 +236,8 @@ extern void pci_resource_to_user(const struct pci_dev *dev, int bar,
const struct resource *rsrc, const struct resource *rsrc,
resource_size_t *start, resource_size_t *end); resource_size_t *start, resource_size_t *end);
extern void pcibios_fixup_of_probed_bus(struct pci_bus *bus); extern void pcibios_setup_bus_devices(struct pci_bus *bus);
extern void pcibios_setup_bus_self(struct pci_bus *bus);
#endif /* __KERNEL__ */ #endif /* __KERNEL__ */
#endif /* __ASM_POWERPC_PCI_H */ #endif /* __ASM_POWERPC_PCI_H */
...@@ -37,6 +37,7 @@ ...@@ -37,6 +37,7 @@
#include <asm/machdep.h> #include <asm/machdep.h>
#include <asm/ppc-pci.h> #include <asm/ppc-pci.h>
#include <asm/firmware.h> #include <asm/firmware.h>
#include <asm/eeh.h>
static DEFINE_SPINLOCK(hose_spinlock); static DEFINE_SPINLOCK(hose_spinlock);
...@@ -1074,31 +1075,17 @@ static void __devinit pcibios_fixup_bridge(struct pci_bus *bus) ...@@ -1074,31 +1075,17 @@ static void __devinit pcibios_fixup_bridge(struct pci_bus *bus)
} }
} }
static void __devinit __pcibios_fixup_bus(struct pci_bus *bus) void __devinit pcibios_setup_bus_devices(struct pci_bus *bus)
{ {
struct pci_dev *dev; struct pci_dev *dev;
pr_debug("PCI: Fixup bus %d (%s)\n", pr_debug("PCI: Fixup bus %d (%s)\n",
bus->number, bus->self ? pci_name(bus->self) : "PHB"); bus->number, bus->self ? pci_name(bus->self) : "PHB");
/* Fixup PCI<->PCI bridges. Host bridges are handled separately, for
* now differently between 32 and 64 bits.
*/
if (bus->self != NULL)
pcibios_fixup_bridge(bus);
/* Setup bus DMA mappings */
if (ppc_md.pci_dma_bus_setup)
ppc_md.pci_dma_bus_setup(bus);
/* Setup DMA for all PCI devices on that bus */ /* Setup DMA for all PCI devices on that bus */
list_for_each_entry(dev, &bus->devices, bus_list) list_for_each_entry(dev, &bus->devices, bus_list)
pcibios_setup_new_device(dev); pcibios_setup_new_device(dev);
/* Platform specific bus fixups */
if (ppc_md.pcibios_fixup_bus)
ppc_md.pcibios_fixup_bus(bus);
/* Read default IRQs and fixup if necessary */ /* Read default IRQs and fixup if necessary */
list_for_each_entry(dev, &bus->devices, bus_list) { list_for_each_entry(dev, &bus->devices, bus_list) {
pci_read_irq_line(dev); pci_read_irq_line(dev);
...@@ -1107,25 +1094,39 @@ static void __devinit __pcibios_fixup_bus(struct pci_bus *bus) ...@@ -1107,25 +1094,39 @@ static void __devinit __pcibios_fixup_bus(struct pci_bus *bus)
} }
} }
void __devinit pcibios_setup_bus_self(struct pci_bus *bus)
{
/* Fix up the bus resources */
if (bus->self != NULL)
pcibios_fixup_bridge(bus);
/* Platform specific bus fixups. This is currently only used
* by fsl_pci and I'm hoping getting rid of it at some point
*/
if (ppc_md.pcibios_fixup_bus)
ppc_md.pcibios_fixup_bus(bus);
/* Setup bus DMA mappings */
if (ppc_md.pci_dma_bus_setup)
ppc_md.pci_dma_bus_setup(bus);
}
void __devinit pcibios_fixup_bus(struct pci_bus *bus) void __devinit pcibios_fixup_bus(struct pci_bus *bus)
{ {
/* When called from the generic PCI probe, read PCI<->PCI bridge /* When called from the generic PCI probe, read PCI<->PCI bridge
* bases before proceeding * bases. This isn't called when generating the PCI tree from
* the OF device-tree.
*/ */
if (bus->self != NULL) if (bus->self != NULL)
pci_read_bridge_bases(bus); pci_read_bridge_bases(bus);
__pcibios_fixup_bus(bus);
}
EXPORT_SYMBOL(pcibios_fixup_bus);
/* When building a bus from the OF tree rather than probing, we need a /* Now fixup the bus bus */
* slightly different version of the fixup which doesn't read the pcibios_setup_bus_self(bus);
* bridge bases using config space accesses
*/ /* Now fixup devices on that bus */
void __devinit pcibios_fixup_of_probed_bus(struct pci_bus *bus) pcibios_setup_bus_devices(bus);
{
__pcibios_fixup_bus(bus);
} }
EXPORT_SYMBOL(pcibios_fixup_bus);
static int skip_isa_ioresource_align(struct pci_dev *dev) static int skip_isa_ioresource_align(struct pci_dev *dev)
{ {
...@@ -1392,6 +1393,7 @@ void __init pcibios_resource_survey(void) ...@@ -1392,6 +1393,7 @@ void __init pcibios_resource_survey(void)
} }
#ifdef CONFIG_HOTPLUG #ifdef CONFIG_HOTPLUG
/* This is used by the pSeries hotplug driver to allocate resource /* This is used by the pSeries hotplug driver to allocate resource
* of newly plugged busses. We can try to consolidate with the * of newly plugged busses. We can try to consolidate with the
* rest of the code later, for now, keep it as-is * rest of the code later, for now, keep it as-is
......
...@@ -189,8 +189,8 @@ struct pci_dev *of_create_pci_dev(struct device_node *node, ...@@ -189,8 +189,8 @@ struct pci_dev *of_create_pci_dev(struct device_node *node,
} }
EXPORT_SYMBOL(of_create_pci_dev); EXPORT_SYMBOL(of_create_pci_dev);
void __devinit of_scan_bus(struct device_node *node, static void __devinit __of_scan_bus(struct device_node *node,
struct pci_bus *bus) struct pci_bus *bus, int rescan_existing)
{ {
struct device_node *child; struct device_node *child;
const u32 *reg; const u32 *reg;
...@@ -215,8 +215,12 @@ void __devinit of_scan_bus(struct device_node *node, ...@@ -215,8 +215,12 @@ void __devinit of_scan_bus(struct device_node *node,
pr_debug(" dev header type: %x\n", dev->hdr_type); pr_debug(" dev header type: %x\n", dev->hdr_type);
} }
/* Ally all fixups */ /* Apply all fixups necessary. We don't fixup the bus "self"
pcibios_fixup_of_probed_bus(bus); * for an existing bridge that is being rescanned
*/
if (!rescan_existing)
pcibios_setup_bus_self(bus);
pcibios_setup_bus_devices(bus);
/* Now scan child busses */ /* Now scan child busses */
list_for_each_entry(dev, &bus->devices, bus_list) { list_for_each_entry(dev, &bus->devices, bus_list) {
...@@ -228,7 +232,20 @@ void __devinit of_scan_bus(struct device_node *node, ...@@ -228,7 +232,20 @@ void __devinit of_scan_bus(struct device_node *node,
} }
} }
} }
EXPORT_SYMBOL(of_scan_bus);
void __devinit of_scan_bus(struct device_node *node,
struct pci_bus *bus)
{
__of_scan_bus(node, bus, 0);
}
EXPORT_SYMBOL_GPL(of_scan_bus);
void __devinit of_rescan_bus(struct device_node *node,
struct pci_bus *bus)
{
__of_scan_bus(node, bus, 1);
}
EXPORT_SYMBOL_GPL(of_rescan_bus);
void __devinit of_scan_pci_bridge(struct device_node *node, void __devinit of_scan_pci_bridge(struct device_node *node,
struct pci_dev *dev) struct pci_dev *dev)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment