The panel_power_savings sysfs entry sets the Adaptive Backlight Management level (abm_level). OLED displays work without backlight, so it is unnecessary for them. Before creating the sysfs entry, make sure the display is not an OLED display. Signed-off-by: Gergo Koteles <soyer@xxxxxx> --- drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c index d6e71aa808d8..d54065a76f63 100644 --- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c +++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c @@ -6535,9 +6535,11 @@ static const struct attribute_group amdgpu_group = { static void amdgpu_dm_connector_unregister(struct drm_connector *connector) { struct amdgpu_dm_connector *amdgpu_dm_connector = to_amdgpu_dm_connector(connector); + union dpcd_sink_ext_caps *ext_caps = + &amdgpu_dm_connector->dc_link->dpcd_sink_ext_caps; if (connector->connector_type == DRM_MODE_CONNECTOR_eDP && - amdgpu_dm_abm_level < 0) + amdgpu_dm_abm_level < 0 && !ext_caps->bits.oled) sysfs_remove_group(&connector->kdev->kobj, &amdgpu_group); drm_dp_aux_unregister(&amdgpu_dm_connector->dm_dp_aux.aux); @@ -6642,10 +6644,12 @@ amdgpu_dm_connector_late_register(struct drm_connector *connector) { struct amdgpu_dm_connector *amdgpu_dm_connector = to_amdgpu_dm_connector(connector); + union dpcd_sink_ext_caps *ext_caps = + &amdgpu_dm_connector->dc_link->dpcd_sink_ext_caps; int r; if (connector->connector_type == DRM_MODE_CONNECTOR_eDP && - amdgpu_dm_abm_level < 0) { + amdgpu_dm_abm_level < 0 && !ext_caps->bits.oled) { r = sysfs_create_group(&connector->kdev->kobj, &amdgpu_group); if (r) base-commit: dccb07f2914cdab2ac3a5b6c98406f765acab803 -- 2.45.0