Re: [PATCH v3 2/3] ACPI: video: Remove CONFIG_X86 handling from nvidia_wmi_ec_supported()
From: Armin Wolf
Date: Wed Sep 02 2026 - 13:31:14 EST
Am 01.09.26 um 23:30 schrieb Deucher, Alexander:
Public
-----Original Message-----The change below makes sense, but this one looks like it should remain unless X86_PLATFORM_DEVICES is handled some other way?
From: Armin Wolf <W_Armin@xxxxxx>
Sent: Thursday, July 23, 2026 11:00 PM
To: hansg@xxxxxxxxxx; ilpo.jarvinen@xxxxxxxxxxxxxxx
Cc: platform-driver-x86@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx;
linux@xxxxxxxxxxxxxx; rafael@xxxxxxxxxx; lenb@xxxxxxxxxx; linux-
acpi@xxxxxxxxxxxxxxx; Deucher, Alexander <Alexander.Deucher@xxxxxxx>;
Koenig, Christian <Christian.Koenig@xxxxxxx>; airlied@xxxxxxxxx;
simona@xxxxxxxx; amd-gfx@xxxxxxxxxxxxxxxxxxxxx; dri-
devel@xxxxxxxxxxxxxxxxxxxxx; patrik.r.jakobsson@xxxxxxxxx;
maarten.lankhorst@xxxxxxxxxxxxxxx; mripard@xxxxxxxxxx;
tzimmermann@xxxxxxx; jani.nikula@xxxxxxxxxxxxxxx;
joonas.lahtinen@xxxxxxxxxxxxxxx; rodrigo.vivi@xxxxxxxxx;
tursulin@xxxxxxxxxxx; intel-gfx@xxxxxxxxxxxxxxxxxxxxx;
matthew.brost@xxxxxxxxx; thomas.hellstrom@xxxxxxxxxxxxxxx; intel-
xe@xxxxxxxxxxxxxxxxxxxxx
Subject: [PATCH v3 2/3] ACPI: video: Remove CONFIG_X86 handling from
nvidia_wmi_ec_supported()
The WMI-ACPI driver now supports architectures other than x86.
Remove the CONFIG_X86 handling from video-detect and the associated
drivers to simplify the resulting code.
Acked-by: Rafael J. Wysocki (Intel) <rafael@xxxxxxxxxx>
Signed-off-by: Armin Wolf <W_Armin@xxxxxx>
---
drivers/acpi/Kconfig | 2 +-
drivers/acpi/video_detect.c | 8 --------
drivers/gpu/drm/amd/amdgpu/Kconfig | 4 +---
drivers/gpu/drm/gma500/Kconfig | 1 -
drivers/gpu/drm/i915/Kconfig | 1 -
drivers/gpu/drm/radeon/Kconfig | 4 +---
drivers/gpu/drm/xe/Kconfig | 5 ++---
drivers/platform/loongarch/Kconfig | 1 +
8 files changed, 6 insertions(+), 20 deletions(-)
diff --git a/drivers/acpi/Kconfig b/drivers/acpi/Kconfig index
f165d14cf61a..cc1e58fa664b 100644
--- a/drivers/acpi/Kconfig
+++ b/drivers/acpi/Kconfig
@@ -224,7 +224,7 @@ config ACPI_VIDEO
tristate "Video"
depends on BACKLIGHT_CLASS_DEVICE
depends on INPUT
- depends on ACPI_WMI || !X86
+ depends on ACPI_WMI
select THERMAL
help
This driver implements the ACPI Extensions For Display Adapters diff
--git a/drivers/acpi/video_detect.c b/drivers/acpi/video_detect.c index
458efa4fe9d4..af9280b244f1 100644
--- a/drivers/acpi/video_detect.c
+++ b/drivers/acpi/video_detect.c
@@ -82,8 +82,6 @@ find_video(acpi_handle handle, u32 lvl, void *context,
void **rv)
return AE_OK;
}
-/* This depends on ACPI_WMI which is X86 only */ -#ifdef CONFIG_X86
static bool nvidia_wmi_ec_supported(void) {
struct wmi_brightness_args args = {
@@ -105,12 +103,6 @@ static bool nvidia_wmi_ec_supported(void)
*/
return args.ret == WMI_BRIGHTNESS_SOURCE_EC; } -#else -static bool
nvidia_wmi_ec_supported(void) -{
- return false;
-}
-#endif
/* Force to use vendor driver when the ACPI device is known to be
* buggy */
diff --git a/drivers/gpu/drm/amd/amdgpu/Kconfig
b/drivers/gpu/drm/amd/amdgpu/Kconfig
index 12e4a41bf1f0..50305630f533 100644
--- a/drivers/gpu/drm/amd/amdgpu/Kconfig
+++ b/drivers/gpu/drm/amd/amdgpu/Kconfig
@@ -30,9 +30,7 @@ config DRM_AMDGPU
# ACPI_VIDEO's dependencies must also be selected.
select INPUT if ACPI
select ACPI_VIDEO if ACPI
- # On x86 ACPI_VIDEO also needs ACPI_WMI
- select X86_PLATFORM_DEVICES if ACPI && X86
Alex
AFAIK the dependency on X86_PLATFORM_DEVICES was only necessary because ACPI-WMI depended on it in the past.
By now ACPI-WMI neither depends on X86_PLATFORM_DEVICES or X86 anymore.
Thanks,
Armin Wolf
- select ACPI_WMI if ACPI && X86
+ select ACPI_WMI if ACPI
help
Choose this option if you have a recent AMD Radeon graphics card.
diff --git a/drivers/gpu/drm/gma500/Kconfig
b/drivers/gpu/drm/gma500/Kconfig index a2acaa699dd5..dc4c56638819
100644
--- a/drivers/gpu/drm/gma500/Kconfig
+++ b/drivers/gpu/drm/gma500/Kconfig
@@ -11,7 +11,6 @@ config DRM_GMA500
select ACPI_VIDEO if ACPI
select BACKLIGHT_CLASS_DEVICE if ACPI
select INPUT if ACPI
- select X86_PLATFORM_DEVICES if ACPI
select ACPI_WMI if ACPI
help
Say yes for an experimental 2D KMS framebuffer driver for the diff --
git a/drivers/gpu/drm/i915/Kconfig b/drivers/gpu/drm/i915/Kconfig index
5e939004b646..71426b9c91f0 100644
--- a/drivers/gpu/drm/i915/Kconfig
+++ b/drivers/gpu/drm/i915/Kconfig
@@ -28,7 +28,6 @@ config DRM_I915
# but for select to work, need to select ACPI_VIDEO's dependencies,
ick
select BACKLIGHT_CLASS_DEVICE if ACPI
select INPUT if ACPI
- select X86_PLATFORM_DEVICES if ACPI
select ACPI_WMI if ACPI
select ACPI_VIDEO if ACPI
select SYNC_FILE
diff --git a/drivers/gpu/drm/radeon/Kconfig
b/drivers/gpu/drm/radeon/Kconfig index c479f0c0dd5c..3cd70aef770e
100644
--- a/drivers/gpu/drm/radeon/Kconfig
+++ b/drivers/gpu/drm/radeon/Kconfig
@@ -25,9 +25,7 @@ config DRM_RADEON
# ACPI_VIDEO's dependencies must also be selected.
select INPUT if ACPI
select ACPI_VIDEO if ACPI
- # On x86 ACPI_VIDEO also needs ACPI_WMI
- select X86_PLATFORM_DEVICES if ACPI && X86
- select ACPI_WMI if ACPI && X86
+ select ACPI_WMI if ACPI
help
Choose this option if you have an ATI Radeon graphics card. There
are both PCI and AGP versions. You don't need to choose this to diff -
-git a/drivers/gpu/drm/xe/Kconfig b/drivers/gpu/drm/xe/Kconfig index
4d7dcaff2b91..502c7b3c1f16 100644
--- a/drivers/gpu/drm/xe/Kconfig
+++ b/drivers/gpu/drm/xe/Kconfig
@@ -4,7 +4,6 @@ config DRM_XE
depends on DRM && PCI
depends on KUNIT || !KUNIT
depends on INTEL_VSEC || !INTEL_VSEC
- depends on X86_PLATFORM_DEVICES || !(X86 && ACPI)
depends on PAGE_SIZE_4KB || COMPILE_TEST || BROKEN
select INTERVAL_TREE
# we need shmfs for the swappable backing store, and in particular
@@ -29,8 +28,8 @@ config DRM_XE
# but for select to work, need to select ACPI_VIDEO's dependencies,
ick
select BACKLIGHT_CLASS_DEVICE if ACPI
select INPUT if ACPI
- select ACPI_VIDEO if X86 && ACPI
- select ACPI_WMI if X86 && ACPI
+ select ACPI_VIDEO if ACPI
+ select ACPI_WMI if ACPI
select SYNC_FILE
select CRC32
select SND_HDA_I915 if SND_HDA_CORE
diff --git a/drivers/platform/loongarch/Kconfig
b/drivers/platform/loongarch/Kconfig
index 447528797d07..db7833842142 100644
--- a/drivers/platform/loongarch/Kconfig
+++ b/drivers/platform/loongarch/Kconfig
@@ -23,6 +23,7 @@ config LOONGSON_LAPTOP
depends on INPUT
depends on MACH_LOONGSON64
select ACPI_VIDEO
+ select ACPI_WMI
select INPUT_SPARSEKMAP
default y
help
--
2.39.5