Revert "drm: Switch DRM_DISPLAY_HELPER to depends on"
authorGeert Uytterhoeven <geert+renesas@glider.be>
Mon, 22 Apr 2024 10:30:37 +0000 (12:30 +0200)
committerMaxime Ripard <mripard@kernel.org>
Thu, 2 May 2024 15:58:23 +0000 (17:58 +0200)
This reverts commit e075e496f516bf92bc0cbaf94d64e8d4a6b58321, as helper
code should always be selected by the driver that needs it, for the
convenience of the final user configuring a kernel.

The user who configures a kernel should not need to know which helpers
are needed for the driver he is interested in.  Making a driver depend
on helper code means that the user needs to know which helpers to enable
first, which is very user-unfriendly.

Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
Acked-by: Arnd Bergmann <arnd@arndb.de>
Link: https://patchwork.freedesktop.org/patch/msgid/1ba76cc4d96a8afefff5d1bc42fb1e1329c5da68.1713780345.git.geert+renesas@glider.be
Signed-off-by: Maxime Ripard <mripard@kernel.org>
19 files changed:
drivers/gpu/drm/Kconfig
drivers/gpu/drm/amd/amdgpu/Kconfig
drivers/gpu/drm/bridge/Kconfig
drivers/gpu/drm/bridge/analogix/Kconfig
drivers/gpu/drm/bridge/cadence/Kconfig
drivers/gpu/drm/bridge/synopsys/Kconfig
drivers/gpu/drm/display/Kconfig
drivers/gpu/drm/exynos/Kconfig
drivers/gpu/drm/i915/Kconfig
drivers/gpu/drm/mediatek/Kconfig
drivers/gpu/drm/msm/Kconfig
drivers/gpu/drm/nouveau/Kconfig
drivers/gpu/drm/panel/Kconfig
drivers/gpu/drm/radeon/Kconfig
drivers/gpu/drm/rockchip/Kconfig
drivers/gpu/drm/tegra/Kconfig
drivers/gpu/drm/vc4/Kconfig
drivers/gpu/drm/xe/Kconfig
drivers/gpu/drm/xlnx/Kconfig

index 33792ca..bf40209 100644 (file)
@@ -74,12 +74,10 @@ config DRM_KUNIT_TEST_HELPERS
 
 config DRM_KUNIT_TEST
        tristate "KUnit tests for DRM" if !KUNIT_ALL_TESTS
-       depends on DRM
-       depends on DRM_DISPLAY_HELPER
-       depends on KUNIT
-       depends on MMU
+       depends on DRM && KUNIT && MMU
        select DRM_BUDDY
        select DRM_DISPLAY_DP_HELPER
+       select DRM_DISPLAY_HELPER
        select DRM_EXEC
        select DRM_EXPORT_FOR_TESTS if m
        select DRM_GEM_SHMEM_HELPER
index cf931b9..22d88f8 100644 (file)
@@ -2,15 +2,13 @@
 
 config DRM_AMDGPU
        tristate "AMD GPU"
-       depends on DRM
-       depends on DRM_DISPLAY_HELPER
-       depends on MMU
-       depends on PCI
+       depends on DRM && PCI && MMU
        depends on !UML
        select FW_LOADER
        select DRM_DISPLAY_DP_HELPER
        select DRM_DISPLAY_HDMI_HELPER
        select DRM_DISPLAY_HDCP_HELPER
+       select DRM_DISPLAY_HELPER
        select DRM_KMS_HELPER
        select DRM_SCHED
        select DRM_TTM
index 8c0883b..c621be1 100644 (file)
@@ -92,10 +92,10 @@ config DRM_FSL_LDB
 
 config DRM_ITE_IT6505
        tristate "ITE IT6505 DisplayPort bridge"
-       depends on DRM_DISPLAY_HELPER
        depends on OF
        select DRM_DISPLAY_DP_HELPER
        select DRM_DISPLAY_HDCP_HELPER
+       select DRM_DISPLAY_HELPER
        select DRM_DISPLAY_DP_AUX_BUS
        select DRM_KMS_HELPER
        select EXTCON
