Re: [PATCH v3 0/5] Signalling last decoded frame by V4L2_BUF_FLAG_LAST and -EPIPE

[Philipp Zabel <p.zabel@pengutronix.de>]

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?

regards
Philipp