[Mesa-dev] [PATCH 01/14] i965: Use the brw_context for the clear color and value setters

Jason Ekstrand jason at jlekstrand.net
Tue Apr 3 22:18:08 UTC 2018


Yeah, brw is always better for i965 functions.

Reviewed-by: Jason Ekstrand <jason at jlekstrand.net>

On Fri, Mar 30, 2018 at 11:12 AM, Nanley Chery <nanleychery at gmail.com>
wrote:

> Do what all the other functions in the miptree API do.
> ---
>  src/mesa/drivers/dri/i965/brw_blorp.c         | 2 +-
>  src/mesa/drivers/dri/i965/brw_clear.c         | 2 +-
>  src/mesa/drivers/dri/i965/intel_mipmap_tree.h | 8 ++++----
>  3 files changed, 6 insertions(+), 6 deletions(-)
>
> diff --git a/src/mesa/drivers/dri/i965/brw_blorp.c
> b/src/mesa/drivers/dri/i965/brw_blorp.c
> index 44064fc0cf3..f5a653fff97 100644
> --- a/src/mesa/drivers/dri/i965/brw_blorp.c
> +++ b/src/mesa/drivers/dri/i965/brw_blorp.c
> @@ -1241,7 +1241,7 @@ do_single_blorp_clear(struct brw_context *brw,
> struct gl_framebuffer *fb,
>                                             &ctx->Color.ClearColor);
>
>        bool same_clear_color =
> -         !intel_miptree_set_clear_color(ctx, irb->mt, clear_color);
> +         !intel_miptree_set_clear_color(brw, irb->mt, clear_color);
>
>        /* If the buffer is already in INTEL_FAST_CLEAR_STATE_CLEAR, the
> clear
>         * is redundant and can be skipped.
> diff --git a/src/mesa/drivers/dri/i965/brw_clear.c
> b/src/mesa/drivers/dri/i965/brw_clear.c
> index 8aa83722ee9..8f7594baf59 100644
> --- a/src/mesa/drivers/dri/i965/brw_clear.c
> +++ b/src/mesa/drivers/dri/i965/brw_clear.c
> @@ -212,7 +212,7 @@ brw_fast_clear_depth(struct gl_context *ctx)
>           }
>        }
>
> -      intel_miptree_set_depth_clear_value(ctx, mt, clear_value);
> +      intel_miptree_set_depth_clear_value(brw, mt, clear_value);
>     }
>
>     bool need_clear = false;
> diff --git a/src/mesa/drivers/dri/i965/intel_mipmap_tree.h
> b/src/mesa/drivers/dri/i965/intel_mipmap_tree.h
> index 07c85807e80..0c24b93e3da 100644
> --- a/src/mesa/drivers/dri/i965/intel_mipmap_tree.h
> +++ b/src/mesa/drivers/dri/i965/intel_mipmap_tree.h
> @@ -717,26 +717,26 @@ intel_miptree_sample_with_hiz(struct brw_context
> *brw,
>
>
>  static inline bool
> -intel_miptree_set_clear_color(struct gl_context *ctx,
> +intel_miptree_set_clear_color(struct brw_context *brw,
>                                struct intel_mipmap_tree *mt,
>                                union isl_color_value clear_color)
>  {
>     if (memcmp(&mt->fast_clear_color, &clear_color, sizeof(clear_color))
> != 0) {
>        mt->fast_clear_color = clear_color;
> -      ctx->NewDriverState |= BRW_NEW_AUX_STATE;
> +      brw->ctx.NewDriverState |= BRW_NEW_AUX_STATE;
>        return true;
>     }
>     return false;
>  }
>
>  static inline bool
> -intel_miptree_set_depth_clear_value(struct gl_context *ctx,
> +intel_miptree_set_depth_clear_value(struct brw_context *brw,
>                                      struct intel_mipmap_tree *mt,
>                                      float clear_value)
>  {
>     if (mt->fast_clear_color.f32[0] != clear_value) {
>        mt->fast_clear_color.f32[0] = clear_value;
> -      ctx->NewDriverState |= BRW_NEW_AUX_STATE;
> +      brw->ctx.NewDriverState |= BRW_NEW_AUX_STATE;
>        return true;
>     }
>     return false;
> --
> 2.16.2
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20180403/82f938d0/attachment-0001.html>


More information about the mesa-dev mailing list