@@ -232,9 +232,9 @@ config DRM_PARADE_PS8622
 
 config DRM_PARADE_PS8640
        tristate "Parade PS8640 MIPI DSI to eDP Converter"
-       depends on DRM_DISPLAY_HELPER
        depends on OF
        select DRM_DISPLAY_DP_HELPER
+       select DRM_DISPLAY_HELPER
        select DRM_DISPLAY_DP_AUX_BUS
        select DRM_KMS_HELPER
        select DRM_MIPI_DSI
@@ -319,9 +319,9 @@ config DRM_TOSHIBA_TC358764
 
 config DRM_TOSHIBA_TC358767
        tristate "Toshiba TC358767 eDP bridge"
-       depends on DRM_DISPLAY_HELPER
        depends on OF
        select DRM_DISPLAY_DP_HELPER
+       select DRM_DISPLAY_HELPER
        select DRM_KMS_HELPER
        select REGMAP_I2C
        select DRM_MIPI_DSI
@@ -342,9 +342,9 @@ config DRM_TOSHIBA_TC358768
 
 config DRM_TOSHIBA_TC358775
        tristate "Toshiba TC358775 DSI/LVDS bridge"
-       depends on DRM_DISPLAY_HELPER
        depends on OF
        select DRM_DISPLAY_DP_HELPER
+       select DRM_DISPLAY_HELPER
        select DRM_KMS_HELPER
        select REGMAP_I2C
        select DRM_PANEL
@@ -387,9 +387,9 @@ config DRM_TI_SN65DSI83
 
 config DRM_TI_SN65DSI86
        tristate "TI SN65DSI86 DSI to eDP bridge"
-       depends on DRM_DISPLAY_HELPER
        depends on OF
        select DRM_DISPLAY_DP_HELPER
+       select DRM_DISPLAY_HELPER
        select DRM_KMS_HELPER
        select REGMAP_I2C
        select DRM_PANEL
index 16d18dd..4846b2e 100644 (file)
@@ -1,10 +1,10 @@
 # SPDX-License-Identifier: GPL-2.0-only
 config DRM_ANALOGIX_ANX6345
        tristate "Analogix ANX6345 bridge"
-       depends on DRM_DISPLAY_HELPER
        depends on OF
        select DRM_ANALOGIX_DP
        select DRM_DISPLAY_DP_HELPER
+       select DRM_DISPLAY_HELPER
        select DRM_KMS_HELPER
        select REGMAP_I2C
        help
@@ -15,9 +15,9 @@ config DRM_ANALOGIX_ANX6345
 
 config DRM_ANALOGIX_ANX78XX
        tristate "Analogix ANX78XX bridge"
-       depends on DRM_DISPLAY_HELPER
        select DRM_ANALOGIX_DP
        select DRM_DISPLAY_DP_HELPER
+       select DRM_DISPLAY_HELPER
        select DRM_KMS_HELPER
        select REGMAP_I2C
        help
@@ -33,10 +33,10 @@ config DRM_ANALOGIX_DP
 config DRM_ANALOGIX_ANX7625
        tristate "Analogix Anx7625 MIPI to DP interface support"
        depends on DRM
-       depends on DRM_DISPLAY_HELPER
        depends on OF
        select DRM_DISPLAY_DP_HELPER
        select DRM_DISPLAY_HDCP_HELPER
+       select DRM_DISPLAY_HELPER
        select DRM_DISPLAY_DP_AUX_BUS
        select DRM_MIPI_DSI
        help
index 20143af..cced816 100644 (file)
@@ -23,12 +23,12 @@ endif
 
 config DRM_CDNS_MHDP8546
        tristate "Cadence DPI/DP bridge"
-       depends on DRM_DISPLAY_HELPER
-       depends on OF
        select DRM_DISPLAY_DP_HELPER
        select DRM_DISPLAY_HDCP_HELPER
+       select DRM_DISPLAY_HELPER
        select DRM_KMS_HELPER
        select DRM_PANEL_BRIDGE
+       depends on OF
        help
          Support Cadence DPI to DP bridge. This is an internal
          bridge and is meant to be directly embedded in a SoC.
index f366ece..15fc182 100644 (file)
@@ -1,8 +1,8 @@
 # SPDX-License-Identifier: GPL-2.0-only
 config DRM_DW_HDMI
        tristate
