Re: [PATCHv3 01/19] [HACK] of: dev_node has struct device pointer

2013-10-25 Thread Grant Likely
On Thu, 24 Oct 2013 04:36:30 -0500, Kumar Gala wrote: > > On Oct 24, 2013, at 4:21 AM, Hiroshi Doyu wrote: > > > Hi Grant, > > > > Grant Likely wrote @ Thu, 24 Oct 2013 10:55:31 > > +0200: > > > >>> diff --git a/include/linux/of.h b/include/linux/of.h > >>> index f95aee3..638a88a 100644

Re: [PATCHv3 10/19] iommu/tegra: smmu: Get "nvidia,swgroups" from DT

2013-10-25 Thread Grant Likely
On Fri, 18 Oct 2013 13:26:51 +0300, Hiroshi Doyu wrote: > This provides the info about which H/W Accelerators are supported on > Tegra SoC. This info is passed from DT. This is necessary to have the > unified SMMU driver among Tegra SoCs. Instead of using platform data, > DT passes "nvidia,swgroup

Re: [PATCHv3 01/19] [HACK] of: dev_node has struct device pointer

2013-10-25 Thread Grant Likely
On Fri, 18 Oct 2013 13:26:42 +0300, Hiroshi Doyu wrote: > To prevent of_platform_populate() from trying to populate duplicate > devices if a device has been already populated. > > Signed-off-by: Hiroshi Doyu > --- > drivers/of/platform.c | 6 ++ > include/linux/of.h| 1 + > 2 files chan

Re: [PATCHv3 01/19] [HACK] of: dev_node has struct device pointer

2013-10-25 Thread Kumar Gala
On Oct 24, 2013, at 4:21 AM, Hiroshi Doyu wrote: > Hi Grant, > > Grant Likely wrote @ Thu, 24 Oct 2013 10:55:31 > +0200: > >>> diff --git a/include/linux/of.h b/include/linux/of.h >>> index f95aee3..638a88a 100644 >>> --- a/include/linux/of.h >>> +++ b/include/linux/of.h >>> @@ -60,6 +60,

Re: [PATCHv3 01/19] [HACK] of: dev_node has struct device pointer

2013-10-25 Thread Grant Likely
On Fri, 25 Oct 2013 12:49:38 +0200, Thierry Reding wrote: > On Fri, Oct 25, 2013 at 11:49:05AM +0200, Hiroshi Doyu wrote: > > Thierry Reding wrote @ Fri, 25 Oct 2013 11:11:05 > > +0200: > > > > > > > This is actually "the other problem that I'm aware of that could > > > > > benefit > > > > >

Re: [PATCHv3 01/19] [HACK] of: dev_node has struct device pointer

2013-10-25 Thread Grant Likely
On Thu, 24 Oct 2013 11:21:15 +0200, Hiroshi Doyu wrote: > Hi Grant, > > Grant Likely wrote @ Thu, 24 Oct 2013 10:55:31 > +0200: > > > > diff --git a/include/linux/of.h b/include/linux/of.h > > > index f95aee3..638a88a 100644 > > > --- a/include/linux/of.h > > > +++ b/include/linux/of.h > >

Re: [PATCHv3 14/19] iommu/tegra: smmu: Get "nvidia, memory-clients" from DT

2013-10-25 Thread Grant Likely
On Fri, 18 Oct 2013 13:26:55 +0300, Hiroshi Doyu wrote: > This provides the info about which swgroups a device belongs to. This > info is passed from DT. This is necessary for the unified SMMU driver > among Tegra SoCs since each has different H/W accelerators. > > Signed-off-by: Hiroshi Doyu >

Re: [PATCHv3 01/19] [HACK] of: dev_node has struct device pointer

