[Freedreno] [PATCH] freedreno: move fb state copy after checking for size change

Rob Clark robdclark at gmail.com
Wed Mar 4 07:37:11 PST 2015


On Mon, Mar 2, 2015 at 9:24 PM, Ilia Mirkin <imirkin at alum.mit.edu> wrote:
> Signed-off-by: Ilia Mirkin <imirkin at alum.mit.edu>

oh, heh, when refactoring-goes-wrong... looks like I broke that in 1f3ca56b :-P

Reviewed-by: Rob Clark <robclark at freedesktop.org>

> ---
>
> Untested. But it seems pretty obvious.
>
>  src/gallium/drivers/freedreno/freedreno_state.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/src/gallium/drivers/freedreno/freedreno_state.c b/src/gallium/drivers/freedreno/freedreno_state.c
> index 6293f43..77aa4f2 100644
> --- a/src/gallium/drivers/freedreno/freedreno_state.c
> +++ b/src/gallium/drivers/freedreno/freedreno_state.c
> @@ -123,12 +123,12 @@ fd_set_framebuffer_state(struct pipe_context *pctx,
>
>         fd_context_render(pctx);
>
> -       util_copy_framebuffer_state(cso, framebuffer);
> -
>         if ((cso->width != framebuffer->width) ||
>                         (cso->height != framebuffer->height))
>                 ctx->needs_rb_fbd = true;
>
> +       util_copy_framebuffer_state(cso, framebuffer);
> +
>         ctx->dirty |= FD_DIRTY_FRAMEBUFFER;
>
>         ctx->disabled_scissor.minx = 0;
> --
> 2.0.5
>
> _______________________________________________
> Freedreno mailing list
> Freedreno at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/freedreno


More information about the Freedreno mailing list