-       depends on DRM_DISPLAY_HELPER
        select DRM_DISPLAY_HDMI_HELPER
+       select DRM_DISPLAY_HELPER
        select DRM_KMS_HELPER
        select REGMAP_MMIO
        select CEC_CORE if CEC_NOTIFIER
index cffa2ac..c77e7f8 100644 (file)
@@ -3,7 +3,6 @@
 config DRM_DISPLAY_HELPER
        tristate "DRM Display Helpers"
        depends on DRM
-       default y
        help
          DRM helpers for display adapters.
 
index 4b0183b..733b109 100644 (file)
@@ -4,6 +4,7 @@ config DRM_EXYNOS
        depends on OF && DRM && COMMON_CLK
        depends on ARCH_S3C64XX || ARCH_S5PV210 || ARCH_EXYNOS || COMPILE_TEST
        depends on MMU
+       select DRM_DISPLAY_HELPER if DRM_EXYNOS_DP
        select DRM_KMS_HELPER
        select VIDEOMODE_HELPERS
        select FB_DMAMEM_HELPERS if DRM_FBDEV_EMULATION
@@ -67,7 +68,6 @@ config DRM_EXYNOS_DSI
 config DRM_EXYNOS_DP
        bool "Exynos specific extensions for Analogix DP driver"
        depends on DRM_EXYNOS_FIMD || DRM_EXYNOS7_DECON
-       depends on DRM_DISPLAY_HELPER
        select DRM_ANALOGIX_DP
        select DRM_DISPLAY_DP_HELPER
        default DRM_EXYNOS
index 43183a6..5932024 100644 (file)
@@ -2,7 +2,6 @@
 config DRM_I915
        tristate "Intel 8xx/9xx/G3x/G4x/HD Graphics"
        depends on DRM
-       depends on DRM_DISPLAY_HELPER
        depends on X86 && PCI
        depends on !PREEMPT_RT
        select INTEL_GTT if X86
@@ -14,6 +13,7 @@ config DRM_I915
        select DRM_DISPLAY_DP_HELPER
        select DRM_DISPLAY_HDCP_HELPER
        select DRM_DISPLAY_HDMI_HELPER
+       select DRM_DISPLAY_HELPER
        select DRM_KMS_HELPER
        select DRM_PANEL
        select DRM_MIPI_DSI
index 50bb283..96cbe02 100644 (file)
@@ -22,9 +22,9 @@ config DRM_MEDIATEK
 
 config DRM_MEDIATEK_DP
        tristate "DRM DPTX Support for MediaTek SoCs"
-       depends on DRM_DISPLAY_HELPER
        depends on DRM_MEDIATEK
        select PHY_MTK_DP
+       select DRM_DISPLAY_HELPER
        select DRM_DISPLAY_DP_HELPER
        select DRM_DISPLAY_DP_AUX_BUS
        help
index 2055266..27d72ed 100644 (file)
@@ -2,10 +2,9 @@
 
 config DRM_MSM
        tristate "MSM DRM"
+       depends on DRM
        depends on ARCH_QCOM || SOC_IMX5 || COMPILE_TEST
        depends on COMMON_CLK
-       depends on DRM
-       depends on DRM_DISPLAY_HELPER
        depends on IOMMU_SUPPORT
        depends on QCOM_AOSS_QMP || QCOM_AOSS_QMP=n
        depends on QCOM_OCMEM || QCOM_OCMEM=n
@@ -17,6 +16,7 @@ config DRM_MSM
        select REGULATOR
        select DRM_DISPLAY_DP_AUX_BUS
        select DRM_DISPLAY_DP_HELPER
+       select DRM_DISPLAY_HELPER
        select DRM_EXEC
        select DRM_KMS_HELPER
        select DRM_PANEL
index 5ac852b..ceef470 100644 (file)
@@ -1,14 +1,12 @@
 # SPDX-License-Identifier: GPL-2.0-only
 config DRM_NOUVEAU
        tristate "Nouveau (NVIDIA) cards"
