[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <8bafc3ec-8ee1-db91-2835-6a77a5dd2539@ozlabs.ru>
Date: Fri, 6 May 2016 16:34:21 +1000
From: Alexey Kardashevskiy <aik@...abs.ru>
To: Yongji Xie <xyjxie@...ux.vnet.ibm.com>, kvm@...r.kernel.org,
linux-kernel@...r.kernel.org, linux-pci@...r.kernel.org,
linuxppc-dev@...ts.ozlabs.org, iommu@...ts.linux-foundation.org
Cc: alex.williamson@...hat.com, bhelgaas@...gle.com,
benh@...nel.crashing.org, paulus@...ba.org, mpe@...erman.id.au,
joro@...tes.org, warrier@...ux.vnet.ibm.com,
zhong@...ux.vnet.ibm.com, nikunj@...ux.vnet.ibm.com,
eric.auger@...aro.org, will.deacon@....com,
gwshan@...ux.vnet.ibm.com, David.Laight@...LAB.COM,
alistair@...ple.id.au, ruscur@...sell.cc
Subject: Re: [PATCH 4/5] pci-ioda: Set PCI_BUS_FLAGS_MSI_REMAP for IODA host
bridge
On 04/27/2016 10:43 PM, Yongji Xie wrote:
> Any IODA host bridge have the capability of IRQ remapping.
> So we set PCI_BUS_FLAGS_MSI_REMAP when this kind of host birdge
> is detected.
>
> Signed-off-by: Yongji Xie <xyjxie@...ux.vnet.ibm.com>
Reviewed-by: Alexey Kardashevskiy <aik@...abs.ru>
> ---
> arch/powerpc/platforms/powernv/pci-ioda.c | 8 ++++++++
> 1 file changed, 8 insertions(+)
>
> diff --git a/arch/powerpc/platforms/powernv/pci-ioda.c b/arch/powerpc/platforms/powernv/pci-ioda.c
> index f90dc04..9557638 100644
> --- a/arch/powerpc/platforms/powernv/pci-ioda.c
> +++ b/arch/powerpc/platforms/powernv/pci-ioda.c
> @@ -3080,6 +3080,12 @@ static void pnv_pci_ioda_fixup(void)
> pnv_npu_ioda_fixup();
> }
>
> +int pnv_pci_ioda_root_bridge_prepare(struct pci_host_bridge *bridge)
> +{
> + bridge->bus->bus_flags |= PCI_BUS_FLAGS_MSI_REMAP;
> + return 0;
> +}
> +
> /*
> * Returns the alignment for I/O or memory windows for P2P
> * bridges. That actually depends on how PEs are segmented.
> @@ -3364,6 +3370,8 @@ static void __init pnv_pci_init_ioda_phb(struct device_node *np,
> */
> ppc_md.pcibios_fixup = pnv_pci_ioda_fixup;
>
> + ppc_md.pcibios_root_bridge_prepare = pnv_pci_ioda_root_bridge_prepare;
> +
> if (phb->type == PNV_PHB_NPU)
> hose->controller_ops = pnv_npu_ioda_controller_ops;
> else
>
--
Alexey
Powered by blists - more mailing lists