[PATCH 06/10] drm/amd/display: Add Replay supported/enabled checks

[Harry Wentland]

On 2023-07-10 15:27, Bhawanpreet Lakha wrote:
> - Add checks for Cursor update and dirty rects (sending updates to dmub)
> - Add checks for dc_notify_vsync, and fbc and subvp
> 
> Signed-off-by: Bhawanpreet Lakha <Bhawanpreet.Lakha at amd.com>
> ---
>  drivers/gpu/drm/amd/display/dc/core/dc.c                    | 6 ++++++
>  drivers/gpu/drm/amd/display/dc/dc_dmub_srv.c                | 3 +++
>  drivers/gpu/drm/amd/display/dc/dce110/dce110_hw_sequencer.c | 6 ++++++
>  3 files changed, 15 insertions(+)
> 
> diff --git a/drivers/gpu/drm/amd/display/dc/core/dc.c b/drivers/gpu/drm/amd/display/dc/core/dc.c
> index dd3a9d06c6e2..dccb49e27f35 100644
> --- a/drivers/gpu/drm/amd/display/dc/core/dc.c
> +++ b/drivers/gpu/drm/amd/display/dc/core/dc.c
> @@ -3367,6 +3367,9 @@ static bool dc_dmub_should_send_dirty_rect_cmd(struct dc *dc, struct dc_stream_s
>  			&& stream->ctx->dce_version >= DCN_VERSION_3_1)
>  		return true;
>  
> +        if (stream->link->replay_settings.config.replay_supported)
> +                return true;
> +
>  	return false;
>  }
>  
> @@ -5116,6 +5119,9 @@ void dc_notify_vsync_int_state(struct dc *dc, struct dc_stream_state *stream, bo
>  	if (link->psr_settings.psr_feature_enabled)
>  		return;
>  
> +        if (link->replay_settings.replay_feature_enabled)
> +                return;

Use tabs, not spaces. I just noticed it here but it's elsewhere in this patch.
Please check the other patches as well.

> +
>  	/*find primary pipe associated with stream*/
>  	for (i = 0; i < MAX_PIPES; i++) {
>  		pipe = &dc->current_state->res_ctx.pipe_ctx[i];
> diff --git a/drivers/gpu/drm/amd/display/dc/dc_dmub_srv.c b/drivers/gpu/drm/amd/display/dc/dc_dmub_srv.c
> index c52c40b16387..706c49e015f6 100644
> --- a/drivers/gpu/drm/amd/display/dc/dc_dmub_srv.c
> +++ b/drivers/gpu/drm/amd/display/dc/dc_dmub_srv.c
> @@ -894,6 +894,9 @@ static bool dc_dmub_should_update_cursor_data(struct pipe_ctx *pipe_ctx)
>  		pipe_ctx->stream->ctx->dce_version >= DCN_VERSION_3_1)
>  		return true;
>  
> +	if (pipe_ctx->stream->link->replay_settings.config.replay_supported)
> +                return true;
> +
>  	return false;
>  }
>  
> diff --git a/drivers/gpu/drm/amd/display/dc/dce110/dce110_hw_sequencer.c b/drivers/gpu/drm/amd/display/dc/dce110/dce110_hw_sequencer.c
> index 6c9ca43d1040..16b53a4c5a42 100644
> --- a/drivers/gpu/drm/amd/display/dc/dce110/dce110_hw_sequencer.c
> +++ b/drivers/gpu/drm/amd/display/dc/dce110/dce110_hw_sequencer.c
> @@ -1585,6 +1585,8 @@ static enum dc_status apply_single_controller_ctx_to_hw(
>  	 */
>  	if (pipe_ctx->stream->mall_stream_config.type != SUBVP_PHANTOM) {
>  		pipe_ctx->stream->link->psr_settings.psr_feature_enabled = false;
> +		printk("######  %s %d\n",__func__,__LINE__);

stray debug printk

Harry

> +		pipe_ctx->stream->link->replay_settings.replay_feature_enabled = false;
>  	}
>  	return DC_OK;
>  }
> @@ -2013,6 +2015,10 @@ static bool should_enable_fbc(struct dc *dc,
>  	if (pipe_ctx->stream->link->psr_settings.psr_feature_enabled)
>  		return false;
>  
> +	/* Replay should not be enabled */
> +        if (pipe_ctx->stream->link->replay_settings.replay_feature_enabled)
> +                return false;
> +
>  	/* Nothing to compress */
>  	if (!pipe_ctx->plane_state)
>  		return false;