-       depends on DRM
-       depends on DRM_DISPLAY_HELPER
-       depends on PCI
-       depends on MMU
+       depends on DRM && PCI && MMU
        select IOMMU_API
        select FW_LOADER
        select DRM_DISPLAY_DP_HELPER
        select DRM_DISPLAY_HDMI_HELPER
+       select DRM_DISPLAY_HELPER
        select DRM_KMS_HELPER
        select DRM_TTM
        select DRM_TTM_HELPER
index 108ced2..a979e9b 100644 (file)
@@ -544,11 +544,11 @@ config DRM_PANEL_RAYDIUM_RM68200
 
 config DRM_PANEL_RAYDIUM_RM692E5
        tristate "Raydium RM692E5-based DSI panel"
-       depends on BACKLIGHT_CLASS_DEVICE
-       depends on DRM_DISPLAY_HELPER
-       depends on DRM_MIPI_DSI
        depends on OF
+       depends on DRM_MIPI_DSI
+       depends on BACKLIGHT_CLASS_DEVICE
        select DRM_DISPLAY_DP_HELPER
+       select DRM_DISPLAY_HELPER
        help
          Say Y here if you want to enable support for Raydium RM692E5-based
          display panels, such as the one found in the Fairphone 5 smartphone.
@@ -582,11 +582,11 @@ config DRM_PANEL_SAMSUNG_S6E88A0_AMS452EF01
 
 config DRM_PANEL_SAMSUNG_ATNA33XC20
        tristate "Samsung ATNA33XC20 eDP panel"
-       depends on BACKLIGHT_CLASS_DEVICE
-       depends on DRM_DISPLAY_HELPER
        depends on OF
+       depends on BACKLIGHT_CLASS_DEVICE
        depends on PM
        select DRM_DISPLAY_DP_HELPER
+       select DRM_DISPLAY_HELPER
        select DRM_DISPLAY_DP_AUX_BUS
        help
          DRM panel driver for the Samsung ATNA33XC20 panel. This panel can't
@@ -822,12 +822,12 @@ config DRM_PANEL_STARTEK_KD070FHFID015
 
 config DRM_PANEL_EDP
        tristate "support for simple Embedded DisplayPort panels"
-       depends on BACKLIGHT_CLASS_DEVICE
-       depends on DRM_DISPLAY_HELPER
        depends on OF
+       depends on BACKLIGHT_CLASS_DEVICE
        depends on PM
        select VIDEOMODE_HELPERS
        select DRM_DISPLAY_DP_HELPER
+       select DRM_DISPLAY_HELPER
        select DRM_DISPLAY_DP_AUX_BUS
        select DRM_KMS_HELPER
        help
@@ -902,11 +902,11 @@ config DRM_PANEL_TRULY_NT35597_WQXGA
 
 config DRM_PANEL_VISIONOX_R66451
        tristate "Visionox R66451"
-       depends on BACKLIGHT_CLASS_DEVICE
-       depends on DRM_DISPLAY_HELPER
-       depends on DRM_MIPI_DSI
        depends on OF
+       depends on DRM_MIPI_DSI
+       depends on BACKLIGHT_CLASS_DEVICE
        select DRM_DISPLAY_DP_HELPER
+       select DRM_DISPLAY_HELPER
        help
          Say Y here if you want to enable support for Visionox
          R66451 1080x2340 AMOLED DSI panel.
index 07d3304..f98356b 100644 (file)
@@ -2,13 +2,11 @@
 
 config DRM_RADEON
        tristate "ATI Radeon"
+       depends on DRM && PCI && MMU
        depends on AGP || !AGP
-       depends on DRM
-       depends on DRM_DISPLAY_HELPER
-       depends on PCI
-       depends on MMU
        select FW_LOADER
        select DRM_DISPLAY_DP_HELPER
+       select DRM_DISPLAY_HELPER
         select DRM_KMS_HELPER
        select DRM_SUBALLOC_HELPER
         select DRM_TTM
index b72c0bb..0d5260e 100644 (file)
@@ -35,8 +35,8 @@ config ROCKCHIP_VOP2
 
 config ROCKCHIP_ANALOGIX_DP
        bool "Rockchip specific extensions for Analogix DP driver"
-       depends on DRM_DISPLAY_HELPER
        depends on ROCKCHIP_VOP
