Commit 9b5bf242 authored by Christoph Hellwig's avatar Christoph Hellwig Committed by Thomas Hellstrom

drm/vmwgfx: remove CONFIG_INTEL_IOMMU ifdefs v2

intel_iommu_enabled is defined as always false for !CONFIG_INTEL_IOMMU,
so remove the ifdefs around it.
Signed-off-by: default avatarChristoph Hellwig <hch@lst.de>
Reviewed-by: default avatarThomas Hellstrom <thellstrom@vmware.com>
Signed-off-by: default avatarThomas Hellstrom <thellstrom@vmware.com>
parent 3f47d00b
...@@ -567,12 +567,10 @@ static int vmw_dma_select_mode(struct vmw_private *dev_priv) ...@@ -567,12 +567,10 @@ static int vmw_dma_select_mode(struct vmw_private *dev_priv)
[vmw_dma_map_bind] = "Giving up DMA mappings early."}; [vmw_dma_map_bind] = "Giving up DMA mappings early."};
const struct dma_map_ops *dma_ops = get_dma_ops(dev_priv->dev->dev); const struct dma_map_ops *dma_ops = get_dma_ops(dev_priv->dev->dev);
#ifdef CONFIG_INTEL_IOMMU
if (intel_iommu_enabled) { if (intel_iommu_enabled) {
dev_priv->map_mode = vmw_dma_map_populate; dev_priv->map_mode = vmw_dma_map_populate;
goto out_fixup; goto out_fixup;
} }
#endif
if (!(vmw_force_iommu || vmw_force_coherent)) { if (!(vmw_force_iommu || vmw_force_coherent)) {
dev_priv->map_mode = vmw_dma_phys; dev_priv->map_mode = vmw_dma_phys;
...@@ -589,9 +587,7 @@ static int vmw_dma_select_mode(struct vmw_private *dev_priv) ...@@ -589,9 +587,7 @@ static int vmw_dma_select_mode(struct vmw_private *dev_priv)
dev_priv->map_mode = vmw_dma_map_populate; dev_priv->map_mode = vmw_dma_map_populate;
#endif #endif
#ifdef CONFIG_INTEL_IOMMU
out_fixup: out_fixup:
#endif
if (dev_priv->map_mode == vmw_dma_map_populate && if (dev_priv->map_mode == vmw_dma_map_populate &&
vmw_restrict_iommu) vmw_restrict_iommu)
dev_priv->map_mode = vmw_dma_map_bind; dev_priv->map_mode = vmw_dma_map_bind;
...@@ -599,13 +595,11 @@ static int vmw_dma_select_mode(struct vmw_private *dev_priv) ...@@ -599,13 +595,11 @@ static int vmw_dma_select_mode(struct vmw_private *dev_priv)
if (vmw_force_coherent) if (vmw_force_coherent)
dev_priv->map_mode = vmw_dma_alloc_coherent; dev_priv->map_mode = vmw_dma_alloc_coherent;
#if !defined(CONFIG_SWIOTLB) && !defined(CONFIG_INTEL_IOMMU) /* No TTM coherent page pool? FIXME: Ask TTM instead! */
/* if (!(IS_ENABLED(CONFIG_SWIOTLB) || IS_ENABLED(CONFIG_INTEL_IOMMU)) &&
* No coherent page pool (dev_priv->map_mode == vmw_dma_alloc_coherent))
*/
if (dev_priv->map_mode == vmw_dma_alloc_coherent)
return -EINVAL; return -EINVAL;
#endif
DRM_INFO("DMA map mode: %s\n", names[dev_priv->map_mode]); DRM_INFO("DMA map mode: %s\n", names[dev_priv->map_mode]);
return 0; return 0;
...@@ -619,7 +613,6 @@ static int vmw_dma_select_mode(struct vmw_private *dev_priv) ...@@ -619,7 +613,6 @@ static int vmw_dma_select_mode(struct vmw_private *dev_priv)
* With 32-bit we can only handle 32 bit PFNs. Optionally set that * With 32-bit we can only handle 32 bit PFNs. Optionally set that
* restriction also for 64-bit systems. * restriction also for 64-bit systems.
*/ */
#ifdef CONFIG_INTEL_IOMMU
static int vmw_dma_masks(struct vmw_private *dev_priv) static int vmw_dma_masks(struct vmw_private *dev_priv)
{ {
struct drm_device *dev = dev_priv->dev; struct drm_device *dev = dev_priv->dev;
...@@ -631,12 +624,6 @@ static int vmw_dma_masks(struct vmw_private *dev_priv) ...@@ -631,12 +624,6 @@ static int vmw_dma_masks(struct vmw_private *dev_priv)
} }
return 0; return 0;
} }
#else
static int vmw_dma_masks(struct vmw_private *dev_priv)
{
return 0;
}
#endif
static int vmw_driver_load(struct drm_device *dev, unsigned long chipset) static int vmw_driver_load(struct drm_device *dev, unsigned long chipset)
{ {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment