On Fri, 11 Oct 2019 07:50:16 -0500
Rob Herring wrote:
> On Fri, Oct 11, 2019 at 4:33 AM Boris Brezillon
> wrote:
> >
> > Document the Rockchip RK3399 Video Decoder bindings.
>
> You need some more headers for the example (or drop defines):
>
> Error:
>
Hi Hans,
On Fri, 11 Oct 2019 12:06:35 +0200
Hans Verkuil wrote:
> > diff --git a/drivers/staging/media/rockchip/Kconfig
> > b/drivers/staging/media/rockchip/Kconfig
> > new file mode 100644
> > index ..8c617ae2c84f
> > --- /dev/null
> > +++ b/drivers/staging/media/rockchip/Kconfig
>
On Fri, 11 Oct 2019 11:43:20 +0200
Hans Verkuil wrote:
> 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
The rockchip vdec block is a stateless decoder that's able to decode
H264, HEVC and VP9 content. This patch only adds H264 support and is
making use of the m2m codec helpers.
Signed-off-by: Boris Brezillon
---
drivers/staging/media/Kconfig |2 +
drivers/staging/
feeback.
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: rockch
RK3399 has a Video decoder, define the node in the dtsi. We also add
the missing power-domain in mmu node and enable the block.
Signed-off-by: Boris Brezillon
---
arch/arm64/boot/dts/rockchip/rk3399.dtsi | 14 +-
1 file changed, 13 insertions(+), 1 deletion(-)
diff --git a/arch
vb2_request_get_buf() returns the N-th buffer attached to a media
request.
Signed-off-by: Boris Brezillon
---
Changes in v2:
* Adjust the kernel doc as suggested by Hans
---
.../media/common/videobuf2/videobuf2-core.c | 23 +++
include/media/videobuf2-core.h
Document the Rockchip RK3399 Video Decoder bindings.
Signed-off-by: Boris Brezillon
---
.../bindings/media/rockchip,vdec.yaml | 71 +++
1 file changed, 71 insertions(+)
create mode 100644 Documentation/devicetree/bindings/media/rockchip,vdec.yaml
diff --git a
RK3399 has a Video decoder, define the node in the dtsi. We also add
the missing power-domain in mmu node and enable the block.
Signed-off-by: Boris Brezillon
---
arch/arm64/boot/dts/rockchip/rk3399.dtsi | 14 +-
1 file changed, 13 insertions(+), 1 deletion(-)
diff --git a/arch
The rockchip vdec block is a stateless decoder that's able to decode
H264, HEVC and VP9 content. This patch only adds H264 support and is
making use of the m2m codec helpers.
Signed-off-by: Boris Brezillon
---
drivers/staging/media/Kconfig |2 +
drivers/staging/
Document the Rockchip RK3399 Video Decoder bindings.
Signed-off-by: Boris Brezillon
---
.../bindings/media/rockchip,vdec.yaml | 71 +++
1 file changed, 71 insertions(+)
create mode 100644 Documentation/devicetree/bindings/media/rockchip,vdec.yaml
diff --git a
ttps://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 De
vb2_request_get_buf() returns the N-th buffer attached to a media
request.
Signed-off-by: Boris Brezillon
---
Changes in v2:
* Adjust the kernel doc as suggested by Hans
---
.../media/common/videobuf2/videobuf2-core.c | 23 +++
include/media/videobuf2-core.h
On Tue, 8 Oct 2019 11:11:13 +0200
Boris Brezillon wrote:
Oops, forgot to update the subject. Should be:
"[RFC PATCH v3 0/6] media: v4l2: Add extended fmt and buffer ioctls"
> Hello,
>
> This RFC follows the discussion started by Hans [1] a few months back.
> It does
This should simplify things a bit as we now have a single
implementation instead of the MPLANE and !MPLANE versions.
Signed-off-by: Boris Brezillon
---
Changes in v3:
- Rebased on top of media/master (post 5.4-rc1)
Changes in v2:
- New patch
---
drivers/media/platform/vivid/vivid-core.c
-by: Boris Brezillon
---
Changes in v3:
- Rebased on top of media/master (post 5.4-rc1)
Changes in v2:
- New patch
---
.../media/common/videobuf2/videobuf2-core.c | 2 +
.../media/common/videobuf2/videobuf2-v4l2.c | 534 +++---
include/media/videobuf2-core.h| 6
bbrezillon/v4l-utils/commits/master
[3]https://github.com/bbrezillon/linux/commit/4882435f80b05a61827649d55cc0f0cee79680a7
[4]https://github.com/bbrezillon/linux/commit/a415216c6aaab2d51f0bd62270b994c8196ddd90
Boris Brezillon (5):
media: v4l2: Extend pixel formats to unify single/multi-planar
handling (and mo
-off-by: Boris Brezillon
---
Changes in v3:
- Rebased on top of media/master (post 5.4-rc1)
Changes in v2:
- Move the modifier in v4l2_ext_format (was formerly placed in
v4l2_ext_plane)
- Fix a few bugs in the converters and add a strict parameter to
allow conversion of uninitialized/mis
support the new hooks, and vice versa.
Note that the timecode field is gone, since there doesn't seem to be
in-kernel users, but can be added back in the reserved area if needed.
Signed-off-by: Hans Verkuil
Signed-off-by: Boris Brezillon
---
Changes in v3:
- Rebased on top of media/master (pos
Convert the driver to the _ext_fmt and _ext_buf API.
Signed-off-by: Boris Brezillon
---
Changes in v3:
- Rebased on top of media/master (post 5.4-rc1)
Changes in v2:
- New patch
---
drivers/media/platform/vimc/vimc-capture.c | 65 +++---
drivers/media/platform/vimc/vimc
Just a new version of v4l2_fill_mbus_format() to deal with the new
v4l2_ext_pix_format struct.
This is needed to convert the VIMC driver to the EXT_FMT/EXT_BUF iocts.
Signed-off-by: Boris Brezillon
---
Changes in v3:
- Rebased on top of media/master (post 5.4-rc1)
Changes in v2:
- New patch
t;
> All sessions are in room Terreaux VIP Lounge - Level 0.
> There is a maximum of 15 people.
>
> The second session deals with libcamera and is on Wednesday morning
> from 9:00 to 13:00.
>
> Attendees for this session:
>
> Kieran Bingham
> Boris Brezillon
I ha
On Mon, 23 Sep 2019 13:41:07 +0200
Hans Verkuil wrote:
> Hi Boris,
>
> On 4/4/19 10:16 AM, Boris Brezillon wrote:
> > Hello,
> >
> > This RFC follows the discussion started by Hans [1] a few months back.
> > It does not try to address all the problem report
On Mon, 9 Sep 2019 09:28:15 +0200
Boris Brezillon wrote:
> Step '8.2.4.1 Decoding process for picture numbers' was missing in the
> reflist creation logic, leading to invalid P reflists when a
> ->frame_num wraparound happens.
>
> Fixes: a9471e25629b ("media:
And use it for all native type comparisons, even if it's not strictly
required. By doing that we make the code more consistent and prevent
from potential incorrect results in case of overflow or when the the
values being compared are both negative.
Signed-off-by: Boris Brezillon
---
dr
So it matches the code and the spec.
Signed-off-by: Boris Brezillon
---
drivers/staging/media/hantro/hantro_h264.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/media/hantro/hantro_h264.c
b/drivers/staging/media/hantro/hantro_h264.c
index 0d758e0c0f99
Step '8.2.4.1 Decoding process for picture numbers' was missing in the
reflist creation logic, leading to invalid P reflists when a
->frame_num wraparound happens.
Fixes: a9471e25629b ("media: hantro: Add core bits to support H264 decoding")
Reported-by: Francois Buergisse
On Mon, 5 Aug 2019 13:53:20 -0300
Hans Verkuil wrote:
> > +/**
> > + * struct v4l2_m2m_codec_ctx - Codec context
> > + * @fh: file handle
> > + * @coded_fmt: current coded format
> > + * @decoded_fmt: current decoded format
> > + * @coded_fmt_desc: current coded format desc
> > + * @decoded_fmt_d
On Mon, 5 Aug 2019 10:12:53 -0300
Hans Verkuil wrote:
> On 8/5/19 6:48 AM, Boris Brezillon wrote:
> > vb2_request_get_buf() returns the N-th buffer attached to a media
> > request.
> >
> > Signed-off-by: Boris Brezillon
> > ---
> > .../media/c
Signed-off-by: Boris Brezillon
---
drivers/media/v4l2-core/Kconfig | 5 +
drivers/media/v4l2-core/Makefile | 1 +
.../media/v4l2-core/v4l2-mem2mem-h264-codec.c | 47
include/media/v4l2-mem2mem-h264-codec.h | 100 ++
4 files changed
objects to the v4l2-mem2mem.o object.
Signed-off-by: Boris Brezillon
---
drivers/media/v4l2-core/Makefile| 1 +
drivers/media/v4l2-core/{v4l2-mem2mem.c => v4l2-mem2mem-core.c} | 0
2 files changed, 1 insertion(+)
rename drivers/media/v4l2-core/{v4l2-mem2me
Most codec drivers do the same thing, let's extract those generic bits
and provide them as m2m_codec helpers.
Each compressed-format will have its own set of sub-helpers.
Signed-off-by: Boris Brezillon
---
drivers/media/v4l2-core/Kconfig |4 +
drivers/media/v4l2-core/Mak
vb2_request_get_buf() returns the N-th buffer attached to a media
request.
Signed-off-by: Boris Brezillon
---
.../media/common/videobuf2/videobuf2-core.c | 23 +++
include/media/videobuf2-core.h| 11 +
2 files changed, 34 insertions(+)
diff --git a
04013/
[2]https://patchwork.kernel.org/project/linux-media/list/?series=129567
Boris Brezillon (5):
media: vb2: Add a helper to get the vb2 buffer attached to a request
media: v4l2: Prepare things for addition of m2m codec helpers
media: v4l2: Add m2m codec helpers
media: v4l2: Provide h
The rockchip vdec block is a stateless decoder that's able to decode
H264, HEVC and VP9 content. This patch only adds H264 support and is
making use of the m2m codec helpers.
Signed-off-by: Boris Brezillon
---
drivers/staging/media/Kconfig | 2 +
drivers/staging/media/Mak
On Mon, 29 Jul 2019 15:25:21 +0200
Paul Kocialkowski wrote:
> Hi,
>
> On Sat 27 Jul 19, 11:46, Boris Brezillon wrote:
> > On Sat, 27 Jul 2019 11:27:43 +0200
> > Paul Kocialkowski wrote:
> >
> > > Hi,
> > >
> > > On Fri 26 Jul 19, 1
On Sat, 27 Jul 2019 09:52:24 -0300
Ezequiel Garcia wrote:
> > >
> > > That's not my understanding of the Annex B section (quoting the spec
> > > for reference):
> > >
> > > "
> > > The byte stream format consists of a sequence of byte stream NAL unit
> > > syntax structures. Each byte stream N
On Sat, 27 Jul 2019 11:27:43 +0200
Paul Kocialkowski wrote:
> Hi,
>
> On Fri 26 Jul 19, 10:53, Hans Verkuil wrote:
> > On 7/26/19 9:30 AM, Boris Brezillon wrote:
> > > On Fri, 26 Jul 2019 08:28:28 +0200
> > > Boris Brezillon wrote:
> > >
On Fri, 26 Jul 2019 08:28:28 +0200
Boris Brezillon wrote:
> On Thu, 25 Jul 2019 23:39:11 -0300
> Ezequiel Garcia wrote:
>
> > On Thu, 2019-07-25 at 21:36 +0200, Paul Kocialkowski wrote:
> > > Hi,
> > >
> > > On Thu 25 Jul 19, 08:42, Boris Brezillo
On Thu, 25 Jul 2019 23:39:11 -0300
Ezequiel Garcia wrote:
> On Thu, 2019-07-25 at 21:36 +0200, Paul Kocialkowski wrote:
> > Hi,
> >
> > On Thu 25 Jul 19, 08:42, Boris Brezillon wrote:
> > > On Fri, 5 Jul 2019 19:16:18 +0200
> > > Boris Brezillon wrote:
On Fri, 5 Jul 2019 19:16:18 +0200
Boris Brezillon wrote:
> On Fri, 05 Jul 2019 13:40:03 -0300
> Ezequiel Garcia wrote:
>
> > Hi Boris, Paul,
> >
> > On Wed, 2019-07-03 at 14:28 +0200, Boris Brezillon wrote:
> > > Looks like some stateless decode
On Wed, 24 Jul 2019 12:39:55 +0900
Tomasz Figa wrote:
> Hi Boris,
>
> On Wed, Jul 3, 2019 at 9:28 PM Boris Brezillon
> wrote:
> >
> > Those lists can be extracted from the dpb, let's simplify userspace
> > life and build that list kernel-side (generic helpe
Hi Hans,
On Mon, 22 Jul 2019 17:29:21 +0200
Hans Verkuil wrote:
> On 7/3/19 2:28 PM, Boris Brezillon wrote:
> > Some stateless decoders don't support per-slice decoding (or at least
> > not in a way that would make them efficient or easy to use).
> > Let's expo
On Fri, 05 Jul 2019 13:40:03 -0300
Ezequiel Garcia wrote:
> Hi Boris, Paul,
>
> On Wed, 2019-07-03 at 14:28 +0200, Boris Brezillon wrote:
> > Looks like some stateless decoders expect slices to be prefixed with
> > ANNEX B start codes (they most likely do some kind of bits
ned-off-by: Boris Brezillon
Reviewed-by: Paul Kocialkowski
---
Changes in v3:
* s/per-{slice,frame} decoding/{slice,frame}-based decoding/
* Add Paul's R-b
Changes in v2:
* Allow decoding multiple slices in per-slice decoding mode
* Minor doc improvement/fixes
---
.../media/uapi/v4l/ext-
Those lists can be extracted from the dpb, let's simplify userspace
life and build that list kernel-side (generic helpers will be provided
for drivers that need this list).
Signed-off-by: Boris Brezillon
---
Changes in v3:
* None
Changes in v2:
* None
---
Documentation/media/uapi/v4l/ext-
to be passed through controls) should be pretty
easy, let's mandate that all slices be prepended with ANNEX B start
codes.
If we ever need to support AVC headers, we can add a new menu control
to select the type of NAL header to use.
Signed-off-by: Boris Brezillon
Reviewed-by: Paul Kocialk
R-b on patch 1 and 2
Changes in v2:
* Allow decoding multiple slices in per-slice decoding mode
* Minor doc improvements/fixes
Boris Brezillon (3):
media: uapi: h264: Clarify our expectations regarding NAL header
format
media: uapi: h264: Add the concept of decoding mode
media: u
On Wed, 26 Jun 2019 13:30:38 +0200
Paul Kocialkowski wrote:
> Hi,
>
> On Mon, 2019-06-10 at 10:52 +0200, Boris Brezillon wrote:
> > Some stateless decoders don't support per-slice decoding (or at least
> > not in a way that would make them efficient or easy to use).
On Wed, 26 Jun 2019 13:33:41 +0200
Paul Kocialkowski wrote:
> Hi,
>
> On Mon, 2019-06-10 at 10:52 +0200, Boris Brezillon wrote:
> > Those lists can be extracted from the dpb, let's simplify userspace
> > life and build that list kernel-side (generic helpers will be pro
cfg->type can be overridden by v4l2_ctrl_fill() and the new value is
stored in the local type var. Fix the tests to use this local var.
Fixes: 0996517cf8ea ("V4L/DVB: v4l2: Add new control handling framework")
Cc:
Signed-off-by: Boris Brezillon
---
drivers/media/v4l2-core/v4l
On Thu, 13 Jun 2019 12:10:40 -0300
Ezequiel Garcia wrote:
> +static void cfg_parts(struct hantro_ctx *ctx,
> + const struct v4l2_ctrl_vp8_frame_header *hdr)
> +{
> + struct hantro_dev *vpu = ctx->dev;
> + struct vb2_v4l2_buffer *vb2_src;
> + u32 first_part_offset =
On Thu, 13 Jun 2019 12:10:39 -0300
Ezequiel Garcia wrote:
> From: Pawel Osciak
>
> Add the parsed VP8 frame pixel format and controls, to be used
> with the new stateless decoder API for VP8 to provide parameters
> for accelerator (aka stateless) codecs.
>
> Signed-off-by: Pawel Osciak
> Sign
On Wed, 12 Jun 2019 10:00:45 +0200
Hans Verkuil wrote:
> On 6/12/19 9:55 AM, Hans Verkuil wrote:
> > On 6/11/19 2:50 PM, Philipp Zabel wrote:
> >> There are several other SoCs that contain Hantro IP based VPUs, such as
> >> NXP i.MX8MQ (Hantro G1 and G2) and i.MX8MM (Hantro G1, G2, and H1). To
On Wed, 12 Jun 2019 08:14:35 +
Jonas Karlman wrote:
> On 2019-06-11 14:50, Philipp Zabel wrote:
> > Rename the driver and all relevant identifiers from Rockchip to Hantro,
> > as other Hantro IP based VPU implementations can be supported by the
> > same driver.
> > The RK3288 decoder is Hantr
On Mon, 10 Jun 2019 10:52:47 +0200
Boris Brezillon wrote:
> Hello,
>
> This is a first batch of adjustments to the stateless H264 decoder
> uAPI. The first one is about adding support for per-frame decoding,
> which is the only mode supported on some codecs (the hantro G1 block
Those lists can be extracted from the dpb, let's simplify userspace
life and build that list kernel-side (generic helpers will be provided
for drivers that need this list).
Signed-off-by: Boris Brezillon
---
Changes in v2:
* None
---
Documentation/media/uapi/v4l/ext-ctrls-codec.rs
ned-off-by: Boris Brezillon
---
Changes in v2:
* Allow decoding multiple slices in per-slice decoding mode
* Minor doc improvement/fixes
---
.../media/uapi/v4l/ext-ctrls-codec.rst| 46 ++-
drivers/media/v4l2-core/v4l2-ctrls.c | 9
include/media/h2
to be passed through controls) should be pretty
easy, let's mandate that all slices be prepended with ANNEX B start
codes.
If we ever need to support AVC headers, we can add a new menu control
to select the type of NAL header to use.
Signed-off-by: Boris Brezillon
---
Changes in v2:
* None
n specific. What's provided by the bitstream is a
list of references that will be used to decode a frame
* ... (add your own)
Feel free to comment on these changes and/or propose alternatives.
Regards,
Boris
Changes in v2:
* Allow decoding multiple slices in per-slice decoding mode
* Minor doc
On Fri, 7 Jun 2019 16:56:09 +0300
Dan Carpenter wrote:
> Thus the address of "&ctx->dev->variant->codec_ops[codec_mode]"
> can't possibly be NULL.
>
> Signed-off-by: Dan Carpenter
Reviewed-by: Boris Brezillon
> ---
> drivers/staging/media/roc
On Thu, 06 Jun 2019 15:13:00 -0300
Ezequiel Garcia wrote:
> On Thu, 2019-06-06 at 19:43 +0200, Boris Brezillon wrote:
> > On Thu, 6 Jun 2019 12:44:21 -0300
> > Ezequiel Garcia wrote:
> >
> > > Hi,
> > >
> > > This patchset introduces a new v
On Thu, 6 Jun 2019 12:44:21 -0300
Ezequiel Garcia wrote:
> Hi,
>
> This patchset introduces a new vb2_get_buffer accessor and then
> uses it on all the drivers that are accessing videobuf2
> private buffer array directly.
Just curious, how did you find all occurrences of direct q->bufs[]
acces
ideration.
> + */
> +static inline struct vb2_buffer *vb2_get_buffer(struct vb2_queue *q,
> + unsigned int index)
> +{
> + if (q->num_buffers > 0 && index < q->num_buffers)
No need to check q->num_buffers > 0 because in that case
i
f-by: Ezequiel Garcia
Reviewed-by: Boris Brezillon
> ---
> Changes from v2:
> * Align parameters to parenthesis
> * Drop unneeded zero initialization
>
> Changes from v1:
> * Drop the s/break/return replacements
> * Drop unneeded default cases
> * Fix memset to take account of
default.
>
> Signed-off-by: Ezequiel Garcia
Reviewed-by: Boris Brezillon
> ---
> Changes from v2:
> * Initialize the controls to zero, instead of default.
>
> Changes from v1:
> * No change.
> ---
> drivers/media/v4l2-core/v4l2-ctrls.c | 4
> 1 file cha
On Wed, 5 Jun 2019 13:22:42 +0200
Boris Brezillon wrote:
> On Fri, 31 May 2019 10:55:14 +0200
> Philipp Zabel wrote:
>
> > Rename the driver and all relevant identifiers from Rockchip to Hantro,
> > as other Hantro IP based VPU implementations can be supported by the
>
On Thu, 6 Jun 2019 08:51:59 +0200
Boris Brezillon wrote:
> On Thu, 6 Jun 2019 12:53:57 +0900
> Tomasz Figa wrote:
>
> > On Thu, Jun 6, 2019 at 1:46 AM Boris Brezillon
> > wrote:
> > >
> > > CAP_M2M_MPLANE means the device supports _MPLANE formats fo
e:
> > > > Le lundi 03 juin 2019 à 23:48 +0200, Jernej Škrabec a écrit :
> > > > > Dne ponedeljek, 03. junij 2019 ob 13:09:45 CEST je Boris Brezillon
> > > > > napisal(a):
> > > > > > The driver only supports per-slice decod
On Wed, 05 Jun 2019 22:57:21 +0200
Paul Kocialkowski wrote:
> Hi,
>
> Le lundi 03 juin 2019 à 13:09 +0200, Boris Brezillon a écrit :
> > We are about to add a menu control, so let's make the code more generic
> > to support other control types.
>
> Did y
On Wed, 05 Jun 2019 22:48:08 +0200
Paul Kocialkowski wrote:
> Hi,
>
> Le mardi 04 juin 2019 à 10:16 +0200, Thierry Reding a écrit :
> > On Mon, Jun 03, 2019 at 05:37:11PM +0200, Boris Brezillon wrote:
> > > On Mon, 3 Jun 2019 16:05:26 +0200
> > > Thierry Redin
On Thu, 6 Jun 2019 12:53:57 +0900
Tomasz Figa wrote:
> On Thu, Jun 6, 2019 at 1:46 AM Boris Brezillon
> wrote:
> >
> > CAP_M2M_MPLANE means the device supports _MPLANE formats for both
> > capture and output. Adjust the check to avoid EINVAL errors on
> &g
CAP_M2M_MPLANE means the device supports _MPLANE formats for both
capture and output. Adjust the check to avoid EINVAL errors on
such devices.
Fixes: 366c719d6479 ("media: v4l2: Get rid of
->vidioc_enum_fmt_vid_{cap,out}_mplane")
Reported-by: Maxime Jourdan
Signed-off-by: Bo
1.
>
> This patch just renames, no functional changes.
>
> Signed-off-by: Philipp Zabel
> Reviewed-by: Boris Brezillon
> ---
> MAINTAINERS | 4 +-
> drivers/staging/media/Kconfig | 2 +-
> drivers/staging/media/Makefi
On Tue, 4 Jun 2019 09:06:25 +0200
Boris Brezillon wrote:
> Support for multiplanar and singleplanar formats is mutually exclusive,
> at least in practice. In our attempt to unify support for support for
s/support for support for/support for/
> m
On Tue, 4 Jun 2019 09:15:28 +
Jonas Karlman wrote:
> On 2019-06-04 11:06, Thierry Reding wrote:
> > On Tue, Jun 04, 2019 at 10:49:21AM +0200, Boris Brezillon wrote:
> >> On Tue, 4 Jun 2019 10:31:57 +0200
> >> Thierry Reding wrote:
> >>
> >&g
On Tue, 4 Jun 2019 10:55:03 +0200
Thierry Reding wrote:
> On Mon, Jun 03, 2019 at 02:52:44PM -0400, Nicolas Dufresne wrote:
> [...]
> > There is one thing that come up though, if we enable per-frame decoding
> > on top of per-slice decoder (like Cedrus), won't we force userspace to
> > always com
On Tue, 4 Jun 2019 10:31:57 +0200
Thierry Reding wrote:
> > > > > - Using flags
> > > > >
> > > > > The current MPEG-2 controls have lots of u8 values that can be
> > > > > represented as flags. Using flags also helps with padding.
> > > > > It's unlikely that we'll get more than 64 flags, so us
On Tue, 4 Jun 2019 10:12:10 +0200
Thierry Reding wrote:
> On Mon, Jun 03, 2019 at 07:55:48PM -0400, Nicolas Dufresne wrote:
> > Le lundi 03 juin 2019 à 23:48 +0200, Jernej Škrabec a écrit :
> > > Dne ponedeljek, 03. junij 2019 ob 13:09:45 CEST je Boris Brezillo
On Tue, 4 Jun 2019 09:02:56 +0200
Hans Verkuil wrote:
> On 6/3/19 3:58 PM, Boris Brezillon wrote:
> > Support for multiplanar and singleplanar formats is mutually exclusive,
> > at least in practice. In our attempt to unify support for support for
> > mplane and !mplane in
ad.
Signed-off-by: Boris Brezillon
---
Changes in v6:
- Add missing parens in a test
Changes in v5:
- None
Changes in v4:
- None
Changes in v3:
- Send this patch separately (was previously part of the EXT_BUF/FMT
rework)
Changes in v2:
- None
---
drivers/media/pci/intel/ipu3/ipu3-cio2.
This is needed if we want the core to be able to check _MPLANE support
without having to call the ->vdioc_querycap() hook.
Signed-off-by: Boris Brezillon
---
Changes in v6:
- None
Changes in v5:
- Drop the fimc-lite and fimc-isp-video hack
Changes in v4:
- Add a hack in fimc-lite and fimc-
On Tue, 4 Jun 2019 09:02:56 +0200
Hans Verkuil wrote:
> On 6/3/19 3:58 PM, Boris Brezillon wrote:
> > Support for multiplanar and singleplanar formats is mutually exclusive,
> > at least in practice. In our attempt to unify support for support for
> > mplane and !mplane in
On Mon, 3 Jun 2019 14:45:37 +0200
Hans Verkuil wrote:
> On 5/31/19 10:55 AM, Philipp Zabel wrote:
> > For now this just enables MPEG-2 decoding on the Hantro G1 on i.MX8MQ.
> >
> > Signed-off-by: Philipp Zabel
> > ---
> > Changes since v2 [1]:
> > - Adapted to changes in patches 4 and 5
> >
>
On Mon, 03 Jun 2019 14:52:44 -0400
Nicolas Dufresne wrote:
> > > - Dropping the DPB concept in H.264/H.265
> > >
> > > As far as I could understand, the decoded picture buffer (DPB) is a
> > > concept that only makes sense relative to a decoder implementation. The
> > > spec mentions how to mana
On Mon, 3 Jun 2019 16:05:26 +0200
Thierry Reding wrote:
> On Mon, Jun 03, 2019 at 02:51:13PM +0200, Boris Brezillon wrote:
> > +Maxime
> >
> > Oops, just realized Maxime was not Cc-ed on this series.
> >
> > On Mon, 3 Jun 2019 14:30:20 +0200
> > Thierry
This is needed if we want the core to be able to check _MPLANE support
without having to call the ->vdioc_querycap() hook.
Signed-off-by: Boris Brezillon
---
Changes in v5:
- Drop the fimc-lite and fimc-isp-video hack
Changes in v4:
- Add a hack in fimc-lite and fimc-isp-video ->qu
ad.
Signed-off-by: Boris Brezillon
---
Changes in v5:
- None
Changes in v4:
- None
Changes in v3:
- Send this patch separately (was previously part of the EXT_BUF/FMT
rework)
Changes in v2:
- None
---
drivers/media/pci/intel/ipu3/ipu3-cio2.c | 2 +-
drivers/media/platform/exynos-gsc
+Maxime
Oops, just realized Maxime was not Cc-ed on this series.
On Mon, 3 Jun 2019 14:30:20 +0200
Thierry Reding wrote:
> On Mon, Jun 03, 2019 at 01:09:42PM +0200, Boris Brezillon wrote:
> > Some stateless decoders don't support per-slice decoding (or at least
> > not in a
ned-off-by: Boris Brezillon
---
.../media/uapi/v4l/ext-ctrls-codec.rst| 42 ++-
drivers/media/v4l2-core/v4l2-ctrls.c | 9
include/media/h264-ctrls.h| 13 ++
3 files changed, 63 insertions(+), 1 deletion(-)
diff --git a/Documentation/
n specific. What's provided by the bitstream is a
list of references that will be used to decode a frame
* ... (add your own)
Feel free to comment on these changes and/or propose alternatives.
Regards,
Boris
Boris Brezillon (6):
media: uapi: h264: Clarify our expectations regarding NAL hea
The H264 uAPI has been modified to expose 2 operation modes: per slice
and per frame decoding.
The cedrus driver only supports per-slice decoding for now.
Signed-off-by: Boris Brezillon
---
drivers/staging/media/sunxi/cedrus/cedrus.c | 13 +
1 file changed, 13 insertions(+)
diff
Those lists can be extracted from the dpb, let's simplify userspace
life and build that list kernel-side (generic helpers will be provided
for drivers that need this list).
Signed-off-by: Boris Brezillon
---
Documentation/media/uapi/v4l/ext-ctrls-codec.rst | 9 -
1 file chang
ing to the
spec, it's actually an array. Make it more explicit by setting dims[0]
and adding a comment explaining why we have this limitation.
Signed-off-by: Boris Brezillon
---
drivers/staging/media/sunxi/cedrus/cedrus.c | 6 ++
1 file changed, 6 insertions(+)
diff --git a/drivers/stagi
to be passed through controls) should be pretty
easy, let's mandate that all slices be prepended with ANNEX B start
codes.
If we ever need to support AVC headers, we can add a new menu control
to select the type of NAL header to use.
Signed-off-by: Boris Brezillon
---
Note: we might want to
We are about to add a menu control, so let's make the code more generic
to support other control types.
Signed-off-by: Boris Brezillon
---
drivers/staging/media/sunxi/cedrus/cedrus.c | 47 -
drivers/staging/media/sunxi/cedrus/cedrus.h | 3 +-
2 files changed, 29 inser
: Philipp Zabel
Reviewed-by: Boris Brezillon
> ---
> New in v3.
> ---
> drivers/staging/media/hantro/hantro.h| 6 ++
> drivers/staging/media/hantro/hantro_drv.c| 5 +
> drivers/staging/media/hantro/imx8m_vpu_hw.c | 10 ++
> drivers/stagi
On Fri, 31 May 2019 10:55:20 +0200
Philipp Zabel wrote:
> Add devicetree binding documentation for the Hantro G1/G2 VPU on i.MX8MQ
> and for the Hantro G1/G2/H1 VPU on i.MX8MM.
>
> Signed-off-by: Philipp Zabel
> ---
> .../devicetree/bindings/media/imx8m-vpu.txt | 56 +++
Hm,
On Fri, 31 May 2019 10:55:19 +0200
Philipp Zabel wrote:
> On i.MX8MQ/MM a separate control block contains registers for per-core
> resets, clock gating, and fuse register control.
>
> Signed-off-by: Philipp Zabel
Reviewed-by: Boris Brezillon
> ---
> drivers/staging/med
ev: Pointer to device for convenient logging using
> * dev_ macros.
> * @clocks: Array of clock handles.
> - * @base:Mapped address of VPU registers.
> + * @bases: Mapped addresses of VPU registers.
I find the name &#
1 - 100 of 362 matches
Mail list logo