+       select DRM_DISPLAY_HELPER
        select DRM_DISPLAY_DP_HELPER
        help
          This selects support for Rockchip SoC specific extensions
@@ -45,8 +45,8 @@ config ROCKCHIP_ANALOGIX_DP
 
 config ROCKCHIP_CDN_DP
        bool "Rockchip cdn DP"
-       depends on DRM_DISPLAY_HELPER
        depends on EXTCON=y || (EXTCON=m && DRM_ROCKCHIP=m)
+       select DRM_DISPLAY_HELPER
        select DRM_DISPLAY_DP_HELPER
        help
          This selects support for Rockchip SoC specific extensions
index 44381ee..782f51d 100644 (file)
@@ -4,10 +4,10 @@ config DRM_TEGRA
        depends on ARCH_TEGRA || COMPILE_TEST
        depends on COMMON_CLK
        depends on DRM
-       depends on DRM_DISPLAY_HELPER
        depends on OF
        select DRM_DISPLAY_DP_HELPER
        select DRM_DISPLAY_HDMI_HELPER
+       select DRM_DISPLAY_HELPER
        select DRM_DISPLAY_DP_AUX_BUS
        select DRM_KMS_HELPER
        select DRM_MIPI_DSI
index 98772a6..91dcf8d 100644 (file)
@@ -2,15 +2,15 @@
 config DRM_VC4
        tristate "Broadcom VC4 Graphics"
        depends on ARCH_BCM || ARCH_BCM2835 || COMPILE_TEST
-       depends on COMMON_CLK
-       depends on DRM
-       depends on DRM_DISPLAY_HELPER
-       depends on PM
        # Make sure not 'y' when RASPBERRYPI_FIRMWARE is 'm'. This can only
        # happen when COMPILE_TEST=y, hence the added !RASPBERRYPI_FIRMWARE.
        depends on RASPBERRYPI_FIRMWARE || (COMPILE_TEST && !RASPBERRYPI_FIRMWARE)
+       depends on DRM
        depends on SND && SND_SOC
+       depends on COMMON_CLK
+       depends on PM
        select DRM_DISPLAY_HDMI_HELPER
+       select DRM_DISPLAY_HELPER
        select DRM_KMS_HELPER
        select DRM_GEM_DMA_HELPER
        select DRM_PANEL_BRIDGE
index 4407e14..63f1e2d 100644 (file)
@@ -1,11 +1,7 @@
 # SPDX-License-Identifier: GPL-2.0-only
 config DRM_XE
        tristate "Intel Xe Graphics"
-       depends on (m || (y && KUNIT=y))
-       depends on DRM
-       depends on DRM_DISPLAY_HELPER
-       depends on MMU
-       depends on PCI
+       depends on DRM && PCI && MMU && (m || (y && KUNIT=y))
        select INTERVAL_TREE
        # we need shmfs for the swappable backing store, and in particular
        # the shmem_readpage() which depends upon tmpfs
@@ -20,6 +16,7 @@ config DRM_XE
        select DRM_DISPLAY_DP_HELPER
        select DRM_DISPLAY_HDCP_HELPER
        select DRM_DISPLAY_HDMI_HELPER
+       select DRM_DISPLAY_HELPER
        select DRM_MIPI_DSI
        select RELAY
        select IRQ_WORK
index 7a14a8c..68ee897 100644 (file)
@@ -1,15 +1,13 @@
 config DRM_ZYNQMP_DPSUB
        tristate "ZynqMP DisplayPort Controller Driver"
        depends on ARCH_ZYNQMP || COMPILE_TEST
-       depends on COMMON_CLK
+       depends on COMMON_CLK && DRM && OF
        depends on DMADEVICES
-       depends on DRM
-       depends on DRM_DISPLAY_HELPER
-       depends on OF
        depends on PHY_XILINX_ZYNQMP
        depends on XILINX_ZYNQMP_DPDMA
        select DMA_ENGINE
        select DRM_DISPLAY_DP_HELPER
+       select DRM_DISPLAY_HELPER
        select DRM_GEM_DMA_HELPER
        select DRM_KMS_HELPER
        select GENERIC_PHY