Commit 3341d30d authored by Praful Swarnakar's avatar Praful Swarnakar Committed by Alex Deucher

drm/amd/display: Add Logging for HDMI color depth information

[Why]
Recent HDMI2.0 HF1-1 V-Swing testing showed that logging deep color
status helps in validation of testcase.

[How]
Add logging based on various color depths and pixel encoding
formats.
Signed-off-by: default avatarPraful Swarnakar <Praful.Swarnakar@amd.com>
Reviewed-by: default avatarHersen Wu <hersenwu@amd.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent 30acef3c
...@@ -522,16 +522,21 @@ void enc1_stream_encoder_hdmi_set_stream_attribute( ...@@ -522,16 +522,21 @@ void enc1_stream_encoder_hdmi_set_stream_attribute(
switch (crtc_timing->display_color_depth) { switch (crtc_timing->display_color_depth) {
case COLOR_DEPTH_888: case COLOR_DEPTH_888:
REG_UPDATE(HDMI_CONTROL, HDMI_DEEP_COLOR_DEPTH, 0); REG_UPDATE(HDMI_CONTROL, HDMI_DEEP_COLOR_DEPTH, 0);
DC_LOG_DEBUG("HDMI source set to 24BPP deep color depth\n");
break; break;
case COLOR_DEPTH_101010: case COLOR_DEPTH_101010:
if (crtc_timing->pixel_encoding == PIXEL_ENCODING_YCBCR422) { if (crtc_timing->pixel_encoding == PIXEL_ENCODING_YCBCR422) {
REG_UPDATE_2(HDMI_CONTROL, REG_UPDATE_2(HDMI_CONTROL,
HDMI_DEEP_COLOR_DEPTH, 1, HDMI_DEEP_COLOR_DEPTH, 1,
HDMI_DEEP_COLOR_ENABLE, 0); HDMI_DEEP_COLOR_ENABLE, 0);
DC_LOG_DEBUG("HDMI source 30BPP deep color depth" \
"disabled for YCBCR422 pixel encoding\n");
} else { } else {
REG_UPDATE_2(HDMI_CONTROL, REG_UPDATE_2(HDMI_CONTROL,
HDMI_DEEP_COLOR_DEPTH, 1, HDMI_DEEP_COLOR_DEPTH, 1,
HDMI_DEEP_COLOR_ENABLE, 1); HDMI_DEEP_COLOR_ENABLE, 1);
DC_LOG_DEBUG("HDMI source 30BPP deep color depth" \
"enabled for YCBCR422 non-pixel encoding\n");
} }
break; break;
case COLOR_DEPTH_121212: case COLOR_DEPTH_121212:
...@@ -539,16 +544,22 @@ void enc1_stream_encoder_hdmi_set_stream_attribute( ...@@ -539,16 +544,22 @@ void enc1_stream_encoder_hdmi_set_stream_attribute(
REG_UPDATE_2(HDMI_CONTROL, REG_UPDATE_2(HDMI_CONTROL,
HDMI_DEEP_COLOR_DEPTH, 2, HDMI_DEEP_COLOR_DEPTH, 2,
HDMI_DEEP_COLOR_ENABLE, 0); HDMI_DEEP_COLOR_ENABLE, 0);
DC_LOG_DEBUG("HDMI source 36BPP deep color depth" \
"disabled for YCBCR422 pixel encoding\n");
} else { } else {
REG_UPDATE_2(HDMI_CONTROL, REG_UPDATE_2(HDMI_CONTROL,
HDMI_DEEP_COLOR_DEPTH, 2, HDMI_DEEP_COLOR_DEPTH, 2,
HDMI_DEEP_COLOR_ENABLE, 1); HDMI_DEEP_COLOR_ENABLE, 1);
DC_LOG_DEBUG("HDMI source 36BPP deep color depth" \
"enabled for non-pixel YCBCR422 encoding\n");
} }
break; break;
case COLOR_DEPTH_161616: case COLOR_DEPTH_161616:
REG_UPDATE_2(HDMI_CONTROL, REG_UPDATE_2(HDMI_CONTROL,
HDMI_DEEP_COLOR_DEPTH, 3, HDMI_DEEP_COLOR_DEPTH, 3,
HDMI_DEEP_COLOR_ENABLE, 1); HDMI_DEEP_COLOR_ENABLE, 1);
DC_LOG_DEBUG("HDMI source deep color depth enabled in" \
"reserved mode\n");
break; break;
default: default:
break; break;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment