[FFmpeg-cvslog] kmsgrab: Do not require the modifier to stay constant.

[Bas Nieuwenhuizen]

ffmpeg | branch: master | Bas Nieuwenhuizen <bas at basnieuwenhuizen.nl> | Sat Nov 14 00:15:47 2020 +0100| [4386060164deb07d9fdda1716d9a3c07b24b271c] | committer: Mark Thompson

kmsgrab: Do not require the modifier to stay constant.

As we get a new set of objects each frame anyway, we
do not gain anything by keeping the modifier constant.

This helps with capturing when switching your setup a
bit, e.g. from ingame to desktop or from X11 to wayland.

Signed-off-by: Mark Thompson <sw at jkqxz.net>

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=4386060164deb07d9fdda1716d9a3c07b24b271c
---

 libavdevice/kmsgrab.c | 7 -------
 1 file changed, 7 deletions(-)

diff --git a/libavdevice/kmsgrab.c b/libavdevice/kmsgrab.c
index b740a32171..94e32b9cae 100644
--- a/libavdevice/kmsgrab.c
+++ b/libavdevice/kmsgrab.c
@@ -176,13 +176,6 @@ static int kmsgrab_get_fb2(AVFormatContext *avctx,
         err = AVERROR(EIO);
         goto fail;
     }
-    if (fb->modifier != ctx->drm_format_modifier) {
-        av_log(avctx, AV_LOG_ERROR, "Plane %"PRIu32" framebuffer "
-               "format modifier changed: now %"PRIx64".\n",
-               ctx->plane_id, fb->modifier);
-        err = AVERROR(EIO);
-        goto fail;
-    }
     if (fb->width != ctx->width || fb->height != ctx->height) {
         av_log(avctx, AV_LOG_ERROR, "Plane %"PRIu32" framebuffer "
                "dimensions changed: now %"PRIu32"x%"PRIu32".\n",