From: Hans Verkuil <hverkuil@xs4all.nl>
To: Boris Brezillon <boris.brezillon@collabora.com>,
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: Rob Herring <robh+dt@kernel.org>,
Mark Rutland <mark.rutland@arm.com>,
devicetree@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>
Subject: Re: [RESEND PATCH v2 0/4] media: rockchip: Add the rkvdec driver
Date: Fri, 11 Oct 2019 11:43:20 +0200 [thread overview]
Message-ID: <ea218a89-d844-4c56-595a-7359af3ef031@xs4all.nl> (raw)
In-Reply-To: <20191011093342.3471-1-boris.brezillon@collabora.com>
On 10/11/19 11:33 AM, Boris Brezillon wrote:
> Hello,
>
> Sorry for the noise, I forgot to Cc the DT maintainers/ML on my last
> attempt.
>
> This is v2 of the rkvdec driver which was initially posted as part of
> my RFC adding codec helpers.
> For this version, I decided to get rid of the helper stuff which is
> likely to take some more time to settle. The patchset now contains a
> proper DT binding doc.
>
> Note that this implementation is likely to be adjusted based on the
> outcome of the 'scaling matrix order' [1] and 'dynamic resolution
> changes' [2] discussions, but I thought it'd be worth posting this
> version to start getting feeback.
I assume this version of the driver does not (yet?) support slicing?
Core slicing support is part of this series:
https://patchwork.linuxtv.org/cover/59415/
Regards,
Hans
>
> Regards,
>
> Boris
>
> [1]https://patchwork.kernel.org/cover/11125207/
> [2]https://lkml.org/lkml/2019/9/3/1033
>
> Boris Brezillon (4):
> media: vb2: Add a helper to get the vb2 buffer attached to a request
> media: dt-bindings: rockchip: Document RK3399 Video Decoder bindings
> media: rockchip: Add the rkvdec driver
> arm64: dts: rockchip: rk3399: Define the rockchip Video Decoder node
>
> .../bindings/media/rockchip,vdec.yaml | 71 ++
> arch/arm64/boot/dts/rockchip/rk3399.dtsi | 14 +-
> .../media/common/videobuf2/videobuf2-core.c | 23 +
> drivers/staging/media/Kconfig | 2 +
> drivers/staging/media/Makefile | 1 +
> drivers/staging/media/rockchip/Kconfig | 16 +
> drivers/staging/media/rockchip/Makefile | 2 +
> drivers/staging/media/rockchip/vdec/Kconfig | 14 +
> drivers/staging/media/rockchip/vdec/Makefile | 3 +
> .../staging/media/rockchip/vdec/rkvdec-h264.c | 960 +++++++++++++++
> .../staging/media/rockchip/vdec/rkvdec-regs.h | 306 +++++
> drivers/staging/media/rockchip/vdec/rkvdec.c | 1089 +++++++++++++++++
> drivers/staging/media/rockchip/vdec/rkvdec.h | 131 ++
> include/media/videobuf2-core.h | 11 +
> 14 files changed, 2642 insertions(+), 1 deletion(-)
> create mode 100644 Documentation/devicetree/bindings/media/rockchip,vdec.yaml
> create mode 100644 drivers/staging/media/rockchip/Kconfig
> create mode 100644 drivers/staging/media/rockchip/Makefile
> create mode 100644 drivers/staging/media/rockchip/vdec/Kconfig
> create mode 100644 drivers/staging/media/rockchip/vdec/Makefile
> create mode 100644 drivers/staging/media/rockchip/vdec/rkvdec-h264.c
> create mode 100644 drivers/staging/media/rockchip/vdec/rkvdec-regs.h
> create mode 100644 drivers/staging/media/rockchip/vdec/rkvdec.c
> create mode 100644 drivers/staging/media/rockchip/vdec/rkvdec.h
>
next prev parent reply other threads:[~2019-10-11 9:43 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-10-11 9:33 [RESEND PATCH v2 0/4] media: rockchip: Add the rkvdec driver Boris Brezillon
2019-10-11 9:33 ` [RESEND PATCH v2 1/4] media: vb2: Add a helper to get the vb2 buffer attached to a request Boris Brezillon
2019-10-11 9:33 ` [RESEND PATCH v2 2/4] media: dt-bindings: rockchip: Document RK3399 Video Decoder bindings Boris Brezillon
2019-10-11 12:50 ` Rob Herring
2019-10-11 12:56 ` Boris Brezillon
2019-10-11 17:37 ` Rob Herring
2019-10-11 9:33 ` [RESEND PATCH v2 3/4] media: rockchip: Add the rkvdec driver Boris Brezillon
2019-10-11 10:06 ` Hans Verkuil
2019-10-11 11:15 ` Boris Brezillon
2019-10-11 11:21 ` Hans Verkuil
2019-10-13 19:08 ` kbuild test robot
2019-10-13 19:08 ` [RFC PATCH] media: rockchip: rkvdec_queue_ops can be static kbuild test robot
2019-10-11 9:33 ` [RESEND PATCH v2 4/4] arm64: dts: rockchip: rk3399: Define the rockchip Video Decoder node Boris Brezillon
2019-10-11 9:43 ` Hans Verkuil [this message]
2019-10-11 9:53 ` [RESEND PATCH v2 0/4] media: rockchip: Add the rkvdec driver 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=ea218a89-d844-4c56-595a-7359af3ef031@xs4all.nl \
--to=hverkuil@xs4all.nl \
--cc=boris.brezillon@collabora.com \
--cc=devicetree@vger.kernel.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-media@vger.kernel.org \
--cc=linux-rockchip@lists.infradead.org \
--cc=mark.rutland@arm.com \
--cc=mchehab@kernel.org \
--cc=nicolas@ndufresne.ca \
--cc=paul.kocialkowski@bootlin.com \
--cc=robh+dt@kernel.org \
--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).