[staging:staging-testing] BUILD SUCCESS WITH WARNING 0e3136771b58dda2bec0a0bfedab119e77a88c9b

2021-05-05 Thread kernel test robot
allnoconfig x86_64 randconfig-a001-20210505 x86_64 randconfig-a003-20210505 x86_64 randconfig-a005-20210505 x86_64 randconfig-a002-20210505 x86_64 randconfig-a006-20210505 x86_64 randconfig-a004-20210505 i386

Re: [PATCH v10 6/9] media: uapi: Add a control for HANTRO driver

2021-05-05 Thread Ezequiel Garcia
On Wed, 2021-05-05 at 16:55 +0200, Hans Verkuil wrote: > On 20/04/2021 14:10, Benjamin Gaignard wrote: > > The HEVC HANTRO driver needs to know the number of bits to skip at > > the beginning of the slice header. > > That is a hardware specific requirement so create a dedicated control > > for

Re: [RFC RESEND 2/3] media: uapi: Add VP9 stateless decoder controls

2021-05-05 Thread Nicolas Dufresne
Hi Hans, just a partial reply, I'll let Andrzej extend. Le jeudi 29 avril 2021 à 12:20 +0200, Hans Verkuil a écrit : > > +  - ``frame_width_minus_1`` > > +  - Add 1 to get the frame width expressed in pixels. > > +    * - __u16 > > +  - ``frame_height_minus_1`` > > +  - Add 1 to

Re: [PATCH v10 6/9] media: uapi: Add a control for HANTRO driver

2021-05-05 Thread Nicolas Dufresne
Le mercredi 05 mai 2021 à 16:18 +0100, John Cox a écrit : > > The HEVC HANTRO driver needs to know the number of bits to skip at > > the beginning of the slice header. > > That is a hardware specific requirement so create a dedicated control > > for this purpose. > > > > Signed-off-by: Benjamin

Re: [PATCH v10 6/9] media: uapi: Add a control for HANTRO driver

2021-05-05 Thread Benjamin Gaignard
Le 05/05/2021 à 16:55, Hans Verkuil a écrit : On 20/04/2021 14:10, Benjamin Gaignard wrote: The HEVC HANTRO driver needs to know the number of bits to skip at the beginning of the slice header. That is a hardware specific requirement so create a dedicated control for this purpose.

Re: [PATCH v10 6/9] media: uapi: Add a control for HANTRO driver

2021-05-05 Thread John Cox
>The HEVC HANTRO driver needs to know the number of bits to skip at >the beginning of the slice header. >That is a hardware specific requirement so create a dedicated control >for this purpose. > >Signed-off-by: Benjamin Gaignard >--- > .../userspace-api/media/drivers/hantro.rst| 19

Re: [PATCH v10 6/9] media: uapi: Add a control for HANTRO driver

2021-05-05 Thread Hans Verkuil
On 20/04/2021 14:10, Benjamin Gaignard wrote: > The HEVC HANTRO driver needs to know the number of bits to skip at > the beginning of the slice header. > That is a hardware specific requirement so create a dedicated control > for this purpose. > > Signed-off-by: Benjamin Gaignard > --- >

Re: [PATCH v10 8/9] media: hantro: Introduce G2/HEVC decoder

2021-05-05 Thread Hans Verkuil
Hi Benjamin, Some comments: On 20/04/2021 14:10, Benjamin Gaignard wrote: > Implement all the logic to get G2 hardware decoding HEVC frames. > It supports up level 5.1 HEVC stream. > It doesn't support yet 10 bits formats or the scaling feature. > > Add HANTRO HEVC dedicated control to skip

Re: [PATCH v10 0/9] Add HANTRO G2/HEVC decoder support for IMX8MQ

2021-05-05 Thread Hans Verkuil
Hi Benjamin, On 20/04/2021 14:10, Benjamin Gaignard wrote: > The IMX8MQ got two VPUs but until now only G1 has been enabled. > This series aim to add the second VPU (aka G2) and provide basic > HEVC decoding support. > > To be able to decode HEVC it is needed to add/update some of the >

