[Mesa-dev] [PATCH 1/3] util/u_format: move utility function from r600g

[Grigori Goronzy]

We need this for radeonsi, and it might be useful for other drivers,
too.
---
 src/gallium/auxiliary/util/u_format.c | 11 +++++++++++
 src/gallium/auxiliary/util/u_format.h |  3 +++
 src/gallium/drivers/r600/r600_blit.c  | 12 +-----------
 3 files changed, 15 insertions(+), 11 deletions(-)

diff --git a/src/gallium/auxiliary/util/u_format.c 
b/src/gallium/auxiliary/util/u_format.c
index 056f82f..a53ed6f 100644
--- a/src/gallium/auxiliary/util/u_format.c
+++ b/src/gallium/auxiliary/util/u_format.c
@@ -187,6 +187,17 @@ util_format_is_intensity(enum pipe_format format)
    return FALSE;
 }
 
+boolean
+util_format_is_subsampled_422(enum pipe_format format)
+{
+   const struct util_format_description *desc =
+      util_format_description(format);
+
+   return desc->layout == UTIL_FORMAT_LAYOUT_SUBSAMPLED &&
+      desc->block.width == 2 &&
+      desc->block.height == 1 &&
+      desc->block.bits == 32;
+}
 
 boolean
 util_format_is_supported(enum pipe_format format, unsigned bind)
diff --git a/src/gallium/auxiliary/util/u_format.h 
b/src/gallium/auxiliary/util/u_format.h
index 1dd5d52..2e2bf02 100644
--- a/src/gallium/auxiliary/util/u_format.h
+++ b/src/gallium/auxiliary/util/u_format.h
@@ -664,6 +664,9 @@ boolean
 util_format_is_intensity(enum pipe_format format);
 
 boolean
+util_format_is_subsampled_422(enum pipe_format format);
+
+boolean
 util_format_is_pure_integer(enum pipe_format format);
 
 boolean
diff --git a/src/gallium/drivers/r600/r600_blit.c 
b/src/gallium/drivers/r600/r600_blit.c
index 3269c47..962be60 100644
--- a/src/gallium/drivers/r600/r600_blit.c
+++ b/src/gallium/drivers/r600/r600_blit.c
@@ -563,16 +563,6 @@ static void r600_clear_buffer(struct pipe_context *ctx, 
struct pipe_resource *ds
        }
 }
 
-static bool util_format_is_subsampled_2x1_32bpp(enum pipe_format format)
-{
-       const struct util_format_description *desc = 
util_format_description(format);
-
-       return desc->layout == UTIL_FORMAT_LAYOUT_SUBSAMPLED &&
-              desc->block.width == 2 &&
-              desc->block.height == 1 &&
-              desc->block.bits == 32;
-}
-
 static void r600_resource_copy_region(struct pipe_context *ctx,
                                      struct pipe_resource *dst,
                                      unsigned dst_level,
@@ -647,7 +637,7 @@ static void r600_resource_copy_region(struct pipe_context 
*ctx,
 
                src_force_level = src_level;
        } else if (!util_blitter_is_copy_supported(rctx->blitter, dst, src)) {
-               if (util_format_is_subsampled_2x1_32bpp(src->format)) {
+               if (util_format_is_subsampled_422(src->format)) {
 
                        src_templ.format = PIPE_FORMAT_R8G8B8A8_UINT;
                        dst_templ.format = PIPE_FORMAT_R8G8B8A8_UINT;
-- 
1.8.3.2

_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev