[PATCH] drm/amd/display: Use context parameters to enable FBC

Wentland, Harry Harry.Wentland at amd.com
Mon Feb 4 15:55:42 UTC 2019


On 2019-02-04 4:20 a.m., S, Shirish wrote:
> [What]
> FBC fails to get enabled when switched between LINEAR(console/VT)
> and non-LINEAR(GUI) based rendering due to default value of
> tiling info stored in the current_state which is used for deciding
> whether or not to turn FBC on or off.
> 
> [How]
> Use context structure's tiling information which is coherant with
> the screen updates.
> 
> Signed-off-by: Shirish S <shirish.s at amd.com>
> ---
>  drivers/gpu/drm/amd/display/dc/dce110/dce110_hw_sequencer.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> 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 db0ef41..fd7cd5b 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
> @@ -2535,7 +2535,7 @@ static void dce110_apply_ctx_for_surface(
>  	}
>  
>  	if (dc->fbc_compressor)
> -		enable_fbc(dc, dc->current_state);
> +		enable_fbc(dc, context);

Looks good, but I'd want Roma's RB.

Acked-by: Harry Wentland <harry.wentland at amd.com>

Harry

>  }
>  
>  static void dce110_power_down_fe(struct dc *dc, struct pipe_ctx *pipe_ctx)
> 


More information about the amd-gfx mailing list