From: Nikola Cornij <nikola.cor...@amd.com>

[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.cor...@amd.com>
Reviewed-by: Wenjing Liu <wenjing....@amd.com>
Acked-by: Leo Li <sunpeng...@amd.com>
---
 drivers/gpu/drm/amd/display/dc/dcn20/dcn20_dsc.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/amd/display/dc/dcn20/dcn20_dsc.c 
b/drivers/gpu/drm/amd/display/dc/dcn20/dcn20_dsc.c
index 14270a98c96d..808f4d154e61 100644
--- a/drivers/gpu/drm/amd/display/dc/dcn20/dcn20_dsc.c
+++ b/drivers/gpu/drm/amd/display/dc/dcn20/dcn20_dsc.c
@@ -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);
-- 
2.22.0

_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx

Reply via email to