Re: [linux-sunxi] [PATCH v2 2/2] media: cedrus: Add H264 decoding support

2018-11-24 Thread Jernej Škrabec
Hi, first, thanks you for working on this! I also spend some time working on H264 and I have some comments below. Dne četrtek, 15. november 2018 ob 15:56:50 CET je Maxime Ripard napisal(a): > Introduce some basic H264 decoding support in cedrus. So far, only the > baseline profile videos have be

Re: [linux-sunxi] [PATCH v2 2/2] media: cedrus: Add H264 decoding support

2018-11-27 Thread Jernej Škrabec
Dne torek, 27. november 2018 ob 16:50:28 CET je Maxime Ripard napisal(a): > Hi Jernej, > > Thanks for your review! > > On Sat, Nov 24, 2018 at 09:43:43PM +0100, Jernej Škrabec wrote: > > > +enum cedrus_h264_sram_off { > > > + CEDRUS_SRAM_H2

Re: [linux-sunxi] [PATCH v2 1/2] media: uapi: Add H264 low-level decoder API compound controls.

2018-11-27 Thread Jernej Škrabec
Hi! Dne četrtek, 15. november 2018 ob 15:56:49 CET je Maxime Ripard napisal(a): > From: Pawel Osciak > > Stateless video codecs will require both the H264 metadata and slices in > order to be able to decode frames. > > This introduces the definitions for a new pixel format for H264 slices that

Re: [linux-sunxi] [PATCH v2 2/2] media: cedrus: Add H264 decoding support

2018-11-27 Thread Jernej Škrabec
Dne torek, 27. november 2018 ob 17:30:00 CET je Jernej Škrabec napisal(a): > Dne torek, 27. november 2018 ob 16:50:28 CET je Maxime Ripard napisal(a): > > Hi Jernej, > > > > Thanks for your review! > > > > On Sat, Nov 24, 2018 at 09:43:43PM +0100, Je

Re: [linux-sunxi] [PATCH v2 2/2] media: cedrus: Add H264 decoding support

2018-11-30 Thread Jernej Škrabec
Dne petek, 30. november 2018 ob 08:30:47 CET je Maxime Ripard napisal(a): > On Tue, Nov 27, 2018 at 05:30:00PM +0100, Jernej Škrabec wrote: > > > > > +static void _cedrus_write_ref_list(struct cedrus_ctx *ctx, > > > > > +

Re: [linux-sunxi] [PATCH v2 1/2] media: v4l: Add definitions for the HEVC slice format and controls

2018-12-05 Thread Jernej Škrabec
Hi! Dne petek, 23. november 2018 ob 14:02:08 CET je Paul Kocialkowski napisal(a): > This introduces the required definitions for HEVC decoding support with > stateless VPUs. The controls associated to the HEVC slice format provide > the required meta-data for decoding slices extracted from the bit

Re: [linux-sunxi] [PATCH v2 2/2] media: cedrus: Add H264 decoding support

2018-12-05 Thread Jernej Škrabec
Hi! Jonas Karlman (in CC) and me managed to solve playback issues with interlaced H264 videos. Please check comments below. You can also build and test LibreELEC for H3 from https://github.com/jernejsk/LibreELEC.tv/tree/hw_dec_ffmpeg It has all changes suggested below, except buffer sizes ar

Re: [linux-sunxi] [PATCH v2 15/15] arm64: dts: allwinner: a64: Add Video Engine node

2018-12-07 Thread Jernej Škrabec
Hi! Dne sreda, 05. december 2018 ob 10:24:44 CET je Paul Kocialkowski napisal(a): > This adds the Video Engine node for the A64. Since it can map the whole > DRAM range, there is no particular need for a reserved memory node > (unlike platforms preceding the A33). > > Signed-off-by: Paul Kocialko

Re: [PATCH 1/3] media: dt: bindings: sunxi-ir: Add A64 compatible

2019-01-12 Thread Jernej Škrabec
Dne sobota, 12. januar 2019 ob 02:56:11 CET je Chen-Yu Tsai napisal(a): > On Sat, Jan 12, 2019 at 1:30 AM Jernej Skrabec wrote: > > A64 IR is compatible with A13, so add A64 compatible with A13 as a > > fallback. > > We ask people to add the SoC-specific compatible as a contigency, > in case thi

Re: [PATCH 1/3] media: dt: bindings: sunxi-ir: Add A64 compatible

2019-01-21 Thread Jernej Škrabec
Dne ponedeljek, 21. januar 2019 ob 10:57:57 CET je Chen-Yu Tsai napisal(a): > On Mon, Jan 21, 2019 at 5:50 PM Maxime Ripard wrote: > > Hi, > > > > I'm a bit late to the party, sorry for that. > > > > On Sat, Jan 12, 2019 at 09:56:11AM +0800, Chen-Yu Tsai wrote: > > > On Sat, Jan 12, 2019 at 1:3

Re: [PATCH 1/3] media: dt: bindings: sunxi-ir: Add A64 compatible

2019-01-21 Thread Jernej Škrabec
Dne ponedeljek, 21. januar 2019 ob 18:39:26 CET je Chen-Yu Tsai napisal(a): > On Tue, Jan 22, 2019 at 1:33 AM Jernej Škrabec wrote: > > Dne ponedeljek, 21. januar 2019 ob 10:57:57 CET je Chen-Yu Tsai napisal(a): > > > On Mon, Jan 21, 2019 at 5:50 PM Maxime Ripard >

Re: [PATCH 1/3] media: dt: bindings: sunxi-ir: Add A64 compatible

2019-01-24 Thread Jernej Škrabec
Dne ponedeljek, 21. januar 2019 ob 10:57:57 CET je Chen-Yu Tsai napisal(a): > On Mon, Jan 21, 2019 at 5:50 PM Maxime Ripard wrote: > > Hi, > > > > I'm a bit late to the party, sorry for that. > > > > On Sat, Jan 12, 2019 at 09:56:11AM +0800, Chen-Yu Tsai wrote: > > > On Sat, Jan 12, 2019 at 1:3

Re: Proposed updates and guidelines for MPEG-2, H.264 and H.265 stateless support

2019-05-18 Thread Jernej Škrabec
Dne sobota, 18. maj 2019 ob 11:50:37 CEST je Paul Kocialkowski napisal(a): > Hi, > > On Fri, 2019-05-17 at 16:43 -0400, Nicolas Dufresne wrote: > > Le jeudi 16 mai 2019 à 20:45 +0200, Paul Kocialkowski a écrit : > > > Hi, > > > > > > Le jeudi 16 mai 2019 à 14:24 -0400, Nicolas Dufresne a écrit :

Re: [PATCH RFC 5/6] media: cedrus: Make the slice_params array size limitation more explicit

2019-06-03 Thread Jernej Škrabec
Dne ponedeljek, 03. junij 2019 ob 13:09:45 CEST je Boris Brezillon napisal(a): > The driver only supports per-slice decoding, and in that mode > decode_params->num_slices must be set to 1 and the slice_params array > should contain only one element. What Cedrus actually needs to know is if this is

Re: [linux-sunxi] [PATCH v3 2/2] media: cedrus: Add H264 decoding support

2019-02-11 Thread Jernej Škrabec
Hi Maxime, I spotted few minor issues, but otherwise it looks very well. I'll do detailed review at a later time. Dne ponedeljek, 11. februar 2019 ob 15:39:03 CET je Maxime Ripard napisal(a): > Introduce some basic H264 decoding support in cedrus. So far, only the > baseline profile videos have

Re: [PATCH v3 2/2] media: cedrus: Add H264 decoding support

2019-02-12 Thread Jernej Škrabec
Dne torek, 12. februar 2019 ob 13:47:13 CET je Maxime Ripard napisal(a): > On Mon, Feb 11, 2019 at 04:48:17PM -0300, Ezequiel Garcia wrote: > > On Mon, 2019-02-11 at 15:39 +0100, Maxime Ripard wrote: > > > Introduce some basic H264 decoding support in cedrus. So far, only the > > > baseline profile

Re: [linux-sunxi] [PATCH v3 2/2] media: cedrus: Add H264 decoding support

2019-02-12 Thread Jernej Škrabec
Dne torek, 12. februar 2019 ob 11:43:14 CET je Maxime Ripard napisal(a): > Hi, > > On Mon, Feb 11, 2019 at 08:21:31PM +0100, Jernej Škrabec wrote: > > > + reg = 0; > > > + /* > > > + * FIXME: This bit tells the video engine to use the default > > > +

Re: [PATCH v3 2/2] media: cedrus: Add H264 decoding support

2019-02-14 Thread Jernej Škrabec
Dne ponedeljek, 11. februar 2019 ob 15:39:03 CET je Maxime Ripard napisal(a): > Introduce some basic H264 decoding support in cedrus. So far, only the > baseline profile videos have been tested, and some more advanced features > used in higher profiles are not even implemented. > > Signed-off-by:

Re: [PATCH v3 2/2] media: cedrus: Add H264 decoding support

2019-02-14 Thread Jernej Škrabec
Dne ponedeljek, 11. februar 2019 ob 15:39:03 CET je Maxime Ripard napisal(a): > Introduce some basic H264 decoding support in cedrus. So far, only the > baseline profile videos have been tested, and some more advanced features > used in higher profiles are not even implemented. > > Signed-off-by:

Re: [PATCH v4 2/2] media: cedrus: Add H264 decoding support

2019-02-20 Thread Jernej Škrabec
Hi! I really wanted to do another review on previous series but got distracted by analyzing one particulary troublesome H264 sample. It still doesn't work correctly, so I would ask you if you can test it with your stack (it might be userspace issue): http://jernej.libreelec.tv/videos/problemat

Re: [PATCH v4 2/2] media: cedrus: Add H264 decoding support

2019-02-21 Thread Jernej Škrabec
Hi, Dne sreda, 20. februar 2019 ob 18:50:54 CET je Jernej Škrabec napisal(a): > Hi! > > I really wanted to do another review on previous series but got distracted > by analyzing one particulary troublesome H264 sample. It still doesn't work > correctly, so I would ask you if

Re: [PATCH v4 2/2] media: cedrus: Add H264 decoding support

2019-03-05 Thread Jernej Škrabec
Dne torek, 05. marec 2019 ob 11:17:32 CET je Maxime Ripard napisal(a): > Hi Jernej, > > On Wed, Feb 20, 2019 at 06:50:54PM +0100, Jernej Škrabec wrote: > > I really wanted to do another review on previous series but got distracted > > by analyzing one particulary troublesome

Re: [PATCH v4 2/2] media: cedrus: Add H264 decoding support

2019-03-06 Thread Jernej Škrabec
Dne sreda, 06. marec 2019 ob 11:57:08 CET je Maxime Ripard napisal(a): > Hi, > > On Tue, Mar 05, 2019 at 06:05:08PM +0100, Jernej Škrabec wrote: > > Dne torek, 05. marec 2019 ob 11:17:32 CET je Maxime Ripard napisal(a): > > > Hi Jernej, > > > > > >

Re: [PATCH 1/3] cedrus: fill in bus_info for media device

2019-09-03 Thread Jernej Škrabec
Hi! Dne petek, 30. avgust 2019 ob 11:26:22 CEST je Hans Verkuil napisal(a): > Fixes this compliance warning: > > $ v4l2-compliance -m0 > v4l2-compliance SHA: b514d615166bdc0901a4c71261b87db31e89f464, 32 bits > > Compliance test for cedrus device /dev/media0: > > Media Driver Info: > Dri

Re: [PATCH 2/3] cedrus: choose default pixelformat in try_fmt

2019-09-03 Thread Jernej Škrabec
Hi! Dne petek, 30. avgust 2019 ob 11:26:23 CEST je Hans Verkuil napisal(a): > If an unsupported pixelformat is passed to try_fmt, then pick > the first valid pixelformat instead. This is more standard V4L2 > behavior. > > Signed-off-by: Hans Verkuil > --- > .../staging/media/sunxi/cedrus/cedrus

Re: [PATCH 3/3] cedrus: fix various format-related compliance issues

2019-09-03 Thread Jernej Škrabec
Hi! Dne petek, 30. avgust 2019 ob 11:26:24 CEST je Hans Verkuil napisal(a): > Initialize the context on open() with valid capture and output > formats. It is good practice to always have valid formats internally. > > This solves one vb2 warning in the kernel log where the sizeimage > value of the

Re: [PATCH 7/8] media: cedrus: Add support for holding capture buffer

2019-09-04 Thread Jernej Škrabec
Dne četrtek, 29. avgust 2019 ob 13:23:29 CEST je Hans Verkuil napisal(a): > On 8/22/19 9:44 PM, Jernej Skrabec wrote: > > When frame contains multiple slices and driver works in slice mode, it's > > more efficient to hold capture buffer in queue until all slices of a > > same frame are decoded. > >

Re: [PATCHv3 0/8] media: cedrus: h264: Support multi-slice frames

2019-10-11 Thread Jernej Škrabec
Dne četrtek, 10. oktober 2019 ob 15:11:44 CEST je Hans Verkuil napisal(a): > This series adds support for decoding multi-slice H264 frames along with > support for V4L2_DEC_CMD_FLUSH and V4L2_BUF_FLAG_M2M_HOLD_CAPTURE_BUF. > > This has only been compile-tested. Jernej, can you test with ffmpeg? S

Re: [PATCHv3 6/8] media: cedrus: Detect first slice of a frame

2019-10-11 Thread Jernej Škrabec
Dne četrtek, 10. oktober 2019 ob 15:11:50 CEST je Hans Verkuil napisal(a): > From: Jernej Skrabec > > When codec supports multiple slices in one frame, VPU has to know when > first slice of each frame is being processed, presumably to correctly > clear/set data in auxiliary buffers. > > Add firs

Re: [PATCHv3 7/8] media: cedrus: h264: Support multiple slices per frame

2019-10-11 Thread Jernej Škrabec
Dne četrtek, 10. oktober 2019 ob 15:11:51 CEST je Hans Verkuil napisal(a): > From: Jernej Skrabec > > With recent changes, support for decoding multi-slice frames can be > easily added now. > > Signal VPU if current slice is first in frame or not and add information > about first macroblock coor

Re: [PATCHv3 8/8] media: cedrus: Add support for holding capture buffer

2019-10-11 Thread Jernej Škrabec
Dne četrtek, 10. oktober 2019 ob 15:11:52 CEST je Hans Verkuil napisal(a): > From: Jernej Skrabec > > When frame contains multiple slices and driver works in slice mode, it's > more efficient to hold capture buffer in queue until all slices of a > same frame are decoded. > > Add support for that

Re: [PATCHv4 0/6] media: cedrus: h264: Support multi-slice frames

2019-10-15 Thread Jernej Škrabec
Dne petek, 11. oktober 2019 ob 11:32:39 CEST je Hans Verkuil napisal(a): > This series adds support for decoding multi-slice H264 frames along with > support for V4L2_DEC_CMD_FLUSH and V4L2_BUF_FLAG_M2M_HOLD_CAPTURE_BUF. > > This has only been compile-tested. Jernej, can you test with ffmpeg? Tes

Re: [RESEND RFC PATCH 0/7] sun8i H3 HDMI glue driver for DW HDMI

2017-09-30 Thread Jernej Škrabec
Hi, Dne sobota, 30. september 2017 ob 13:58:03 CEST je Alexey Kardashevskiy napisal(a): > On 21/09/17 06:01, Jernej Skrabec wrote: > > [added media mailing list due to CEC question] > > > > This patch series adds a HDMI glue driver for Allwinner H3 SoC. For now, > > only video and CEC functional