drm/radeon: Don't register backlight when another backlight should be used (v3)
authorHans de Goede <hdegoede@redhat.com>
Fri, 15 Apr 2022 20:24:21 +0000 (22:24 +0200)
committerHans de Goede <hdegoede@redhat.com>
Thu, 25 Aug 2022 08:56:20 +0000 (10:56 +0200)
commit1eb67781117ca2095f0dbede79f22f6b2ffa3189
tree32f05094738742ab6d8eb46c0b4ac46212a9f345
parentda11ef832972e311475fcba802398e720ed36c54
drm/radeon: Don't register backlight when another backlight should be used (v3)

Before this commit when we want userspace to use the acpi_video backlight
device we register both the GPU's native backlight device and acpi_video's
firmware acpi_video# backlight device. This relies on userspace preferring
firmware type backlight devices over native ones.

Registering 2 backlight devices for a single display really is
undesirable, don't register the GPU's native backlight device when
another backlight device should be used.

Changes in v2:
- To avoid linker errors when amdgpu is builtin and video_detect.c is in
  a module, select ACPI_VIDEO and its deps if ACPI is enabled.
  When ACPI is disabled, ACPI_VIDEO is also always disabled, ensuring
  the stubs from acpi/video.h will be used.

Changes in v3:
- Use drm_info(drm_dev, "...") to log messages
- ACPI_VIDEO can now be enabled on non X86 too,
  adjust the Kconfig changes to match this.

Acked-by: Alex Deucher <alexander.deucher@amd.com>
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
drivers/gpu/drm/Kconfig
drivers/gpu/drm/radeon/atombios_encoders.c
drivers/gpu/drm/radeon/radeon_legacy_encoders.c