On Tuesday 27 January 2015 22:50:59 Suman Tripathi wrote: > @@ -174,6 +176,13 @@ static int sdhci_arasan_probe(struct platform_device *pdev) > pltfm_host->priv = sdhci_arasan; > pltfm_host->clk = clk_xin; > > + ret = xgene_ahbc_iommu_attach_device(&pdev->dev); > + if (ret < 0 && IS_ENABLED(CONFIG_XGENE_AHBC_IOMMU)) { > + dev_err(&pdev->dev, > + "unable to attach device to iommu %x\n", ret); > + goto clk_disable_all; > + } > + > ret = sdhci_add_host(host); > if (ret) { > dev_err(&pdev->dev, "platform register failed (%u)\n", ret); > This looks wrong: the iommu should be automatically configured for devices that need it, without any interaction from the driver. Arnd -- To unsubscribe from this list: send the line "unsubscribe linux-mmc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html