Re: [PATCH 07/14] drm/amd/display: Initialize DSC PPS variables to 0

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 8/19/19 11:50 AM, David Francis wrote:
> For DSC MST, sometimes monitors would break out
> in full-screen static. The issue traced back to the
> PPS generation code, where these variables were being used
> uninitialized and were picking up garbage.
> 
> memset to 0 to avoid this
> 
> Signed-off-by: David Francis <David.Francis@xxxxxxx>

Patches 1-5 and 7 are:

Reviewed-by: Nicholas Kazlauskas <nicholas.kazlauskas@xxxxxxx>

> ---
>   drivers/gpu/drm/amd/display/dc/core/dc_link_hwss.c | 3 +++
>   drivers/gpu/drm/amd/display/dc/dcn20/dcn20_dsc.c   | 3 +++
>   2 files changed, 6 insertions(+)
> 
> diff --git a/drivers/gpu/drm/amd/display/dc/core/dc_link_hwss.c b/drivers/gpu/drm/amd/display/dc/core/dc_link_hwss.c
> index 35c5467e60e8..619ac48edd05 100644
> --- a/drivers/gpu/drm/amd/display/dc/core/dc_link_hwss.c
> +++ b/drivers/gpu/drm/amd/display/dc/core/dc_link_hwss.c
> @@ -491,6 +491,9 @@ bool dp_set_dsc_pps_sdp(struct pipe_ctx *pipe_ctx, bool enable)
>   		struct dsc_config dsc_cfg;
>   		uint8_t dsc_packed_pps[128];
>   
> +		memset(&dsc_cfg, 0, sizeof(dsc_cfg));
> +		memset(dsc_packed_pps, 0, 128);
> +
>   		/* Enable DSC hw block */
>   		dsc_cfg.pic_width = stream->timing.h_addressable + stream->timing.h_border_left + stream->timing.h_border_right;
>   		dsc_cfg.pic_height = stream->timing.v_addressable + stream->timing.v_border_top + stream->timing.v_border_bottom;
> 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 379c9e4ac63b..16debe6d89f2 100644
> --- a/drivers/gpu/drm/amd/display/dc/dcn20/dcn20_dsc.c
> +++ b/drivers/gpu/drm/amd/display/dc/dcn20/dcn20_dsc.c
> @@ -207,6 +207,9 @@ static bool dsc2_get_packed_pps(struct display_stream_compressor *dsc, const str
>   	struct dsc_reg_values dsc_reg_vals;
>   	struct dsc_optc_config dsc_optc_cfg;
>   
> +	memset(&dsc_reg_vals, 0, sizeof(dsc_reg_vals));
> +	memset(&dsc_optc_cfg, 0, sizeof(dsc_optc_cfg));
> +
>   	DC_LOG_DSC("Getting packed DSC PPS for DSC Config:");
>   	dsc_config_log(dsc, dsc_cfg);
>   	DC_LOG_DSC("DSC Picture Parameter Set (PPS):");
> 

_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/dri-devel




[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux