Hi Suman, On Mon, Apr 03, 2017 at 03:35:46PM -0500, Suman Anna wrote: > > + iommu = platform_get_drvdata(pdev); > > + if (!iommu) { > > + of_node_put(np); > > + return -EINVAL; > > + } > > This change is causing the issues. OMAP IOMMU driver is not probed yet, > but this gets called during the driver's init function in > bus_set_iommu() and bus's iommu_ops gets set to NULL. The add_device > today is used to add the linking to an IOMMU device (currently name > primarily to support the legacy non-DT mode which is no longer an issue, > but a dev pointer can be used instead here, and the real enabling is > done during the domain's attach_dev callback.
Yeah, okay. Solving this problem requires more sophisticated handling in the iommu core code, which is not implemented yet. I drop this patch for now and do the device-linking and group-handling in attach_dev. This makes iommu-groups on omap useless for now, but it is a start. Joerg _______________________________________________ iommu mailing list iommu@lists.linux-foundation.org https://lists.linuxfoundation.org/mailman/listinfo/iommu