drm/amdgpu: Don't register backlight when another backlight should be used (v3)
authorHans de Goede <hdegoede@redhat.com>
Fri, 15 Apr 2022 20:22:41 +0000 (22:22 +0200)
committerHans de Goede <hdegoede@redhat.com>
Thu, 25 Aug 2022 08:56:20 +0000 (10:56 +0200)
commitda11ef832972e311475fcba802398e720ed36c54
tree8f8b50e7da9ffae0f8843f6d40c64b8a2fbdf0e3
parentb1d36e73cc1c68d28f63cbd14767d0d0861147e0
drm/amdgpu: 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/amd/amdgpu/atombios_encoders.c
drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c