From: Boris Brezillon <boris.brezillon@collabora.com>
To: Mauro Carvalho Chehab <mchehab@kernel.org>,
Hans Verkuil <hans.verkuil@cisco.com>,
Laurent Pinchart <laurent.pinchart@ideasonboard.com>,
Sakari Ailus <sakari.ailus@iki.fi>,
linux-media@vger.kernel.org
Cc: linux-kernel@vger.kernel.org, Tomasz Figa <tfiga@chromium.org>,
Nicolas Dufresne <nicolas@ndufresne.ca>,
kernel@collabora.com,
Paul Kocialkowski <paul.kocialkowski@bootlin.com>,
Ezequiel Garcia <ezequiel@collabora.com>,
Jonas Karlman <jonas@kwiboo.se>,
linux-rockchip@lists.infradead.org,
Heiko Stuebner <heiko@sntech.de>,
Andrew Morton <akpm@linux-foundation.org>,
Rasmus Villemoes <linux@rasmusvillemoes.dk>,
Philipp Zabel <p.zabel@pengutronix.de>,
Boris Brezillon <boris.brezillon@collabora.com>
Subject: [PATCH 0/9] media: hantro: Add support for H264 decoding
Date: Wed, 19 Jun 2019 14:15:31 +0200 [thread overview]
Message-ID: <20190619121540.29320-1-boris.brezillon@collabora.com> (raw)
Hello,
This patch series adds support H264 decoding support to the hantro
driver and doing some consilidation cleanup in the driver along the
way.
Some details about the patches forming this patchset:
* The first patch is adding support for the sort_r() variant and has
been posted separately by Rasmus. I put it back there because Andrew
told me to repost it with the patch series using this new variant.
As mentioned in the patch itself, I'd like this patch to be merged
as soon as possible to avoid the synchronisation burden that might
appear if we decide to delay it.
* Patch 2 is needed to properly propagate the output buf timestamp to
the capture buf one, which is required for intra-frame references.
* Patches 3 to 6 are consolidating the code by providing helpers that
can be used by all hantro backend and simplifying the ctrl
initialization logic. We also constify the controls array.
* Patches 7 to 8 are adding common H264 decoding bits and patch 9 is
enabling H264 decoding on rk3288
Now, a few words about the dependencies. Unfortunately there are a lot,
and that'd be great to have some of them merged.
* This series is based on top of Ezequiel's VP8 work [1].
* It depends on [2] which defines/described the H264 decoding mode
control.
* Depends on [3] since I'm using vb2_get_buffer() to retrieve a
reference buffer
* The final dep is a fix I sent this morning allowing me to simplify the
ctrl initialization logic
Regards,
Boris
Boris Brezillon (5):
media: hantro: Move copy_metadata() before doing a decode operation
media: hantro: Constify the control array
media: hantro: Simplify the controls creation logic
media: hantro: Add hantro_get_{src,dst}_buf() helpers
media: hantro: Add helpers to prepare/finish a run
Hertz Wong (3):
media: hantro: Add core bits to support H264 decoding
media: hantro: Add support for H264 decoding on G1
media: hantro: Enable H264 decoding on rk3288
Rasmus Villemoes (1):
lib/sort.c: implement sort() variant taking context argument
drivers/staging/media/hantro/Makefile | 2 +
drivers/staging/media/hantro/hantro.h | 24 +-
drivers/staging/media/hantro/hantro_drv.c | 95 ++-
.../staging/media/hantro/hantro_g1_h264_dec.c | 295 ++++++++
.../media/hantro/hantro_g1_mpeg2_dec.c | 14 +-
.../staging/media/hantro/hantro_g1_vp8_dec.c | 17 +-
.../staging/media/hantro/hantro_h1_jpeg_enc.c | 11 +-
drivers/staging/media/hantro/hantro_h264.c | 638 ++++++++++++++++++
drivers/staging/media/hantro/hantro_hw.h | 55 ++
drivers/staging/media/hantro/hantro_v4l2.c | 15 +-
drivers/staging/media/hantro/rk3288_vpu_hw.c | 21 +-
.../media/hantro/rk3399_vpu_hw_jpeg_enc.c | 12 +-
.../media/hantro/rk3399_vpu_hw_mpeg2_dec.c | 14 +-
include/linux/sort.h | 5 +
lib/sort.c | 34 +-
15 files changed, 1175 insertions(+), 77 deletions(-)
create mode 100644 drivers/staging/media/hantro/hantro_g1_h264_dec.c
create mode 100644 drivers/staging/media/hantro/hantro_h264.c
--
2.20.1
next reply other threads:[~2019-06-19 12:16 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-06-19 12:15 Boris Brezillon [this message]
2019-06-19 12:15 ` [PATCH 1/9] lib/sort.c: implement sort() variant taking context argument Boris Brezillon
2019-07-25 12:40 ` Boris Brezillon
2019-07-26 0:05 ` Andrew Morton
2019-07-26 6:59 ` Rasmus Villemoes
2019-06-19 12:15 ` [PATCH 2/9] media: hantro: Move copy_metadata() before doing a decode operation Boris Brezillon
2019-06-19 12:15 ` [PATCH 3/9] media: hantro: Constify the control array Boris Brezillon
2019-07-05 16:05 ` Ezequiel Garcia
2019-07-05 17:17 ` Boris Brezillon
2019-06-19 12:15 ` [PATCH 4/9] media: hantro: Simplify the controls creation logic Boris Brezillon
2019-06-19 12:15 ` [PATCH 5/9] media: hantro: Add hantro_get_{src,dst}_buf() helpers Boris Brezillon
2019-06-19 12:15 ` [PATCH 6/9] media: hantro: Add helpers to prepare/finish a run Boris Brezillon
2019-06-19 12:15 ` [PATCH 7/9] media: hantro: Add core bits to support H264 decoding Boris Brezillon
2019-07-25 13:31 ` Rasmus Villemoes
2019-07-26 10:06 ` Boris Brezillon
2019-08-01 4:06 ` Tomasz Figa
2019-08-01 5:42 ` Boris Brezillon
2019-08-01 7:04 ` Paul Kocialkowski
2019-08-01 7:12 ` Tomasz Figa
2019-08-05 18:38 ` Ezequiel Garcia
2019-06-19 12:15 ` [PATCH 8/9] media: hantro: Add support for H264 decoding on G1 Boris Brezillon
2019-06-19 12:15 ` [PATCH 9/9] media: hantro: Enable H264 decoding on rk3288 Boris Brezillon
2019-06-19 12:23 ` [PATCH 0/9] media: hantro: Add support for H264 decoding Boris Brezillon
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=20190619121540.29320-1-boris.brezillon@collabora.com \
--to=boris.brezillon@collabora.com \
--cc=akpm@linux-foundation.org \
--cc=ezequiel@collabora.com \
--cc=hans.verkuil@cisco.com \
--cc=heiko@sntech.de \
--cc=jonas@kwiboo.se \
--cc=kernel@collabora.com \
--cc=laurent.pinchart@ideasonboard.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-media@vger.kernel.org \
--cc=linux-rockchip@lists.infradead.org \
--cc=linux@rasmusvillemoes.dk \
--cc=mchehab@kernel.org \
--cc=nicolas@ndufresne.ca \
--cc=p.zabel@pengutronix.de \
--cc=paul.kocialkowski@bootlin.com \
--cc=sakari.ailus@iki.fi \
--cc=tfiga@chromium.org \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).