Re: [PATCH 3/7] [Media] vcodec: mediatek: Add Mediatek V4L2 Video Decoder Driver

From	tiffany lin <tiffany.lin@mediatek.com>
Newsgroups	linux.kernel
Subject	Re: [PATCH 3/7] [Media] vcodec: mediatek: Add Mediatek V4L2 Video Decoder Driver
Date	2016-04-19 08:50 +0200
Message-ID	<rpxDX-2sD-5@gated-at.bofh.it> (permalink)
References	(4 earlier) <rocUX-4mc-35@gated-at.bofh.it> <rpael-jl-1@gated-at.bofh.it> <rpbWO-1C9-9@gated-at.bofh.it> <rpcJb-2fT-5@gated-at.bofh.it> <rplt7-13z-3@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

Hi Nicolas,

On Mon, 2016-04-18 at 13:48 -0400, Nicolas Dufresne wrote:
> Le lundi 18 avril 2016 à 16:22 +0800, tiffany lin a écrit :
> > > > We are plaining to remove m2m framework in th feature, although
> > we think
> > > 
> > > Remove it for just the decoder driver or both encoder and decoder?
> > > 
> > Remove it from decoder driver.
> 
> Did you look at how CODA handle it (drivers/media/platform/coda/coda-
> common.c) ? I don't know any detail, but they do have the same issue
> and use both v4l2_m2m_fop_poll and v4l2_m2m_fop_mmap.
> 
I check coda-common.c, it use v4l2_m2m_set_src_buffered to allow
device_run be triggered without OUTPUT buffer.

Double check the patch "[media] mem2mem: add support for hardware
buffered queue".
This patch make m2m framework could support that
1. out-of-order frames, causing a few mem2mem device runs in the
beginning, that don't produce any decompressed buffer at the v4l2
capture side.
2. the last few frames can be decoded from the bitstream with mem2mem
device runs that don't need a new input buffer at the v4l2 output side.

This is similar our requirement that we want start decode without
CAPTURE buffer.
Is there any restriction that when v4l2_m2m_set_src_buffered can be
used?

best regards,
Tiffany

> cheers,
> Nicolas

Back to linux.kernel | Previous | Next — Previous in thread | Find similar | Unroll thread

Thread

Re: [PATCH 3/7] [Media] vcodec: mediatek: Add Mediatek V4L2 Video  Decoder Driver tiffany lin <tiffany.lin@mediatek.com> - 2016-04-18 07:50 +0200
  Re: [PATCH 3/7] [Media] vcodec: mediatek: Add Mediatek V4L2 Video  Decoder Driver Hans Verkuil <hverkuil@xs4all.nl> - 2016-04-18 09:40 +0200
    Re: [PATCH 3/7] [Media] vcodec: mediatek: Add Mediatek V4L2 Video  Decoder Driver tiffany lin <tiffany.lin@mediatek.com> - 2016-04-18 10:30 +0200
      Re: [PATCH 3/7] [Media] vcodec: mediatek: Add Mediatek V4L2 Video  Decoder Driver Nicolas Dufresne <nicolas.dufresne@gmail.com> - 2016-04-18 19:50 +0200
        Re: [PATCH 3/7] [Media] vcodec: mediatek: Add Mediatek V4L2 Video  Decoder Driver tiffany lin <tiffany.lin@mediatek.com> - 2016-04-19 08:50 +0200

csiph-web