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 devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