2013-10-25 Thread Thierry Reding
On Fri, Oct 25, 2013 at 02:20:51PM +0100, Will Deacon wrote: > On Fri, Oct 25, 2013 at 09:22:02AM +0100, Hiroshi Doyu wrote: > > Thierry Reding wrote @ Fri, 25 Oct 2013 09:56:55 > > +0200: > > > > > I suspect that there will be enough differences between the various > > > IOMMU implementations t

Re: [PATCHv3 01/19] [HACK] of: dev_node has struct device pointer

2013-10-25 Thread Will Deacon
On Fri, Oct 25, 2013 at 09:22:02AM +0100, Hiroshi Doyu wrote: > Thierry Reding wrote @ Fri, 25 Oct 2013 09:56:55 > +0200: > > > I suspect that there will be enough differences between the various > > IOMMU implementations that we won't be able to have a unified binding > > (especially for how to

Re: [PATCHv3 01/19] [HACK] of: dev_node has struct device pointer

2013-10-25 Thread Hiroshi Doyu
Thierry Reding wrote @ Fri, 25 Oct 2013 12:49:38 +0200: > The way notifiers work is that they run completely hidden from whatever > triggers them. For instance you register the IOMMU bus notifier from the > IOMMU driver (by calling bus_set_iommu()). That registers a function to > be called when

Re: [PATCHv3 01/19] [HACK] of: dev_node has struct device pointer

2013-10-25 Thread Thierry Reding
On Fri, Oct 25, 2013 at 11:49:05AM +0200, Hiroshi Doyu wrote: > Thierry Reding wrote @ Fri, 25 Oct 2013 11:11:05 > +0200: > > > > > This is actually "the other problem that I'm aware of that could benefit > > > > from [interrupt resolution at probe time]". My idea was that once we had > > > > a

Re: [PATCHv3 01/19] [HACK] of: dev_node has struct device pointer

2013-10-25 Thread Hiroshi Doyu
Thierry Reding wrote @ Fri, 25 Oct 2013 11:11:05 +0200: > > > This is actually "the other problem that I'm aware of that could benefit > > > from [interrupt resolution at probe time]". My idea was that once we had > > > a way within the driver core to resolve interrupt references at probe > > >

Re: [PATCHv3 01/19] [HACK] of: dev_node has struct device pointer

2013-10-25 Thread Thierry Reding
On Fri, Oct 25, 2013 at 10:25:49AM +0200, Hiroshi Doyu wrote: > Thierry Reding wrote @ Fri, 25 Oct 2013 09:56:55 > +0200: > > > > > This patch is a part of HACK to control device instanciation order. We > > > > have an IOMMU device(platform) which needs to be instanciated earlier > > > > than ot

Re: [PATCHv3 01/19] [HACK] of: dev_node has struct device pointer

2013-10-25 Thread Hiroshi Doyu
Thierry Reding wrote @ Fri, 25 Oct 2013 09:56:55 +0200: > > > This patch is a part of HACK to control device instanciation order. We > > > have an IOMMU device(platform) which needs to be instanciated earlier > > > than other (platform)devices so that IOMMU driver would configure them > > > as I

Re: [PATCHv3 01/19] [HACK] of: dev_node has struct device pointer

2013-10-25 Thread Hiroshi Doyu
Thierry Reding wrote @ Fri, 25 Oct 2013 09:56:55 +0200: > I suspect that there will be enough differences between the various > IOMMU implementations that we won't be able to have a unified binding > (especially for how to associate devices with a particular virtual > address space), but perhaps

Re: [PATCHv3 01/19] [HACK] of: dev_node has struct device pointer

2013-10-25 Thread Thierry Reding
On Fri, Oct 25, 2013 at 01:10:38AM +0100, Grant Likely wrote: > On Thu, 24 Oct 2013 11:21:15 +0200, Hiroshi Doyu wrote: > > Hi Grant, > > > > Grant Likely wrote @ Thu, 24 Oct 2013 10:55:31 > > +0200: > > > > > > diff --git a/include/linux/of.h b/include/linux/of.h > > > > index f95aee3..63