On Wed, Nov 08, 2017 at 05:36:33PM +0800, Karthik Ramanan wrote:
> Looks good. ACK.
Where's the patch?
> 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
_______________________________________________
meta-arago mailing list
meta-arago@arago-project.org
http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
