| Message ID | 20210831094900.203283-1-benjamin.gaignard@collabora.com (mailing list archive) |
|---|---|
| Headers | show |
| Series | media: HEVC: RPS clean up | expand |
As per discussion with Ezequiel, I think this patch should perhaps be made an RFC titled "media: uapi: clean and make HEVC uAPI public". Keep it RFC until we have collected all the needed changes. Opinion ? Le mardi 31 août 2021 à 11:48 +0200, Benjamin Gaignard a écrit : > This series aims to clean up Reference Picture Set usage and flags. > > Long term flag was named with RPS prefix while it is not used for RPS > but for mark long term references in DBP. Remane it and remove the two > other useless RPS flags. > > Clarify documentation about RPS lists content and make sure that Hantro > driver use them correctly (i.e without look up in DBP). > > Benjamin > > Benjamin Gaignard (2): > media: hevc: Remove RPS named flags > media: hevc: Embedded indexes in RPS > > .../media/v4l/ext-ctrls-codec.rst | 12 ++++----- > .../staging/media/hantro/hantro_g2_hevc_dec.c | 27 +++++-------------- > .../staging/media/sunxi/cedrus/cedrus_h265.c | 2 +- > include/media/hevc-ctrls.h | 4 +-- > 4 files changed, 14 insertions(+), 31 deletions(-) >
Hans, Nicolas, On Tue, 31 Aug 2021 at 16:19, Nicolas Dufresne <nicolas@ndufresne.ca> wrote: > > As per discussion with Ezequiel, I think this patch should perhaps be made an > RFC titled "media: uapi: clean and make HEVC uAPI public". Keep it RFC until we > have collected all the needed changes. > > Opinion ? I agree. IMO, it makes no sense to keep merging anything on Hantro G2 HEVC support, until the BLK-CTRL part is ready [1]. Without that, we can't test anything so we may introduce regressions without knowing. Also, I think it's time to start discussing the HEVC uAPI as a whole, with the goal is having a stable interface that can be used by userspace frameworks (GStreamer). [1] https://www.spinics.net/lists/arm-kernel/msg913881.html Thanks, Ezequiel