Re: [PATCH v2 0/3] media: cedrus: improvements

From: Jernej Åkrabec
Date: Thu Oct 03 2019 - 01:21:38 EST


Dne Äetrtek, 03. oktober 2019 ob 00:23:07 CEST je Paul Kocialkowski
napisal(a):
> Hi,
>
> On Wed 02 Oct 19, 21:35, Jernej Skrabec wrote:
> > This is continuation of https://lkml.org/lkml/2019/5/30/1459 with several
> > patches removed (2 merged, others needs redesign) and one added.
>
> Thanks for the continued effort on this, these fixes are greatly appreciated
> (and more generally, all the work you are putting into cedrus)!
>
> Although I've been out of the loop on cedrus, it is very nice to see
> development happening. Keep up the good work!

Thanks! Those fixes are just cleaned up version of patches I'm using in
LibreELEC for some time now. I hate maintaining out-of-tree patches over a
long period, so pushing them upstream is beneficial for all :)

I'll send more improvements once your HEVC patches are merged.

Best regards,
Jernej

>
> Cheers,
>
> Paul
>
> > Patch 1 fixes h264 playback issue which happens in rare cases.
> >
> > Patch 2 sets PPS default reference index count in register from PPS
> > control. Currently it was set to values from slice control.
> >
> > Patch 3 replaces direct accesses to capture queue from m2m contex with
> > helpers which was designed exactly for that. It's also safer since
> > helpers do some checks.
> >
> > Best regards,
> > Jernej
> >
> > Jernej Skrabec (3):
> > media: cedrus: Fix decoding for some H264 videos
> > media: cedrus: Fix H264 default reference index count
> > media: cedrus: Use helpers to access capture queue
> >
> > drivers/staging/media/sunxi/cedrus/cedrus.h | 8 +++-
> > .../staging/media/sunxi/cedrus/cedrus_h264.c | 46 ++++++++++++++-----
> > .../staging/media/sunxi/cedrus/cedrus_regs.h | 3 ++
> > 3 files changed, 44 insertions(+), 13 deletions(-)