* [PATCH] drm/arm: Suppress "ARM devices" menu if not essential
@ 2025-03-10 7:23 WangYuli
2025-03-10 10:52 ` Liviu Dudau
0 siblings, 1 reply; 2+ messages in thread
From: WangYuli @ 2025-03-10 7:23 UTC (permalink / raw)
To: liviu.dudau, maarten.lankhorst, mripard, tzimmermann, airlied,
simona
Cc: dri-devel, linux-kernel, zhanjun, niecheng1, guanwentao,
chenlinxuan, WangYuli
Manifestly, the "ARM devices" menu is appropriate only for the ARM
architecture.
Inasmuch as every single item in this menu is predicated on
(ARM || ARM64 || COMPILE_TEST), it is therefore illogical for the
"ARM devices" menu itself not to be predicated upon
(ARM || ARM64 || COMPILE_TEST), and displaying a blank menu in other
architectures is unwarranted.
Signed-off-by: WangYuli <wangyuli@uniontech.com>
---
drivers/gpu/drm/arm/Kconfig | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/arm/Kconfig b/drivers/gpu/drm/arm/Kconfig
index c901ac00c0c3..0aa6a54567b6 100644
--- a/drivers/gpu/drm/arm/Kconfig
+++ b/drivers/gpu/drm/arm/Kconfig
@@ -1,6 +1,6 @@
# SPDX-License-Identifier: GPL-2.0
menu "ARM devices"
- depends on DRM
+ depends on DRM && (ARM || ARM64 || COMPILE_TEST)
config DRM_HDLCD
tristate "ARM HDLCD"
--
2.47.2
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] drm/arm: Suppress "ARM devices" menu if not essential
2025-03-10 7:23 [PATCH] drm/arm: Suppress "ARM devices" menu if not essential WangYuli
@ 2025-03-10 10:52 ` Liviu Dudau
0 siblings, 0 replies; 2+ messages in thread
From: Liviu Dudau @ 2025-03-10 10:52 UTC (permalink / raw)
To: WangYuli
Cc: maarten.lankhorst, mripard, tzimmermann, airlied, simona,
dri-devel, linux-kernel, zhanjun, niecheng1, guanwentao,
chenlinxuan
On Mon, Mar 10, 2025 at 03:23:57PM +0800, WangYuli wrote:
> Manifestly, the "ARM devices" menu is appropriate only for the ARM
> architecture.
>
> Inasmuch as every single item in this menu is predicated on
> (ARM || ARM64 || COMPILE_TEST), it is therefore illogical for the
> "ARM devices" menu itself not to be predicated upon
> (ARM || ARM64 || COMPILE_TEST), and displaying a blank menu in other
> architectures is unwarranted.
Arm's Display IP is not tied to the Arm architecture. Anyone can license
and use the IP with any CPU architecture. The fact that so far the IP has
only been present in a handful of devices doesn't mean we should restrict
the drivers.
What is the issue that you're trying to solve here?
Best regards,
Liviu
>
> Signed-off-by: WangYuli <wangyuli@uniontech.com>
> ---
> drivers/gpu/drm/arm/Kconfig | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/arm/Kconfig b/drivers/gpu/drm/arm/Kconfig
> index c901ac00c0c3..0aa6a54567b6 100644
> --- a/drivers/gpu/drm/arm/Kconfig
> +++ b/drivers/gpu/drm/arm/Kconfig
> @@ -1,6 +1,6 @@
> # SPDX-License-Identifier: GPL-2.0
> menu "ARM devices"
> - depends on DRM
> + depends on DRM && (ARM || ARM64 || COMPILE_TEST)
>
> config DRM_HDLCD
> tristate "ARM HDLCD"
> --
> 2.47.2
>
--
====================
| I would like to |
| fix the world, |
| but they're not |
| giving me the |
\ source code! /
---------------
¯\_(ツ)_/¯
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2025-03-10 10:52 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-03-10 7:23 [PATCH] drm/arm: Suppress "ARM devices" menu if not essential WangYuli
2025-03-10 10:52 ` Liviu Dudau
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox