drm/radeon: drop confusing message about backlight control
authorAlex Deucher <alexander.deucher@amd.com>
Wed, 27 Jul 2016 20:39:19 +0000 (16:39 -0400)
committerAlex Deucher <alexander.deucher@amd.com>
Fri, 29 Jul 2016 18:36:45 +0000 (14:36 -0400)
The message is harmless and confusing.  On PX systems,
there is one ATIF method, but potentially multiple GPUs
leading to an error on the GPU with no backlight control.

Bug: https://bugzilla.kernel.org/show_bug.cgi?id=115011
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
Cc: Hans de Goede <jwrdegoede@fedoraproject.org>
drivers/gpu/drm/radeon/radeon_acpi.c

index 59acd0e5c2c6384705fdda0a6b89e65be5f5abc4..31c9a92d6a1b89da90863e761fa4701f15d3ad3f 100644 (file)
@@ -741,13 +741,6 @@ int radeon_acpi_init(struct radeon_device *rdev)
                }
 
                atif->encoder_for_bl = target;
-               if (!target) {
-                       /* Brightness change notification is enabled, but we
-                        * didn't find a backlight controller, this should
-                        * never happen.
-                        */
-                       DRM_ERROR("Cannot find a backlight controller\n");
-               }
        }
 
        if (atif->functions.sbios_requests && !atif->functions.system_params) {