Skip to content

Commit f2ad3ac

Browse files
committed
drm/amdgpu/display: only enable aux backlight control for OLED panels
We've gotten a number of reports about backlight control not working on panels which indicate that they use aux backlight control. A recent patch: commit 2d73eab Author: Camille Cho <[email protected]> Date: Thu Jul 8 18:28:37 2021 +0800 drm/amd/display: Only set default brightness for OLED [Why] We used to unconditionally set backlight path as AUX for panels capable of backlight adjustment via DPCD in set default brightness. [How] This should be limited to OLED panel only since we control backlight via PWM path for SDR mode in LCD HDR panel. Reviewed-by: Krunoslav Kovac <[email protected]> Acked-by: Rodrigo Siqueira <[email protected]> Signed-off-by: Camille Cho <[email protected]> Signed-off-by: Alex Deucher <[email protected]> Changes some other code to only use aux for backlight control on OLED panels. The commit message seems to indicate that PWM should be used for SDR mode on HDR panels. Do something similar for backlight control in general. This may need to be revisited if and when HDR started to get used. Bug: https://gitlab.freedesktop.org/drm/amd/-/issues/1438 Bug: https://bugzilla.kernel.org/show_bug.cgi?id=213715 Reviewed-by: Roman Li <[email protected]> Signed-off-by: Alex Deucher <[email protected]>
1 parent b53e041 commit f2ad3ac

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2429,9 +2429,9 @@ static void update_connector_ext_caps(struct amdgpu_dm_connector *aconnector)
24292429
max_cll = conn_base->hdr_sink_metadata.hdmi_type1.max_cll;
24302430
min_cll = conn_base->hdr_sink_metadata.hdmi_type1.min_cll;
24312431

2432-
if (caps->ext_caps->bits.oled == 1 ||
2432+
if (caps->ext_caps->bits.oled == 1 /*||
24332433
caps->ext_caps->bits.sdr_aux_backlight_control == 1 ||
2434-
caps->ext_caps->bits.hdr_aux_backlight_control == 1)
2434+
caps->ext_caps->bits.hdr_aux_backlight_control == 1*/)
24352435
caps->aux_support = true;
24362436

24372437
if (amdgpu_backlight == 0)

0 commit comments

Comments
 (0)