Re: [PATCH] drm/amd/display: Remove redundant logs from DSC code

[Hamza Mahfooz <hamza.mahfooz@xxxxxxx>]

On 12/16/22 05:35, Praful Swarnakar wrote:
> [Why & How]
> Remove redundant log in DSC that just add additional blank prints
>
> Signed-off-by: Praful Swarnakar <praful.swarnakar@xxxxxxx>

Applied, thanks!

> ---
>   drivers/gpu/drm/amd/display/dc/core/dc_link_dp.c | 1 -
>   drivers/gpu/drm/amd/display/dc/dcn20/dcn20_dsc.c | 1 -
>   2 files changed, 2 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/display/dc/core/dc_link_dp.c b/drivers/gpu/drm/amd/display/dc/core/dc_link_dp.c
> index af9411ee3c74..f2b6d40e4f5c 100644
> --- a/drivers/gpu/drm/amd/display/dc/core/dc_link_dp.c
> +++ b/drivers/gpu/drm/amd/display/dc/core/dc_link_dp.c
> @@ -7510,7 +7510,6 @@ bool dp_set_dsc_pps_sdp(struct pipe_ctx *pipe_ctx, bool enable, bool immediate_u
>   		dsc_cfg.is_odm = pipe_ctx->next_odm_pipe ? true : false;
>   		dsc_cfg.dc_dsc_cfg = stream->timing.dsc_cfg;
>   
> -		DC_LOG_DSC(" ");
>   		dsc->funcs->dsc_get_packed_pps(dsc, &dsc_cfg, &dsc_packed_pps[0]);
>   		memcpy(&stream->dsc_packed_pps[0], &dsc_packed_pps[0], sizeof(stream->dsc_packed_pps));
>   		if (dc_is_dp_signal(stream->signal)) {
> 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 784a8b6f360d..c08c01e05dcf 100644
> --- a/drivers/gpu/drm/amd/display/dc/dcn20/dcn20_dsc.c
> +++ b/drivers/gpu/drm/amd/display/dc/dcn20/dcn20_dsc.c
> @@ -200,7 +200,6 @@ static void dsc2_set_config(struct display_stream_compressor *dsc, const struct
>   	bool is_config_ok;
>   	struct dcn20_dsc *dsc20 = TO_DCN20_DSC(dsc);
>   
> -	DC_LOG_DSC(" ");
>   	DC_LOG_DSC("Setting DSC Config at DSC inst %d", dsc->inst);
>   	dsc_config_log(dsc, dsc_cfg);
>   	is_config_ok = dsc_prepare_config(dsc_cfg, &dsc20->reg_vals, dsc_optc_cfg);

--
Hamza