On Tue Aug 16 14:49:05 2022 +0100, Robin Murphy wrote:
> Depending on IOMMU_DMA is suspicious, since the low-level DMA ops
> internals and MSI helpers is provides are not of interest to general
> drivers. In practice it mostly seems to be serving as a dependency on
> ARM64 here, so do that properly. The venus driver looks like it won't
> actually be useful at runtime without the IOMMU API, so add that too.
> 
> Signed-off-by: Robin Murphy <robin.mur...@arm.com>
> Reviewed-by: Robert Foss <robert.f...@linaro.org>
> Signed-off-by: Hans Verkuil <hverk...@xs4all.nl>

Patch committed.

Thanks,
Hans Verkuil

 drivers/media/platform/qcom/venus/Kconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

---

diff --git a/drivers/media/platform/qcom/venus/Kconfig 
b/drivers/media/platform/qcom/venus/Kconfig
index bc2e410b29cb..ffb731ecd48c 100644
--- a/drivers/media/platform/qcom/venus/Kconfig
+++ b/drivers/media/platform/qcom/venus/Kconfig
@@ -2,7 +2,7 @@ config VIDEO_QCOM_VENUS
        tristate "Qualcomm Venus V4L2 encoder/decoder driver"
        depends on V4L_MEM2MEM_DRIVERS
        depends on VIDEO_DEV && QCOM_SMEM
-       depends on (ARCH_QCOM && IOMMU_DMA) || COMPILE_TEST
+       depends on (ARCH_QCOM && ARM64 && IOMMU_API) || COMPILE_TEST
        select OF_DYNAMIC if ARCH_QCOM
        select QCOM_MDT_LOADER if ARCH_QCOM
        select QCOM_SCM

Reply via email to