]> git.ipfire.org Git - thirdparty/kernel/linux.git/commitdiff
media: tegra-vde: Add HAS_IOMEM dependency to match SRAM select
authorSasha Levin <sashal@kernel.org>
Sun, 26 Apr 2026 00:03:16 +0000 (20:03 -0400)
committerHans Verkuil <hverkuil+cisco@kernel.org>
Wed, 6 May 2026 07:05:56 +0000 (09:05 +0200)
kconfiglint reports:

  K002: config VIDEO_TEGRA_VDE selects visible symbol SRAM which has
        dependencies

VIDEO_TEGRA_VDE selects SRAM, which is defined in drivers/misc/Kconfig as:

  config SRAM
      bool "Generic on-chip SRAM driver"
      depends on HAS_IOMEM

The NVIDIA Tegra video decoder driver was originally introduced in
commit cd6c56feb591 ("media: staging: media: Introduce NVIDIA Tegra video
decoder driver") as a staging driver with
`depends on ARCH_TEGRA || COMPILE_TEST` and
`select SRAM`. Since all Tegra SoCs have HAS_IOMEM, the SRAM dependency was
implicitly satisfied for real hardware configurations.

The driver was later de-staged in commit 8bd4aaf438e3 ("media: staging:
tegra-vde: De-stage driver") and relocated to
drivers/media/platform/nvidia/tegra-vde/ in commit 9b18ef7c9ff4 ("media:
platform: rename tegra/vde/ to nvidia/tegra-vde/"). Throughout these moves,
the `select SRAM` remained without a corresponding HAS_IOMEM dependency.

Under COMPILE_TEST on a hypothetical architecture without HAS_IOMEM (such
as UML in some configurations), the select would force SRAM on without its
HAS_IOMEM dependency being met. Add an explicit `depends on HAS_IOMEM` to
make the dependency chain complete and prevent this misconfiguration under
COMPILE_TEST.

Assisted-by: Claude:claude-opus-4-6 kconfiglint
Signed-off-by: Sasha Levin <sashal@kernel.org>
Signed-off-by: Hans Verkuil <hverkuil+cisco@kernel.org>
drivers/media/platform/nvidia/tegra-vde/Kconfig

index 2fe13f39c95bbf826140a3ba156f6c22dd6f9a69..f05fcc94decafac4fe173478f289595a246cbb0f 100644 (file)
@@ -2,6 +2,7 @@ config VIDEO_TEGRA_VDE
        tristate "NVIDIA Tegra Video Decoder Engine driver"
        depends on V4L_MEM2MEM_DRIVERS
        depends on ARCH_TEGRA || COMPILE_TEST
+       depends on HAS_IOMEM
        depends on VIDEO_DEV
        select DMA_SHARED_BUFFER
        select IOMMU_IOVA