drm: fix DRM_DISPLAY_DP_HELPER dependencies
authorArnd Bergmann <arnd@arndb.de>
Thu, 4 Apr 2024 12:40:51 +0000 (14:40 +0200)
committerMaxime Ripard <mripard@kernel.org>
Thu, 4 Apr 2024 14:20:57 +0000 (16:20 +0200)
Both the exynos and rockchip drivers ran into link failures after
a Kconfig cleanup:

aarch64-linux-ld: drivers/gpu/drm/exynos/exynos_dp.o: in function `exynos_dp_resume':
exynos_dp.c:(.text+0xc0): undefined reference to `analogix_dp_resume'
aarch64-linux-ld: drivers/gpu/drm/exynos/exynos_dp.o: in function `exynos_dp_suspend':
exynos_dp.c:(.text+0xf4): undefined reference to `analogix_dp_suspend'

x86_64-linux-ld: drivers/gpu/drm/rockchip/cdn-dp-core.o: in function `cdn_dp_connector_mode_valid':
cdn-dp-core.c:(.text+0x13a): undefined reference to `drm_dp_bw_code_to_link_rate'
x86_64-linux-ld: cdn-dp-core.c:(.text+0x148): undefined reference to `drm_dp_bw_code_to_link_rate'
x86_64-linux-ld: drivers/gpu/drm/rockchip/cdn-dp-core.o: in function `cdn_dp_check_link_status':
cdn-dp-core.c:(.text+0x1396): undefined reference to `drm_dp_channel_eq_ok'

In both cases, the problem is that ROCKCHIP_CDN_DP and DRM_EXYNOS_DP
are 'bool' symbols that depend on the the 'tristate' DRM_DISPLAY_HELPER
symbol, but end up not working when the SoC specific part is built-in
but the helper is in a loadable module.

Use the same trick that DRM_ROCKCHIP already uses for the EXTCON
dependency and disallow DP support when it would not work.

Fixes: 0323287de87d ("drm: Switch DRM_DISPLAY_DP_HELPER to depends on")
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Link: https://patchwork.freedesktop.org/patch/msgid/20240404124101.2988099-1-arnd@kernel.org
Signed-off-by: Maxime Ripard <mripard@kernel.org>
drivers/gpu/drm/exynos/Kconfig
drivers/gpu/drm/rockchip/Kconfig

index 6a26a0b8eff2c021caa22b709bc4eebe57a2e76d..58cd772207413e94deef99b43f88a302a5489b81 100644 (file)
@@ -68,7 +68,7 @@ config DRM_EXYNOS_DP
        bool "Exynos specific extensions for Analogix DP driver"
        depends on DRM_EXYNOS_FIMD || DRM_EXYNOS7_DECON
        depends on DRM_DISPLAY_DP_HELPER
-       depends on DRM_DISPLAY_HELPER
+       depends on DRM_DISPLAY_HELPER=y || (DRM_DISPLAY_HELPER=m && DRM_EXYNOS=m)
        select DRM_ANALOGIX_DP
        default DRM_EXYNOS
        select DRM_PANEL
index 4b49a14758fe04128e0385ada0d5451d53344d9e..4b4ad75032fda17e87622d0b8f51f8450b4d71c5 100644 (file)
@@ -46,7 +46,7 @@ config ROCKCHIP_ANALOGIX_DP
 config ROCKCHIP_CDN_DP
        bool "Rockchip cdn DP"
        depends on DRM_DISPLAY_DP_HELPER
-       depends on DRM_DISPLAY_HELPER
+       depends on DRM_DISPLAY_HELPER=y || (DRM_DISPLAY_HELPER=m && DRM_ROCKCHIP=m)
        depends on EXTCON=y || (EXTCON=m && DRM_ROCKCHIP=m)
        help
          This selects support for Rockchip SoC specific extensions