On Fri, Dec 9, 2022, at 23:05, Uwe Kleine-König wrote:
> Some of these dependencies used to be sensible when only a small part of
> the platforms supported by ARCH=arm could be compiled together in a
> single kernel image. Nowadays ARCH_MULTIPLATFORM is only used as a guard
> for kernel options incompatible with a multiplatform image. See commit
> 84fc86360623 ("ARM: make ARCH_MULTIPLATFORM user-visible") for some more
> details.
>
> Signed-off-by: Uwe Kleine-König <u.kleine-koe...@pengutronix.de>

Makes sense,

Acked-by: Arnd Bergmann <a...@arndb.de>

> diff --git a/drivers/gpu/drm/omapdrm/Kconfig 
> b/drivers/gpu/drm/omapdrm/Kconfig
> index 455e1a91f0e5..76ded1568bd0 100644
> --- a/drivers/gpu/drm/omapdrm/Kconfig
> +++ b/drivers/gpu/drm/omapdrm/Kconfig
> @@ -2,7 +2,7 @@
>  config DRM_OMAP
>       tristate "OMAP DRM"
>       depends on DRM && OF
> -     depends on ARCH_OMAP2PLUS || ARCH_MULTIPLATFORM
> +     depends on ARCH_OMAP2PLUS
>       select DRM_KMS_HELPER
>       select VIDEOMODE_HELPERS
>       select HDMI

Since the original purpose of the ||ARCH_MULTIPLATFORM was to allow
building the driver on more targets, I wonder if we should instead
make that ||COMPILE_TEST, which would also allow building it on
x86 and others.

> diff --git a/drivers/gpu/drm/sti/Kconfig b/drivers/gpu/drm/sti/Kconfig
> index f2a880c48485..3c7a5feff8de 100644
> --- a/drivers/gpu/drm/sti/Kconfig
> +++ b/drivers/gpu/drm/sti/Kconfig
> @@ -1,7 +1,7 @@
>  # SPDX-License-Identifier: GPL-2.0-only
>  config DRM_STI
>       tristate "DRM Support for STMicroelectronics SoC stiH4xx Series"
> -     depends on OF && DRM && (ARCH_STI || ARCH_MULTIPLATFORM)
> +     depends on OF && DRM && ARCH_STI
>       select RESET_CONTROLLER
>       select DRM_KMS_HELPER
>       select DRM_GEM_DMA_HELPER
> diff --git a/drivers/gpu/drm/stm/Kconfig b/drivers/gpu/drm/stm/Kconfig
> index ded72f879482..fa49cde43bb2 100644
> --- a/drivers/gpu/drm/stm/Kconfig
> +++ b/drivers/gpu/drm/stm/Kconfig
> @@ -1,7 +1,7 @@
>  # SPDX-License-Identifier: GPL-2.0-only
>  config DRM_STM
>       tristate "DRM Support for STMicroelectronics SoC Series"
> -     depends on DRM && (ARCH_STM32 || ARCH_MULTIPLATFORM)
> +     depends on DRM && ARCH_STM32
>       select DRM_KMS_HELPER
>       select DRM_GEM_DMA_HELPER
>       select DRM_PANEL_BRIDGE

Same here.

     arnd

Reply via email to