From 2f410f26bf2d1540a41d0357d41912a458898653 Mon Sep 17 00:00:00 2001
From: Damian Hobson-Garcia <dhobsong@igel.co.jp>
Date: Fri, 15 Oct 2021 15:32:22 +0900
Subject: [PATCH] v4l2src: Reset the compose window to the default after
 setting format

When the size of V4L2 capture or output is changes with VIDIOC_S_FMT,
the device is only required to update the compisition window to fit
inside the new frame size.  This can result in captured data only being
updated on a portion of the frame after a resize.

Update the composition window to the default value determined by the
V4L2 device driver whenever the format is changed to make sure that
all image data is composed to its full size.

Fixes #765

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1806>
---
 .../gst-plugins-good/sys/v4l2/gstv4l2object.c | 38 +++++++++++++++++++
 1 file changed, 38 insertions(+)

diff --git a/subprojects/gst-plugins-good/sys/v4l2/gstv4l2object.c b/subprojects/gst-plugins-good/sys/v4l2/gstv4l2object.c
index 18665168ca..c739f17b99 100644
--- a/subprojects/gst-plugins-good/sys/v4l2/gstv4l2object.c
+++ b/subprojects/gst-plugins-good/sys/v4l2/gstv4l2object.c
@@ -3170,6 +3170,41 @@ no_supported_capture_method:
   }
 }
 
+static gboolean
+gst_v4l2_object_reset_compose_region (GstV4l2Object * obj)
+{
+  struct v4l2_selection sel = { 0 };
+
+  GST_V4L2_CHECK_OPEN (obj);
+  GST_V4L2_CHECK_NOT_ACTIVE (obj);
+
+  sel.type = obj->type;
+  sel.target = V4L2_SEL_TGT_COMPOSE_DEFAULT;
+
+  if (obj->ioctl (obj->video_fd, VIDIOC_G_SELECTION, &sel) < 0) {
+    if (errno == ENOTTY) {
+      /* No-op when selection API is not supported */
+      return TRUE;
+    } else {
+      GST_WARNING_OBJECT (obj->dbg_obj,
+          "Failed to get default compose rectangle with VIDIOC_G_SELECTION: %s",
+          g_strerror (errno));
+      return FALSE;
+    }
+  }
+
+  sel.target = V4L2_SEL_TGT_COMPOSE;
+
+  if (obj->ioctl (obj->video_fd, VIDIOC_S_SELECTION, &sel) < 0) {
+    GST_WARNING_OBJECT (obj->dbg_obj,
+        "Failed to set default compose rectangle with VIDIOC_S_SELECTION: %s",
+        g_strerror (errno));
+    return FALSE;
+  }
+
+  return TRUE;
+}
+
 static void
 gst_v4l2_object_set_stride (GstVideoInfo * info, GstVideoAlignment * align,
     gint plane, gint stride)
@@ -3979,6 +4014,9 @@ done:
   /* add boolean return, so we can fail on drivers bugs */
   gst_v4l2_object_save_format (v4l2object, fmtdesc, &format, &info, &align);
 
+  /* reset composition region to match the S_FMT size */
+  gst_v4l2_object_reset_compose_region (v4l2object);
+
   /* now configure the pool */
   if (!gst_v4l2_object_setup_pool (v4l2object, caps))
     goto pool_failed;