[Mesa-dev] [PATCH] i965: Rename brw_format variable names to isl_format

Jason Ekstrand jason at jlekstrand.net
Tue May 23 20:09:31 UTC 2017


Bonus points if you also update the types to "enum isl_format"


On May 23, 2017 9:53:12 AM Anuj Phogat <anuj.phogat at gmail.com> wrote:

> This patch makes non functional changes. Renaming is just to
> make the code more readable.
>
> Signed-off-by: Anuj Phogat <anuj.phogat at gmail.com>
> ---
>  src/mesa/drivers/dri/i965/brw_context.c          | 4 ++--
>  src/mesa/drivers/dri/i965/brw_wm_surface_state.c | 6 +++---
>  src/mesa/drivers/dri/i965/intel_mipmap_tree.c    | 4 ++--
>  3 files changed, 7 insertions(+), 7 deletions(-)
>
> diff --git a/src/mesa/drivers/dri/i965/brw_context.c 
> b/src/mesa/drivers/dri/i965/brw_context.c
> index d3ed871..e74e069 100644
> --- a/src/mesa/drivers/dri/i965/brw_context.c
> +++ b/src/mesa/drivers/dri/i965/brw_context.c
> @@ -205,9 +205,9 @@ intel_texture_view_requires_resolve(struct brw_context 
> *brw,
>         !intel_miptree_is_lossless_compressed(brw, intel_tex->mt))
>       return false;
>
> -   const uint32_t brw_format = 
> brw_isl_format_for_mesa_format(intel_tex->_Format);
> +   const uint32_t isl_format = 
> brw_isl_format_for_mesa_format(intel_tex->_Format);
>
> -   if (isl_format_supports_ccs_e(&brw->screen->devinfo, brw_format))
> +   if (isl_format_supports_ccs_e(&brw->screen->devinfo, isl_format))
>        return false;
>
>     perf_debug("Incompatible sampling format (%s) for rbc (%s)\n",
> diff --git a/src/mesa/drivers/dri/i965/brw_wm_surface_state.c 
> b/src/mesa/drivers/dri/i965/brw_wm_surface_state.c
> index a0fed60..faa7412 100644
> --- a/src/mesa/drivers/dri/i965/brw_wm_surface_state.c
> +++ b/src/mesa/drivers/dri/i965/brw_wm_surface_state.c
> @@ -686,7 +686,7 @@ brw_update_buffer_texture_surface(struct gl_context *ctx,
>     uint32_t size = tObj->BufferSize;
>     struct brw_bo *bo = NULL;
>     mesa_format format = tObj->_BufferObjectFormat;
> -   uint32_t brw_format = brw_isl_format_for_mesa_format(format);
> +   uint32_t isl_format = brw_isl_format_for_mesa_format(format);
>     int texel_size = _mesa_get_format_bytes(format);
>
>     if (intel_obj) {
> @@ -712,14 +712,14 @@ brw_update_buffer_texture_surface(struct gl_context *ctx,
>      */
>     size = MIN2(size, ctx->Const.MaxTextureBufferSize * (unsigned) texel_size);
>
> -   if (brw_format == 0 && format != MESA_FORMAT_RGBA_FLOAT32) {
> +   if (isl_format == 0 && format != MESA_FORMAT_RGBA_FLOAT32) {
>        _mesa_problem(NULL, "bad format %s for texture buffer\n",
>  		    _mesa_get_format_name(format));
>     }
>
>     brw_emit_buffer_surface_state(brw, surf_offset, bo,
>                                   tObj->BufferOffset,
> -                                 brw_format,
> +                                 isl_format,
>                                   size,
>                                   texel_size,
>                                   false /* rw */);
> diff --git a/src/mesa/drivers/dri/i965/intel_mipmap_tree.c 
> b/src/mesa/drivers/dri/i965/intel_mipmap_tree.c
> index db0a397..4eacd3c 100644
> --- a/src/mesa/drivers/dri/i965/intel_mipmap_tree.c
> +++ b/src/mesa/drivers/dri/i965/intel_mipmap_tree.c
> @@ -209,8 +209,8 @@ intel_miptree_supports_non_msrt_fast_clear(struct 
> brw_context *brw,
>
>     if (brw->gen >= 9) {
>        mesa_format linear_format = _mesa_get_srgb_format_linear(mt->format);
> -      const uint32_t brw_format = 
> brw_isl_format_for_mesa_format(linear_format);
> -      return isl_format_supports_ccs_e(&brw->screen->devinfo, brw_format);
> +      const uint32_t isl_format = 
> brw_isl_format_for_mesa_format(linear_format);
> +      return isl_format_supports_ccs_e(&brw->screen->devinfo, isl_format);
>     } else
>        return true;
>  }
> --
> 2.9.3
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev




More information about the mesa-dev mailing list