[PATCH] drm/mediatek: Do not select IOMMU_DMA
Thierry Reding
thierry.reding at gmail.com
Fri Jun 3 15:31:48 UTC 2016
From: Thierry Reding <treding at nvidia.com>
The IOMMU_DMA symbol depends on IOMMU_API, but DRM_MEDIATEK itself does
not have a dependency on that symbol.
Furthermore the driver doesn't use any of the code enabled by IOMMU_DMA
and the only IOMMU API call that it uses, iommu_present(), has a dummy
implementation if IOMMU_API isn't selected. Effectively there really is
no build time dependency on the IOMMU API.
Because of all of the above, the dependency on IOMMU_DMA or IOMMU_API
doesn't belong in the Mediatek DRM driver, but is really a matter of
platform integration. A default configuration should be enabling IOMMU
support if the driver should take advantage of it.
Note that the driver currently defers probe indefinitely if no IOMMU is
registered on the parent bus. There's really no reason for it to do that
since the driver exclusively uses the DMA API to allocate and map memory
and should therefore work even if no IOMMU is registered and for example
the contiguous memory allocator is enabled instead.
Signed-off-by: Thierry Reding <treding at nvidia.com>
---
drivers/gpu/drm/mediatek/Kconfig | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/gpu/drm/mediatek/Kconfig b/drivers/gpu/drm/mediatek/Kconfig
index eeefc971801a..9eefecedc3da 100644
--- a/drivers/gpu/drm/mediatek/Kconfig
+++ b/drivers/gpu/drm/mediatek/Kconfig
@@ -6,7 +6,6 @@ config DRM_MEDIATEK
select DRM_KMS_HELPER
select DRM_MIPI_DSI
select DRM_PANEL
- select IOMMU_DMA
select MEMORY
select MTK_SMI
help
--
2.8.3
More information about the dri-devel
mailing list