Commit 798248a3 authored by Daniel Axtens's avatar Daniel Axtens Committed by Michael Ellerman

powerpc: dart_iommu: optionally populate controller_ops on init

If a pci_controller_ops struct is provided to iommu_init_early_dart,
populate that with the DMA setup ops, rather than ppc_md. If NULL is
provided, populate ppc_md as before.

This also patches the call sites for Maple and Power Mac to pass
NULL, so existing behaviour is preserved.

The benefit of making this optional is that it means we don't have
to change dart, Maple and Power Mac over to the controller_ops
system in one fell swoop.
Signed-off-by: default avatarDaniel Axtens <dja@axtens.net>
Signed-off-by: default avatarMichael Ellerman <mpe@ellerman.id.au>
parent cd16c7ba
...@@ -29,6 +29,7 @@ ...@@ -29,6 +29,7 @@
#include <linux/bitops.h> #include <linux/bitops.h>
#include <asm/machdep.h> #include <asm/machdep.h>
#include <asm/types.h> #include <asm/types.h>
#include <asm/pci-bridge.h>
#define IOMMU_PAGE_SHIFT_4K 12 #define IOMMU_PAGE_SHIFT_4K 12
#define IOMMU_PAGE_SIZE_4K (ASM_CONST(1) << IOMMU_PAGE_SHIFT_4K) #define IOMMU_PAGE_SIZE_4K (ASM_CONST(1) << IOMMU_PAGE_SHIFT_4K)
...@@ -169,7 +170,7 @@ extern void iommu_unmap_page(struct iommu_table *tbl, dma_addr_t dma_handle, ...@@ -169,7 +170,7 @@ extern void iommu_unmap_page(struct iommu_table *tbl, dma_addr_t dma_handle,
struct dma_attrs *attrs); struct dma_attrs *attrs);
extern void iommu_init_early_pSeries(void); extern void iommu_init_early_pSeries(void);
extern void iommu_init_early_dart(void); extern void iommu_init_early_dart(struct pci_controller_ops *controller_ops);
extern void iommu_init_early_pasemi(void); extern void iommu_init_early_pasemi(void);
extern void alloc_dart_table(void); extern void alloc_dart_table(void);
......
...@@ -203,7 +203,7 @@ static void __init maple_init_early(void) ...@@ -203,7 +203,7 @@ static void __init maple_init_early(void)
{ {
DBG(" -> maple_init_early\n"); DBG(" -> maple_init_early\n");
iommu_init_early_dart(); iommu_init_early_dart(NULL);
DBG(" <- maple_init_early\n"); DBG(" <- maple_init_early\n");
} }
......
...@@ -473,7 +473,7 @@ static void __init pmac_init_early(void) ...@@ -473,7 +473,7 @@ static void __init pmac_init_early(void)
udbg_adb_init(!!strstr(boot_command_line, "btextdbg")); udbg_adb_init(!!strstr(boot_command_line, "btextdbg"));
#ifdef CONFIG_PPC64 #ifdef CONFIG_PPC64
iommu_init_early_dart(); iommu_init_early_dart(NULL);
#endif #endif
/* SMP Init has to be done early as we need to patch up /* SMP Init has to be done early as we need to patch up
......
...@@ -369,7 +369,7 @@ static int dart_dma_set_mask(struct device *dev, u64 dma_mask) ...@@ -369,7 +369,7 @@ static int dart_dma_set_mask(struct device *dev, u64 dma_mask)
return 0; return 0;
} }
void __init iommu_init_early_dart(void) void __init iommu_init_early_dart(struct pci_controller_ops *controller_ops)
{ {
struct device_node *dn; struct device_node *dn;
...@@ -395,15 +395,23 @@ void __init iommu_init_early_dart(void) ...@@ -395,15 +395,23 @@ void __init iommu_init_early_dart(void)
if (dart_is_u4) if (dart_is_u4)
ppc_md.dma_set_mask = dart_dma_set_mask; ppc_md.dma_set_mask = dart_dma_set_mask;
ppc_md.pci_dma_dev_setup = pci_dma_dev_setup_dart; if (controller_ops) {
ppc_md.pci_dma_bus_setup = pci_dma_bus_setup_dart; controller_ops->dma_dev_setup = pci_dma_dev_setup_dart;
controller_ops->dma_bus_setup = pci_dma_bus_setup_dart;
} else {
ppc_md.pci_dma_dev_setup = pci_dma_dev_setup_dart;
ppc_md.pci_dma_bus_setup = pci_dma_bus_setup_dart;
}
/* Setup pci_dma ops */ /* Setup pci_dma ops */
set_pci_dma_ops(&dma_iommu_ops); set_pci_dma_ops(&dma_iommu_ops);
return; return;
bail: bail:
/* If init failed, use direct iommu and null setup functions */ /* If init failed, use direct iommu and null setup functions */
if (controller_ops) {
controller_ops->dma_dev_setup = NULL;
controller_ops->dma_bus_setup = NULL;
}
ppc_md.pci_dma_dev_setup = NULL; ppc_md.pci_dma_dev_setup = NULL;
ppc_md.pci_dma_bus_setup = NULL; ppc_md.pci_dma_bus_setup = NULL;
......
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