Mesa (main): anv/blorp: Drop some can_ycbcr checks

GitLab Mirror gitlab-mirror at kemper.freedesktop.org
Mon Aug 9 16:19:12 UTC 2021


Module: Mesa
Branch: main
Commit: 9a267be0394741764edc9d9a0d6c85db290ebb06
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=9a267be0394741764edc9d9a0d6c85db290ebb06

Author: Jason Ekstrand <jason at jlekstrand.net>
Date:   Wed Jul 21 17:52:58 2021 -0500

anv/blorp: Drop some can_ycbcr checks

Vulkan allows us to, in theory, support ycbcr on single-plane formats if
the client really wants it.  Also, these functions should work on a
multi-plane color image as long as the client specifies the right
aspect.  This gets rid of our usage of can_ycbcr outside of anv_image.c.

Reviewed-by: Lionel Landwerlin <lionel.g.landwerlin at intel.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/12141>

---

 src/intel/vulkan/anv_blorp.c | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/src/intel/vulkan/anv_blorp.c b/src/intel/vulkan/anv_blorp.c
index ed1d4c685ef..ee405cce5f6 100644
--- a/src/intel/vulkan/anv_blorp.c
+++ b/src/intel/vulkan/anv_blorp.c
@@ -1390,8 +1390,6 @@ anv_image_msaa_resolve(struct anv_cmd_buffer *cmd_buffer,
    assert(dst_image->type == VK_IMAGE_TYPE_2D);
    assert(dst_image->samples == 1);
    assert(src_image->n_planes == dst_image->n_planes);
-   assert(!src_image->format->can_ycbcr);
-   assert(!dst_image->format->can_ycbcr);
 
    struct blorp_surf src_surf, dst_surf;
    get_blorp_surf_for_anv_image(cmd_buffer->device, src_image, aspect,
@@ -1492,8 +1490,6 @@ void anv_CmdResolveImage2KHR(
    ANV_FROM_HANDLE(anv_image, src_image, pResolveImageInfo->srcImage);
    ANV_FROM_HANDLE(anv_image, dst_image, pResolveImageInfo->dstImage);
 
-   assert(!src_image->format->can_ycbcr);
-
    for (uint32_t r = 0; r < pResolveImageInfo->regionCount; r++) {
       resolve_image(cmd_buffer,
                     src_image, pResolveImageInfo->srcImageLayout,



More information about the mesa-commit mailing list