[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