From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from metis.ext.pengutronix.de ([92.198.50.35]:41654 "EHLO metis.ext.pengutronix.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754706AbbCRQ14 (ORCPT ); Wed, 18 Mar 2015 12:27:56 -0400 Message-ID: <1426696069.30356.52.camel@pengutronix.de> Subject: Re: [PATCH v3 0/5] Signalling last decoded frame by V4L2_BUF_FLAG_LAST and -EPIPE From: Philipp Zabel To: Kamil Debski Cc: 'Hans Verkuil' , linux-media@vger.kernel.org, kernel@pengutronix.de, 'Pawel Osciak' , 'Laurent Pinchart' , 'Sakari Ailus' , 'Nicolas Dufresne' Date: Wed, 18 Mar 2015 17:27:49 +0100 In-Reply-To: <016b01d06197$69e7f6b0$3db7e410$%debski@samsung.com> References: <1425637110-12100-1-git-send-email-p.zabel@pengutronix.de> <1426589206.3709.14.camel@pengutronix.de> <550851A7.8070004@xs4all.nl> <016b01d06197$69e7f6b0$3db7e410$%debski@samsung.com> Content-Type: text/plain; charset="UTF-8" Mime-Version: 1.0 Content-Transfer-Encoding: 7bit Sender: linux-media-owner@vger.kernel.org List-ID: Am Mittwoch, den 18.03.2015, 17:20 +0100 schrieb Kamil Debski: > Hi Philipp, > > > From: Hans Verkuil [mailto:hverkuil@xs4all.nl] > > Sent: Tuesday, March 17, 2015 5:09 PM > > > > On 03/17/2015 11:46 AM, Philipp Zabel wrote: > > > Hi, > > > > > > Am Freitag, den 06.03.2015, 11:18 +0100 schrieb Philipp Zabel: > > >> At the V4L2 codec API session during ELC-E 2014, we agreed that for > > >> the decoder draining flow, after a V4L2_DEC_CMD_STOP decoder command > > >> was issued, the last decoded buffer should get dequeued with a > > >> V4L2_BUF_FLAG_LAST set. After that, poll should immediately return > > >> and all following VIDIOC_DQBUF should return -EPIPE until the stream > > is stopped or decoding continued via V4L2_DEC_CMD_START. > > >> (or STREAMOFF/STREAMON). > > >> > > >> Changes since v2: > > >> - Made V4L2_BUF_FLAG_LAST known to trace events > > >> > > >> regards > > >> Philipp > > >> > > >> Peter Seiderer (1): > > >> [media] videodev2: Add V4L2_BUF_FLAG_LAST > > >> > > >> Philipp Zabel (4): > > >> [media] videobuf2: return -EPIPE from DQBUF after the last buffer > > >> [media] coda: Set last buffer flag and fix EOS event > > >> [media] s5p-mfc: Set last buffer flag > > >> [media] DocBooc: mention mem2mem codecs for encoder/decoder > > >> commands > > >> > > >> Documentation/DocBook/media/v4l/io.xml | 10 ++++++++ > > >> .../DocBook/media/v4l/vidioc-decoder-cmd.xml | 6 ++++- > > >> .../DocBook/media/v4l/vidioc-encoder-cmd.xml | 5 +++- > > >> Documentation/DocBook/media/v4l/vidioc-qbuf.xml | 8 +++++++ > > >> drivers/media/platform/coda/coda-bit.c | 4 ++-- > > >> drivers/media/platform/coda/coda-common.c | 27 +++++++++-- > > ----------- > > >> drivers/media/platform/coda/coda.h | 3 +++ > > >> drivers/media/platform/s5p-mfc/s5p_mfc.c | 1 + > > >> drivers/media/v4l2-core/v4l2-mem2mem.c | 10 +++++++- > > >> drivers/media/v4l2-core/videobuf2-core.c | 18 > > ++++++++++++++- > > >> include/media/videobuf2-core.h | 10 ++++++++ > > >> include/trace/events/v4l2.h | 3 ++- > > >> include/uapi/linux/videodev2.h | 2 ++ > > >> 13 files changed, 84 insertions(+), 23 deletions(-) > > > > > > are there any further changes that I should make to this series? > > > > I'd like to see some Acks, esp. from Kamil and Pawel. > > > > I'll take another look as well, probably on Friday. > > The patches look good IMHO. I am not sure about the order - should the > patch with code follow the documentation or should it be the other way around? > Anyway, it should be consistent. Here the documentation patches are first and > last, such that first precedes the code and then second follows the change. > > Maybe the documentation should be in the same patch that makes the API change? > > In the "DocBooc: mention mem2mem codecs for encoder/decoder commands" I would > mention that dequeue can return EPIPE. Thanks, unless otherwise instructed, I'll add that, move docs to the fron and s/DocBooc/DocBook/ in the commit message. regards Philipp