On 11/01/2013 12:49 AM, Hiroshi Doyu wrote: > On Thu, 31 Oct 2013 18:53:22 +0100 > Stephen Warren <swar...@wwwdotorg.org> wrote: > ... >> We're talking about memory-mapped on-SoC devices here, that generally >> only exist inside Tegra SoCs. >> >> Even ignoring that (i.e. expanding the argument to arbitrary modules), >> having drivers that perform bus-master transactions call a function >> of_iommu_attach() or similar, which does nothing if the device isn't >> behind an IOMMU but otherwise does whatever is required, seems like it >> isn't much of an imposition. > > Where do you expect of_iommu_attach() to be called? > I thought something below:
> diff --git a/drivers/base/dd.c b/drivers/base/dd.c > @@ -278,6 +278,10 @@ static int really_probe(struct device *dev, struct > device_driver *drv) > if (ret) > goto probe_failed; > > + ret = of_iommu_attach(dev); > + if (ret) > + goto probe_failed; That sounds reasonable. _______________________________________________ iommu mailing list iommu@lists.linux-foundation.org https://lists.linuxfoundation.org/mailman/listinfo/iommu