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 33792ca3eeb7ae8de9cef00924faf4596e400017..bf4020915e299861846a03053589fb398614a737 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 cf931b94a1889746277d744d7f7266d66d22612d..22d88f8ef5279a0f8ab1b1fc0f318a749b06ff46 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 8c0883bc3f2aed2c04d7409c3cb6011315a1d3b6..c621be1a99a89c441f75ace64b113c1da1c19df3 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 16d18dde483ae9c43905d2f7b4f49f80400251be..4846b2e9be7c2a5da18f6a3cdec53ef5766455e0 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 20143afded40e4378cc3b4a13080a9792dad8efb..cced81633ddcda03d89946e10684727887b3e792 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 f366ece471462a7061f32049a058b09c38236dd9..15fc182d05ef02e65541542610964ae028687491 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 cffa2acdbc6c0988114857783b1b578b571e60cc..c77e7f85bd674dc91fca9fc93e2bedc7467a9022 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 4b0183bf221c8bb26caaa7c0639d03ca5e87b5b6..733b109a509525f7c17bd69fe2e534073cb796e8 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 43183a68a09557b711cba134df2cee471e45ffc6..5932024f8f9547e591b3c022a057d298fbcb5f36 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 50bb28327f65fbf53ea03ec6b9e0c3766b7218fd..96cbe020f493ab1915b446277429d6a95150e18c 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 2055266506e5adf0d6867370fffd89178e364274..27d72ed8b3896b64761cf15ed0a70cef04d0c468 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 5ac852b816db262bc4a10c23c2decb4461f822a9..ceef470c9fbfcfb08be6abd69627b7e7bc66366d 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 108ced2e8794118a4f2e99c99a95137e8cd8d69f..a979e9bb0905cd4c691f9d01593846466d6a88c8 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 07d330450f05f899920250688b4b91561699ba60..f98356be0af288f5ebf5ec3d50138c7cbc1a385a 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 b72c0bbf346da5fa8c0839e3590fc4dd6aaa761e..0d5260e10f272d3b72b5052ea83d5efc01d9e75c 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 44381ee6ea9e36f755f67b2ea64255e5a28d1055..782f51d3044af1fc84af3ea581be438f55764782 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 98772a6b5bf0df54738db2e63f156bebab03c0a4..91dcf8d174d6c6e1e98fcb5f0932e188db11813f 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 4407e14338fce9c8f9a2324551af7495cdc5e9f3..63f1e2d1649fc9032006dd92166dbdb79ede6c97 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 7a14a8c2e7be9e597ab66c58cde72e2aa2da2eb6..68ee897de9d758848fe1632d44a3fba566f97a13 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