mbox

[GIT,PULL,FOR,v5.1] vb2 + cedrus fixes/improvements

Message ID 342f754a-0ef9-e534-7076-802875b53417@xs4all.nl (mailing list archive)
State Accepted, archived
Headers

Pull-request

git://linuxtv.org/hverkuil/media_tree.git tags/br-v5.1i3

Message

Hans Verkuil Jan. 29, 2019, 12:19 p.m. UTC
The first two patches simplify the function to find buffers based on a
timestamp: the current code only considered DONE and DEQUEUED buffers,
but that is unnecessarily restrictive.

The next two patches keep dmabuf-based buffers mapped when they are
dequeued. This avoids having to re-map them if they are referenced as key
frames in a stateless codec. This patch has been in use by ChromeOS for
quite some time and it makes sense.

It is more efficient in general anyway since you don't want to have to
map a dmabuf every time you queue it if there is no need for it.

The next 5 patches fix v4l2-compliance bugs in vivid and vimc for output
field handling in requests.

The final patch fixes a vivid buf_prepare bug.

Regards,

	Hans

This PR supersedes https://patchwork.linuxtv.org/patch/54168/, adding
patch 'vivid: fix vid_out_buf_prepare()'.

The following changes since commit 337e90ed028643c7acdfd0d31e3224d05ca03d66:

  media: imx-csi: Input connections to CSI should be optional (2019-01-21 16:46:02 -0200)

are available in the Git repository at:

  git://linuxtv.org/hverkuil/media_tree.git tags/br-v5.1i3

for you to fetch changes up to ceae9b654c11f3d3490995f563529cf53ffdf7ac:

  vivid: fix vid_out_buf_prepare() (2019-01-29 13:15:46 +0100)

----------------------------------------------------------------
Tag branch

----------------------------------------------------------------
Hans Verkuil (7):
      vb2: vb2_find_timestamp: drop restriction on buffer state
      vb2: add buf_out_validate callback
      vim2m: add buf_out_validate callback
      vivid: add buf_out_validate callback
      cedrus: add buf_out_validate callback
      vb2: check that buf_out_validate is present
      vivid: fix vid_out_buf_prepare()

Paul Kocialkowski (2):
      Revert "media: cedrus: Allow using the current dst buffer as reference"
      media: cedrus: Remove completed item from TODO list (dma-buf references)

Pawel Osciak (1):
      media: vb2: Keep dma-buf buffers mapped until they are freed

 drivers/media/common/videobuf2/videobuf2-core.c   | 33 ++++++++++++++++++++++-----------
 drivers/media/common/videobuf2/videobuf2-v4l2.c   | 20 ++++++++++++--------
 drivers/media/platform/vim2m.c                    | 27 ++++++++++++++++-----------
 drivers/media/platform/vivid/vivid-vid-out.c      | 41 +++++++++++++++++++++++++++--------------
 drivers/staging/media/sunxi/cedrus/TODO           |  5 -----
 drivers/staging/media/sunxi/cedrus/cedrus_dec.c   | 13 -------------
 drivers/staging/media/sunxi/cedrus/cedrus_dec.h   |  2 --
 drivers/staging/media/sunxi/cedrus/cedrus_mpeg2.c | 10 ++++------
 drivers/staging/media/sunxi/cedrus/cedrus_video.c |  9 +++++++++
 include/media/videobuf2-core.h                    |  5 +++++
 include/media/videobuf2-v4l2.h                    |  3 +--
 11 files changed, 96 insertions(+), 72 deletions(-)