[RFCv2 2/3] media: uapi: Add VP9 v4l2 library

2021-05-05 Thread Andrzej Pietrasiewicz
Provide code common to vp9 drivers in one central location. Signed-off-by: Andrzej Pietrasiewicz --- drivers/media/v4l2-core/Kconfig|4 + drivers/media/v4l2-core/Makefile |1 + drivers/media/v4l2-core/v4l2-vp9.c | 1831 include/media/v4l2-vp9.h

[RFCv2 3/3] media: rkvdec: Add the VP9 backend

2021-05-05 Thread Andrzej Pietrasiewicz
From: Boris Brezillon The Rockchip VDEC supports VP9 profile 0 up to 4096x2304@30fps. Add a backend for this new format. Signed-off-by: Boris Brezillon Signed-off-by: Ezequiel Garcia Signed-off-by: Adrian Ratiu Signed-off-by: Andrzej Pietrasiewicz --- drivers/staging/media/rkvdec/Kconfig

[RFCv2 1/3] media: uapi: Add VP9 stateless decoder controls

2021-05-05 Thread Andrzej Pietrasiewicz
Add the VP9 stateless decoder controls plus the documentation that goes with it. Signed-off-by: Boris Brezillon Signed-off-by: Ezequiel Garcia Signed-off-by: Adrian Ratiu Signed-off-by: Andrzej Pietrasiewicz --- .../userspace-api/media/v4l/biblio.rst| 10 +

[RFCv2 0/3] vp9 v4l2 stateless uapi

2021-05-05 Thread Andrzej Pietrasiewicz
Dear All, This is a follow-up work for https://patchwork.linuxtv.org/project/linux-media/list/?series=5268 I addressed Hans's comments. Changes: v1..v2: - improve documentation - imrpove coding style - factor out of common vp9 code into v4l2-vp9.c - rename

[PATCHv2 3/3] media: cedrus: Fix .buf_prepare

2021-05-05 Thread Andrzej Pietrasiewicz
The driver should only set the payload on .buf_prepare if the buffer is CAPTURE type. If an OUTPUT buffer has a zero bytesused set by userspace then v4l2-core will set it to buffer length. If we overwrite bytesused for OUTPUT buffers, too, then vb2_get_plane_payload() will return incorrect value

[PATCHv2 2/3] media: hantro: Fix .buf_prepare

2021-05-05 Thread Andrzej Pietrasiewicz
The driver should only set the payload on .buf_prepare if the buffer is CAPTURE type. If an OUTPUT buffer has a zero bytesused set by userspace then v4l2-core will set it to buffer length. If we overwrite bytesused for OUTPUT buffers, too, then vb2_get_plane_payload() will return incorrect value

[PATCHv2 1/3] media: rkvdec: Fix .buf_prepare

2021-05-05 Thread Andrzej Pietrasiewicz
From: Ezequiel Garcia The driver should only set the payload on .buf_prepare if the buffer is CAPTURE type. If an OUTPUT buffer has a zero bytesused set by userspace then v4l2-core will set it to buffer length. If we overwrite bytesused for OUTPUT buffers, too, then vb2_get_plane_payload() will

[PATCHv2 0/3] Fix .buf_prepare

2021-05-05 Thread Andrzej Pietrasiewicz
Drivers should only set the payload on .buf_prepare if the buffer is CAPTURE type. If an OUTPUT buffer has a zero bytesused set by userspace then v4l2-core will set it to buffer length. If we overwrite bytesused for OUTPUT buffers, too, then vb2_get_plane_payload() will return incorrect value

Re: [PATCH] media: rkvdec: Fix .buf_prepare

2021-05-05 Thread Andrzej Pietrasiewicz
Hi Ezequiel, W dniu 04.05.2021 o 13:56, Ezequiel Garcia pisze: Hi Andrzej, Thanks a lot for picking this up. On Tue, 2021-05-04 at 13:37 +0200, Andrzej Pietrasiewicz wrote: From: Ezequiel Garcia The driver should only set the payload on .buf_prepare if the buffer is CAPTURE type. If an