Looks good. ACK.
On 08-Nov-17 11:41 AM, Ramprasad N wrote:
Signed-off-by: Ramprasad N <x0038...@ti.com> --- ...v4l2src-Increase-minimum-num-buffers-by-4.patch | 33 ++++++++++++++++++++++ .../gstreamer1.0-plugins-good_1.8.3.bbappend | 8 ++++++ 2 files changed, 41 insertions(+) create mode 100644 meta-arago-extras/recipes-multimedia/gstreamer/gstreamer1.0-plugins-good/0001-v4l2src-Increase-minimum-num-buffers-by-4.patch create mode 100644 meta-arago-extras/recipes-multimedia/gstreamer/gstreamer1.0-plugins-good_1.8.3.bbappend diff --git a/meta-arago-extras/recipes-multimedia/gstreamer/gstreamer1.0-plugins-good/0001-v4l2src-Increase-minimum-num-buffers-by-4.patch b/meta-arago-extras/recipes-multimedia/gstreamer/gstreamer1.0-plugins-good/0001-v4l2src-Increase-minimum-num-buffers-by-4.patch new file mode 100644 index 0000000..1af90c1 --- /dev/null +++ b/meta-arago-extras/recipes-multimedia/gstreamer/gstreamer1.0-plugins-good/0001-v4l2src-Increase-minimum-num-buffers-by-4.patch @@ -0,0 +1,33 @@ +From 73254df370fec6a2eb1ae6711955307a1d4bf173 Mon Sep 17 00:00:00 2001 +From: Ramprasad N <x0038...@ti.com> +Date: Fri, 13 Oct 2017 15:48:06 +0530 +Subject: [PATCH] v4l2src: Increase minimum num buffers by 4 + +v4l2src plugin allocates 4 buffers for the pipeline. +To capture video in realtime the pipeline without frameloss, +four buffers are not sufficient. Increase the +number of buffers by 4 more. + +Signed-off-by: Ramprasad N <x0038...@ti.com> +--- + sys/v4l2/gstv4l2object.c | 4 +++- + 1 file changed, 3 insertions(+), 1 deletion(-) + +diff --git a/sys/v4l2/gstv4l2object.c b/sys/v4l2/gstv4l2object.c +index 549e06f..773f46d 100644 +--- a/sys/v4l2/gstv4l2object.c ++++ b/sys/v4l2/gstv4l2object.c +@@ -3939,7 +3939,9 @@ gst_v4l2_object_decide_allocation (GstV4l2Object * obj, GstQuery * query) + * to fill the pipeline, the minimum required to decoder according to the + * driver and 1 more, so we don't endup up with everything downstream or + * held by the decoder. */ +- own_min = min + obj->min_buffers + 1; ++ /* Increase the number of buffers required by 4 more to capture video in ++ * realtime. With default number of 4 buffers, frame loss is observed */ ++ own_min = min + obj->min_buffers + 5; + + /* If no allocation parameters where provided, allow for a little more + * buffers and enable copy threshold */ +-- +1.9.1 + diff --git a/meta-arago-extras/recipes-multimedia/gstreamer/gstreamer1.0-plugins-good_1.8.3.bbappend b/meta-arago-extras/recipes-multimedia/gstreamer/gstreamer1.0-plugins-good_1.8.3.bbappend new file mode 100644 index 0000000..29a07b2 --- /dev/null +++ b/meta-arago-extras/recipes-multimedia/gstreamer/gstreamer1.0-plugins-good_1.8.3.bbappend @@ -0,0 +1,8 @@ +PR_append = ".arago0" + +FILESEXTRAPATHS_prepend := "${THISDIR}/${PN}:" + +SRC_URI += " \ + file://0001-v4l2src-Increase-minimum-num-buffers-by-4.patch \ + " +
_______________________________________________ meta-arago mailing list meta-arago@arago-project.org http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago