drm/amd/display: Correct DSC PPS log
authorNikola Cornij <nikola.cornij@amd.com>
Thu, 25 Jul 2019 21:27:03 +0000 (17:27 -0400)
committerAlex Deucher <alexander.deucher@amd.com>
Thu, 15 Aug 2019 15:56:13 +0000 (10:56 -0500)
[why]
A misleading message "Programming PPS" appears before both programming
and "query PPS" functions

[how]
Move the message from "log PPS" function to "program PPS" function

Signed-off-by: Nikola Cornij <nikola.cornij@amd.com>
Reviewed-by: Wenjing Liu <Wenjing.Liu@amd.com>
Acked-by: Leo Li <sunpeng.li@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/amd/display/dc/dcn20/dcn20_dsc.c

index 14270a98c96d86c93bb8ad4fba8f5cae08523685..808f4d154e611a2621e635806ea88a45bba6ed31 100644 (file)
@@ -192,6 +192,7 @@ static void dsc2_set_config(struct display_stream_compressor *dsc, const struct
        dsc_config_log(dsc, dsc_cfg);
        is_config_ok = dsc_prepare_config(dsc_cfg, &dsc20->reg_vals, dsc_optc_cfg);
        ASSERT(is_config_ok);
+       DC_LOG_DSC("programming DSC Picture Parameter Set (PPS):");
        dsc_log_pps(dsc, &dsc20->reg_vals.pps);
        dsc_write_to_registers(dsc, &dsc20->reg_vals);
 }
@@ -250,7 +251,6 @@ static void dsc_log_pps(struct display_stream_compressor *dsc, struct drm_dsc_co
        int i;
        int bits_per_pixel = pps->bits_per_pixel;
 
-       DC_LOG_DSC("programming DSC Picture Parameter Set (PPS):");
        DC_LOG_DSC("\tdsc_version_major %d", pps->dsc_version_major);
        DC_LOG_DSC("\tdsc_version_minor %d", pps->dsc_version_minor);
        DC_LOG_DSC("\tbits_per_component %d", pps->bits_per_component);