Re: [PATCH 2/6] powerpc/fsl_pci: Store the platform device information corresponding to the pci controller.

2013-03-07 Thread Joerg Roedel
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 +, 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:ga...@kernel.crashing.org]
> > Sent: Thursday, February 28, 2013 9:15 PM
> > To: Sethi Varun-B16395
> > Cc: Joerg Roedel; Stuart Yoder; io...@lists.linux-foundation.org;
> > linuxppc-...@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
> > 
> 
> 

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


RE: [PATCH 2/6] powerpc/fsl_pci: Store the platform device information corresponding to the pci controller.

2013-03-07 Thread Sethi Varun-B16395
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:ga...@kernel.crashing.org]
> Sent: Thursday, February 28, 2013 9:15 PM
> To: Sethi Varun-B16395
> Cc: Joerg Roedel; Stuart Yoder; io...@lists.linux-foundation.org;
> linuxppc-...@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
> 


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


RE: [PATCH 2/6] powerpc/fsl_pci: Store the platform device information corresponding to the pci controller.

2013-03-07 Thread Sethi Varun-B16395
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:ga...@kernel.crashing.org]
 Sent: Thursday, February 28, 2013 9:15 PM
 To: Sethi Varun-B16395
 Cc: Joerg Roedel; Stuart Yoder; io...@lists.linux-foundation.org;
 linuxppc-...@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
 


--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH 2/6] powerpc/fsl_pci: Store the platform device information corresponding to the pci controller.

2013-03-07 Thread Joerg Roedel
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 +, 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:ga...@kernel.crashing.org]
  Sent: Thursday, February 28, 2013 9:15 PM
  To: Sethi Varun-B16395
  Cc: Joerg Roedel; Stuart Yoder; io...@lists.linux-foundation.org;
  linuxppc-...@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
  
 
 

--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH 2/6] powerpc/fsl_pci: Store the platform device information corresponding to the pci controller.

2013-02-28 Thread Kumar Gala

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

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH 2/6] powerpc/fsl_pci: Store the platform device information corresponding to the pci controller.

2013-02-28 Thread Kumar Gala

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

--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


RE: [PATCH 2/6] powerpc/fsl_pci: Store the platform device information corresponding to the pci controller.

2013-02-27 Thread Sethi Varun-B16395
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:j...@8bytes.org]
> Sent: Wednesday, February 27, 2013 4:15 PM
> To: Sethi Varun-B16395
> Cc: Stuart Yoder; io...@lists.linux-foundation.org; linuxppc-
> d...@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 +, 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
> 
> 


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH 2/6] powerpc/fsl_pci: Store the platform device information corresponding to the pci controller.

2013-02-27 Thread Joerg Roedel
On Tue, Feb 26, 2013 at 06:16:10AM +, 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


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH 2/6] powerpc/fsl_pci: Store the platform device information corresponding to the pci controller.

2013-02-27 Thread Joerg Roedel
On Tue, Feb 26, 2013 at 06:16:10AM +, 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


--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


RE: [PATCH 2/6] powerpc/fsl_pci: Store the platform device information corresponding to the pci controller.

2013-02-27 Thread Sethi Varun-B16395
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:j...@8bytes.org]
 Sent: Wednesday, February 27, 2013 4:15 PM
 To: Sethi Varun-B16395
 Cc: Stuart Yoder; io...@lists.linux-foundation.org; linuxppc-
 d...@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 +, 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
 
 


--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


RE: [PATCH 2/6] powerpc/fsl_pci: Store the platform device information corresponding to the pci controller.

2013-02-25 Thread Sethi Varun-B16395
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:b08...@gmail.com]
> Sent: Tuesday, February 26, 2013 5:39 AM
> To: Sethi Varun-B16395
> Cc: io...@lists.linux-foundation.org; linuxppc-...@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 
> 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 
> > ---
> >  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 = >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
> > io...@lists.linux-foundation.org
> > https://lists.linuxfoundation.org/mailman/listinfo/iommu


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH 2/6] powerpc/fsl_pci: Store the platform device information corresponding to the pci controller.

2013-02-25 Thread Stuart Yoder
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  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 
> ---
>  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 = >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
> io...@lists.linux-foundation.org
> https://lists.linuxfoundation.org/mailman/listinfo/iommu
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


RE: [PATCH 2/6] powerpc/fsl_pci: Store the platform device information corresponding to the pci controller.

2013-02-25 Thread Sethi Varun-B16395
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:b08...@gmail.com]
 Sent: Tuesday, February 26, 2013 5:39 AM
 To: Sethi Varun-B16395
 Cc: io...@lists.linux-foundation.org; linuxppc-...@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.se...@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.se...@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
  io...@lists.linux-foundation.org
  https://lists.linuxfoundation.org/mailman/listinfo/iommu


--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH 2/6] powerpc/fsl_pci: Store the platform device information corresponding to the pci controller.

2013-02-25 Thread Stuart Yoder
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.se...@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.se...@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
 io...@lists.linux-foundation.org
 https://lists.linuxfoundation.org/mailman/listinfo/iommu
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


[PATCH 2/6] powerpc/fsl_pci: Store the platform device information corresponding to the pci controller.

2013-02-18 Thread Varun Sethi
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 
---
 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 = >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


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


[PATCH 2/6] powerpc/fsl_pci: Store the platform device information corresponding to the pci controller.

2013-02-18 Thread Varun Sethi
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.se...@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


--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/