anv/image: Use planes[i]->primary_surface.isl.format in check_drm_format_mod
authorJason Ekstrand <jason@jlekstrand.net>
Wed, 21 Jul 2021 23:06:14 +0000 (18:06 -0500)
committerMarge Bot <eric+marge@anholt.net>
Tue, 17 Aug 2021 21:29:34 +0000 (21:29 +0000)
In theory, with linear vs. tiled differences, it could be different
(RGBA vs. RGB etc.) but it won't matter for the two checks we do with
it.  Also, we probably want to be checking the real format here anyway.

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

src/intel/vulkan/anv_image.c

index da6bbe2..2fcf4da 100644 (file)
@@ -979,10 +979,8 @@ check_drm_format_mod(const struct anv_device *device,
 
    for (int i = 0; i < image->n_planes; ++i) {
       const struct anv_image_plane *plane = &image->planes[i];
-      ASSERTED const struct anv_format_plane *plane_format =
-         &image->format->planes[i];
       ASSERTED const struct isl_format_layout *isl_layout =
-         isl_format_get_layout(plane_format->isl_format);
+         isl_format_get_layout(plane->primary_surface.isl.format);
 
       /* Enforced by us, not the Vulkan spec. */
       assert(isl_layout->txc == ISL_TXC_NONE);