On Tuesday 31 March 2015 16:00:42 Daniel Axtens wrote:
> Introduces the pci_controller_ops structure.
> Add pci_controller_ops.dma_dev_setup, shadowing ppc_md.pci_dma_dev_setup.
> Add a shim, and change the callsites to use the shim.
>
> Signed-off-by: Daniel Axtens <d...@axtens.net>
>
> ---
>
> v1 --> v2:
> - Better commit message
> - Use phb in favour of hose
> - Make shim name match ppc_md name, not pci_controller_ops name.
> ---
> arch/powerpc/include/asm/pci-bridge.h | 21 +++++++++++++++++++++
> arch/powerpc/kernel/pci-common.c | 3 +--
> 2 files changed, 22 insertions(+), 2 deletions(-)
>
> diff --git a/arch/powerpc/include/asm/pci-bridge.h
> b/arch/powerpc/include/asm/pci-bridge.h
> index 546d036..347d49d 100644
> --- a/arch/powerpc/include/asm/pci-bridge.h
> +++ b/arch/powerpc/include/asm/pci-bridge.h
> @@ -15,6 +15,13 @@
> struct device_node;
>
> /*
> + * PCI controller operations
> + */
> +struct pci_controller_ops {
> + void (*dma_dev_setup)(struct pci_dev *dev);
> +};
> +
> +/*
Please see https://patchwork.ozlabs.org/patch/431333/ for related work.
I think it would be better not to introduce another architecture-specific
pci host bridge operations structure, but instead consolidate into
the one that is already there. We are also adding a generic way to set up
PCI DMA, so it would seems reasonable to hook into that place.
Arnd
_______________________________________________
Linuxppc-dev mailing list
Linuxppc-dev@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/linuxppc-dev
