| Message ID | 1361191939-21260-3-git-send-email-Varun.Sethi@freescale.com (mailing list archive) |
|---|---|
| State | Superseded |
| Delegated to: | Kumar Gala |
| Headers | show |
This patch was submitted separately to linuxppc-dev (and was already applied). You don't need it in this patch set, right? Stuart On Mon, Feb 18, 2013 at 6:52 AM, Varun Sethi <Varun.Sethi@freescale.com> wrote: > The pci controller structure has a provision to store the device strcuture > pointer of the corresponding platform device. Currently this information is > not stored during fsl pci controller initialization. This information is > required while dealing with iommu groups for pci devices connected to the fsl > pci controller. For the case where the pci devices can't be paritioned, they > would fall under the same device group as the pci controller. > > This patch stores the platform device information in the pci controller > structure during initialization. > > Signed-off-by: Varun Sethi <Varun.Sethi@freescale.com> > --- > arch/powerpc/sysdev/fsl_pci.c | 9 +++++++-- > arch/powerpc/sysdev/fsl_pci.h | 2 +- > 2 files changed, 8 insertions(+), 3 deletions(-) > > diff --git a/arch/powerpc/sysdev/fsl_pci.c b/arch/powerpc/sysdev/fsl_pci.c > index 92a5915..b393ae7 100644 > --- a/arch/powerpc/sysdev/fsl_pci.c > +++ b/arch/powerpc/sysdev/fsl_pci.c > @@ -421,13 +421,16 @@ void fsl_pcibios_fixup_bus(struct pci_bus *bus) > } > } > > -int __init fsl_add_bridge(struct device_node *dev, int is_primary) > +int __init fsl_add_bridge(struct platform_device *pdev, int is_primary) > { > int len; > struct pci_controller *hose; > struct resource rsrc; > const int *bus_range; > u8 hdr_type, progif; > + struct device_node *dev; > + > + dev = pdev->dev.of_node; > > if (!of_device_is_available(dev)) { > pr_warning("%s: disabled\n", dev->full_name); > @@ -453,6 +456,8 @@ int __init fsl_add_bridge(struct device_node *dev, int is_primary) > if (!hose) > return -ENOMEM; > > + /* set platform device as the parent */ > + hose->parent = &pdev->dev; > hose->first_busno = bus_range ? bus_range[0] : 0x0; > hose->last_busno = bus_range ? bus_range[1] : 0xff; > > @@ -880,7 +885,7 @@ static int fsl_pci_probe(struct platform_device *pdev) > #endif > > node = pdev->dev.of_node; > - ret = fsl_add_bridge(node, fsl_pci_primary == node); > + ret = fsl_add_bridge(pdev, fsl_pci_primary == node); > > #ifdef CONFIG_SWIOTLB > if (ret == 0) { > diff --git a/arch/powerpc/sysdev/fsl_pci.h b/arch/powerpc/sysdev/fsl_pci.h > index d078537..c495c00 100644 > --- a/arch/powerpc/sysdev/fsl_pci.h > +++ b/arch/powerpc/sysdev/fsl_pci.h > @@ -91,7 +91,7 @@ struct ccsr_pci { > __be32 pex_err_cap_r3; /* 0x.e34 - PCIE error capture register 0 */ > }; > > -extern int fsl_add_bridge(struct device_node *dev, int is_primary); > +extern int fsl_add_bridge(struct platform_device *pdev, int is_primary); > extern void fsl_pcibios_fixup_bus(struct pci_bus *bus); > extern int mpc83xx_add_bridge(struct device_node *dev); > u64 fsl_pci_immrbar_base(struct pci_controller *hose); > -- > 1.7.4.1 > > > _______________________________________________ > iommu mailing list > iommu@lists.linux-foundation.org > https://lists.linuxfoundation.org/mailman/listinfo/iommu
This patch is not present in Joerg's tree and the add_device API in the PAMU driver requires this patch. -Varun > -----Original Message----- > From: Stuart Yoder [mailto:b08248@gmail.com] > Sent: Tuesday, February 26, 2013 5:39 AM > To: Sethi Varun-B16395 > Cc: iommu@lists.linux-foundation.org; linuxppc-dev@lists.ozlabs.org; > linux-kernel@vger.kernel.org; Wood Scott-B07421; Joerg Roedel; Yoder > Stuart-B08248 > Subject: Re: [PATCH 2/6] powerpc/fsl_pci: Store the platform device > information corresponding to the pci controller. > > This patch was submitted separately to linuxppc-dev (and was already > applied). You don't need it in this patch set, right? > > Stuart > > On Mon, Feb 18, 2013 at 6:52 AM, Varun Sethi <Varun.Sethi@freescale.com> > wrote: > > The pci controller structure has a provision to store the device > > strcuture pointer of the corresponding platform device. Currently this > > information is not stored during fsl pci controller initialization. > > This information is required while dealing with iommu groups for pci > > devices connected to the fsl pci controller. For the case where the > > pci devices can't be paritioned, they would fall under the same device > group as the pci controller. > > > > This patch stores the platform device information in the pci > > controller structure during initialization. > > > > Signed-off-by: Varun Sethi <Varun.Sethi@freescale.com> > > --- > > arch/powerpc/sysdev/fsl_pci.c | 9 +++++++-- > > arch/powerpc/sysdev/fsl_pci.h | 2 +- > > 2 files changed, 8 insertions(+), 3 deletions(-) > > > > diff --git a/arch/powerpc/sysdev/fsl_pci.c > > b/arch/powerpc/sysdev/fsl_pci.c index 92a5915..b393ae7 100644 > > --- a/arch/powerpc/sysdev/fsl_pci.c > > +++ b/arch/powerpc/sysdev/fsl_pci.c > > @@ -421,13 +421,16 @@ void fsl_pcibios_fixup_bus(struct pci_bus *bus) > > } > > } > > > > -int __init fsl_add_bridge(struct device_node *dev, int is_primary) > > +int __init fsl_add_bridge(struct platform_device *pdev, int > > +is_primary) > > { > > int len; > > struct pci_controller *hose; > > struct resource rsrc; > > const int *bus_range; > > u8 hdr_type, progif; > > + struct device_node *dev; > > + > > + dev = pdev->dev.of_node; > > > > if (!of_device_is_available(dev)) { > > pr_warning("%s: disabled\n", dev->full_name); @@ > > -453,6 +456,8 @@ int __init fsl_add_bridge(struct device_node *dev, int > is_primary) > > if (!hose) > > return -ENOMEM; > > > > + /* set platform device as the parent */ > > + hose->parent = &pdev->dev; > > hose->first_busno = bus_range ? bus_range[0] : 0x0; > > hose->last_busno = bus_range ? bus_range[1] : 0xff; > > > > @@ -880,7 +885,7 @@ static int fsl_pci_probe(struct platform_device > > *pdev) #endif > > > > node = pdev->dev.of_node; > > - ret = fsl_add_bridge(node, fsl_pci_primary == node); > > + ret = fsl_add_bridge(pdev, fsl_pci_primary == node); > > > > #ifdef CONFIG_SWIOTLB > > if (ret == 0) { > > diff --git a/arch/powerpc/sysdev/fsl_pci.h > > b/arch/powerpc/sysdev/fsl_pci.h index d078537..c495c00 100644 > > --- a/arch/powerpc/sysdev/fsl_pci.h > > +++ b/arch/powerpc/sysdev/fsl_pci.h > > @@ -91,7 +91,7 @@ struct ccsr_pci { > > __be32 pex_err_cap_r3; /* 0x.e34 - PCIE error capture > register 0 */ > > }; > > > > -extern int fsl_add_bridge(struct device_node *dev, int is_primary); > > +extern int fsl_add_bridge(struct platform_device *pdev, int > > +is_primary); > > extern void fsl_pcibios_fixup_bus(struct pci_bus *bus); extern int > > mpc83xx_add_bridge(struct device_node *dev); > > u64 fsl_pci_immrbar_base(struct pci_controller *hose); > > -- > > 1.7.4.1 > > > > > > _______________________________________________ > > iommu mailing list > > iommu@lists.linux-foundation.org > > https://lists.linuxfoundation.org/mailman/listinfo/iommu
On Tue, Feb 26, 2013 at 06:16:10AM +0000, Sethi Varun-B16395 wrote: > This patch is not present in Joerg's tree and the add_device API in > the PAMU driver requires this patch. Will this patch be part of v3.9-rc1? Joerg
This patch is present in the "next branch" of linux ppc tree maintained by Kumar Gala. Following is the commit id: 52c5affc545053d37c0b05224bbf70f5336caa20 I am not sure if this would be part of 3.9-rc1. Regards varun > -----Original Message----- > From: Joerg Roedel [mailto:joro@8bytes.org] > Sent: Wednesday, February 27, 2013 4:15 PM > To: Sethi Varun-B16395 > Cc: Stuart Yoder; iommu@lists.linux-foundation.org; linuxppc- > dev@lists.ozlabs.org; linux-kernel@vger.kernel.org; Wood Scott-B07421; > Yoder Stuart-B08248 > Subject: Re: [PATCH 2/6] powerpc/fsl_pci: Store the platform device > information corresponding to the pci controller. > > On Tue, Feb 26, 2013 at 06:16:10AM +0000, Sethi Varun-B16395 wrote: > > This patch is not present in Joerg's tree and the add_device API in > > the PAMU driver requires this patch. > > Will this patch be part of v3.9-rc1? > > > Joerg > >
On Feb 27, 2013, at 4:56 AM, Sethi Varun-B16395 wrote: > This patch is present in the "next branch" of linux ppc tree maintained by Kumar Gala. > Following is the commit id: > 52c5affc545053d37c0b05224bbf70f5336caa20 > > I am not sure if this would be part of 3.9-rc1. > > Regards > varun This is now in Linus's tree so will be in 3.9-rc1 - k
Hi Joerg, I have to post the next version of my patchset, should I base it on top of 3.9-rc1? By when would you move the iommu git tree to 3.9-rc1? Regards Varun > -----Original Message----- > From: Kumar Gala [mailto:galak@kernel.crashing.org] > Sent: Thursday, February 28, 2013 9:15 PM > To: Sethi Varun-B16395 > Cc: Joerg Roedel; Stuart Yoder; iommu@lists.linux-foundation.org; > linuxppc-dev@lists.ozlabs.org; linux-kernel@vger.kernel.org; Wood Scott- > B07421; Yoder Stuart-B08248 > Subject: Re: [PATCH 2/6] powerpc/fsl_pci: Store the platform device > information corresponding to the pci controller. > > > On Feb 27, 2013, at 4:56 AM, Sethi Varun-B16395 wrote: > > > This patch is present in the "next branch" of linux ppc tree maintained > by Kumar Gala. > > Following is the commit id: > > 52c5affc545053d37c0b05224bbf70f5336caa20 > > > > I am not sure if this would be part of 3.9-rc1. > > > > Regards > > varun > > This is now in Linus's tree so will be in 3.9-rc1 > > - k >
Yes, please base your patches on the latest upstream-tag. I will move my tree to v3.9-rc1 soon, there are some fixes that need to go upstream. On Thu, Mar 07, 2013 at 09:14:21AM +0000, Sethi Varun-B16395 wrote: > Hi Joerg, > I have to post the next version of my patchset, should I base it on top of 3.9-rc1? > By when would you move the iommu git tree to 3.9-rc1? > > Regards > Varun > > > -----Original Message----- > > From: Kumar Gala [mailto:galak@kernel.crashing.org] > > Sent: Thursday, February 28, 2013 9:15 PM > > To: Sethi Varun-B16395 > > Cc: Joerg Roedel; Stuart Yoder; iommu@lists.linux-foundation.org; > > linuxppc-dev@lists.ozlabs.org; linux-kernel@vger.kernel.org; Wood Scott- > > B07421; Yoder Stuart-B08248 > > Subject: Re: [PATCH 2/6] powerpc/fsl_pci: Store the platform device > > information corresponding to the pci controller. > > > > > > On Feb 27, 2013, at 4:56 AM, Sethi Varun-B16395 wrote: > > > > > This patch is present in the "next branch" of linux ppc tree maintained > > by Kumar Gala. > > > Following is the commit id: > > > 52c5affc545053d37c0b05224bbf70f5336caa20 > > > > > > I am not sure if this would be part of 3.9-rc1. > > > > > > Regards > > > varun > > > > This is now in Linus's tree so will be in 3.9-rc1 > > > > - k > > > >
diff --git a/arch/powerpc/sysdev/fsl_pci.c b/arch/powerpc/sysdev/fsl_pci.c index 92a5915..b393ae7 100644 --- a/arch/powerpc/sysdev/fsl_pci.c +++ b/arch/powerpc/sysdev/fsl_pci.c @@ -421,13 +421,16 @@ void fsl_pcibios_fixup_bus(struct pci_bus *bus) } } -int __init fsl_add_bridge(struct device_node *dev, int is_primary) +int __init fsl_add_bridge(struct platform_device *pdev, int is_primary) { int len; struct pci_controller *hose; struct resource rsrc; const int *bus_range; u8 hdr_type, progif; + struct device_node *dev; + + dev = pdev->dev.of_node; if (!of_device_is_available(dev)) { pr_warning("%s: disabled\n", dev->full_name); @@ -453,6 +456,8 @@ int __init fsl_add_bridge(struct device_node *dev, int is_primary) if (!hose) return -ENOMEM; + /* set platform device as the parent */ + hose->parent = &pdev->dev; hose->first_busno = bus_range ? bus_range[0] : 0x0; hose->last_busno = bus_range ? bus_range[1] : 0xff; @@ -880,7 +885,7 @@ static int fsl_pci_probe(struct platform_device *pdev) #endif node = pdev->dev.of_node; - ret = fsl_add_bridge(node, fsl_pci_primary == node); + ret = fsl_add_bridge(pdev, fsl_pci_primary == node); #ifdef CONFIG_SWIOTLB if (ret == 0) { diff --git a/arch/powerpc/sysdev/fsl_pci.h b/arch/powerpc/sysdev/fsl_pci.h index d078537..c495c00 100644 --- a/arch/powerpc/sysdev/fsl_pci.h +++ b/arch/powerpc/sysdev/fsl_pci.h @@ -91,7 +91,7 @@ struct ccsr_pci { __be32 pex_err_cap_r3; /* 0x.e34 - PCIE error capture register 0 */ }; -extern int fsl_add_bridge(struct device_node *dev, int is_primary); +extern int fsl_add_bridge(struct platform_device *pdev, int is_primary); extern void fsl_pcibios_fixup_bus(struct pci_bus *bus); extern int mpc83xx_add_bridge(struct device_node *dev); u64 fsl_pci_immrbar_base(struct pci_controller *hose);
The pci controller structure has a provision to store the device strcuture pointer of the corresponding platform device. Currently this information is not stored during fsl pci controller initialization. This information is required while dealing with iommu groups for pci devices connected to the fsl pci controller. For the case where the pci devices can't be paritioned, they would fall under the same device group as the pci controller. This patch stores the platform device information in the pci controller structure during initialization. Signed-off-by: Varun Sethi <Varun.Sethi@freescale.com> --- arch/powerpc/sysdev/fsl_pci.c | 9 +++++++-- arch/powerpc/sysdev/fsl_pci.h | 2 +- 2 files changed, 8 insertions(+), 3 deletions(-)