Mesa (staging/21.1): v3d: YUV formats at is_dmabuf_modifier_supported are external_only

[GitLab Mirror]

Module: Mesa
Branch: staging/21.1
Commit: e633a4a3a86dfb6d60fce6fd514989b93697ea49
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=e633a4a3a86dfb6d60fce6fd514989b93697ea49

Author: Jose Maria Casanova Crespo <jmcasanova at igalia.com>
Date:   Thu Apr 29 18:15:27 2021 +0200

v3d: YUV formats at is_dmabuf_modifier_supported are external_only

This fixes Issue https://github.com/Igalia/meta-webkit/issues/185
"Issue Raspberry 4-64 + Mesa VC4 driver + Gstreamer = red Label on video"

Fixes: 6ee10ab3de86 "gallium: Add pipe_screen::is_dmabuf_modifier_supported"
Reviewed-by: Alejandro Piñeiro <apinheiro at igalia.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/10524>
(cherry picked from commit 3d7b378980cdcf67e282649b06e5792466313e8f)

---

 .pick_status.json                    | 2 +-
 src/gallium/drivers/v3d/v3d_screen.c | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/.pick_status.json b/.pick_status.json
index b5595de12e8..1338ed0683a 100644
--- a/.pick_status.json
+++ b/.pick_status.json
@@ -247,7 +247,7 @@
         "description": "v3d: YUV formats at is_dmabuf_modifier_supported are external_only",
         "nominated": true,
         "nomination_type": 1,
-        "resolution": 0,
+        "resolution": 1,
         "master_sha": null,
         "because_sha": "6ee10ab3de8664011a6c75e2357faba1ec95c094"
     },
diff --git a/src/gallium/drivers/v3d/v3d_screen.c b/src/gallium/drivers/v3d/v3d_screen.c
index 059f41f459e..d68e48e5688 100644
--- a/src/gallium/drivers/v3d/v3d_screen.c
+++ b/src/gallium/drivers/v3d/v3d_screen.c
@@ -691,7 +691,7 @@ v3d_screen_is_dmabuf_modifier_supported(struct pipe_screen *pscreen,
         for (i = 0; i < ARRAY_SIZE(v3d_available_modifiers) - 1; i++) {
                 if (v3d_available_modifiers[i] == modifier) {
                         if (external_only)
-                                *external_only = false;
+                                *external_only = util_format_is_yuv(format);
 
                         return true;
                 }