All of lore.kernel.org
 help / color / mirror / Atom feed
From: Hans Verkuil <hverkuil@xs4all.nl>
To: Linux Media Mailing List <linux-media@vger.kernel.org>
Cc: Paul Kocialkowski <paul.kocialkowski@bootlin.com>
Subject: [GIT PULL FOR v5.1] vb2 + cedrus fixes/improvements
Date: Fri, 25 Jan 2019 16:29:10 +0100	[thread overview]
Message-ID: <c861f137-c626-090d-b853-5a5c39b50a51@xs4all.nl> (raw)

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 final 5 patches fix the sole remaining v4l2-compliance bug vivid and
vimc for output field handling in requests.

Regards,

	Hans

This PR supersedes https://patchwork.linuxtv.org/patch/54136/, updating
the last patch with a newer version: https://patchwork.linuxtv.org/patch/54167/

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.1i2

for you to fetch changes up to a50568612b16fd5e42375f2f31a123ff8bc51b60:

  vb2: check that buf_out_validate is present (2019-01-25 16:24:45 +0100)

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

----------------------------------------------------------------
Hans Verkuil (6):
      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

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      | 23 ++++++++++++++++-------
 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, 85 insertions(+), 65 deletions(-)

             reply	other threads:[~2019-01-25 15:29 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-01-25 15:29 Hans Verkuil [this message]
  -- strict thread matches above, loose matches on Subject: below --
2019-01-29 12:19 [GIT PULL FOR v5.1] vb2 + cedrus fixes/improvements Hans Verkuil
2019-01-24 14:40 Hans Verkuil

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=c861f137-c626-090d-b853-5a5c39b50a51@xs4all.nl \
    --to=hverkuil@xs4all.nl \
    --cc=linux-media@vger.kernel.org \
    --cc=paul.kocialkowski@bootlin.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.