drm/amd/display: Add Logging for HDMI color depth information
authorPraful Swarnakar <Praful.Swarnakar@amd.com>
Thu, 19 Aug 2021 16:01:37 +0000 (21:31 +0530)
committerAlex Deucher <alexander.deucher@amd.com>
Tue, 24 Aug 2021 19:36:12 +0000 (15:36 -0400)
[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: Praful Swarnakar <Praful.Swarnakar@amd.com>
Reviewed-by: Hersen Wu <hersenwu@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/amd/display/dc/dcn10/dcn10_stream_encoder.c

index f1a08a7..cf364ae 100644 (file)
@@ -522,16 +522,21 @@ void enc1_stream_encoder_hdmi_set_stream_attribute(
        switch (crtc_timing->display_color_depth) {
        case COLOR_DEPTH_888:
                REG_UPDATE(HDMI_CONTROL, HDMI_DEEP_COLOR_DEPTH, 0);
+               DC_LOG_DEBUG("HDMI source set to 24BPP deep color depth\n");
                break;
        case COLOR_DEPTH_101010:
                if (crtc_timing->pixel_encoding == PIXEL_ENCODING_YCBCR422) {
                        REG_UPDATE_2(HDMI_CONTROL,
                                        HDMI_DEEP_COLOR_DEPTH, 1,
                                        HDMI_DEEP_COLOR_ENABLE, 0);
+                       DC_LOG_DEBUG("HDMI source 30BPP deep color depth"  \
+                               "disabled for YCBCR422 pixel encoding\n");
                } else {
                        REG_UPDATE_2(HDMI_CONTROL,
                                        HDMI_DEEP_COLOR_DEPTH, 1,
                                        HDMI_DEEP_COLOR_ENABLE, 1);
+                       DC_LOG_DEBUG("HDMI source 30BPP deep color depth"  \
+                               "enabled for YCBCR422 non-pixel encoding\n");
                        }
                break;
        case COLOR_DEPTH_121212:
@@ -539,16 +544,22 @@ void enc1_stream_encoder_hdmi_set_stream_attribute(
                        REG_UPDATE_2(HDMI_CONTROL,
                                        HDMI_DEEP_COLOR_DEPTH, 2,
                                        HDMI_DEEP_COLOR_ENABLE, 0);
+                       DC_LOG_DEBUG("HDMI source 36BPP deep color depth"  \
+                               "disabled for YCBCR422 pixel encoding\n");
                } else {
                        REG_UPDATE_2(HDMI_CONTROL,
                                        HDMI_DEEP_COLOR_DEPTH, 2,
                                        HDMI_DEEP_COLOR_ENABLE, 1);
+                       DC_LOG_DEBUG("HDMI source 36BPP deep color depth"  \
+                               "enabled for non-pixel YCBCR422 encoding\n");
                        }
                break;
        case COLOR_DEPTH_161616:
                REG_UPDATE_2(HDMI_CONTROL,
                                HDMI_DEEP_COLOR_DEPTH, 3,
                                HDMI_DEEP_COLOR_ENABLE, 1);
+               DC_LOG_DEBUG("HDMI source deep color depth enabled in"  \
+                               "reserved mode\n");
                break;
        default:
                break;