[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <62750a74-f856-c927-72a3-17d1a2624550@xs4all.nl>
Date: Mon, 12 Aug 2019 12:20:36 +0200
From: Hans Verkuil <hverkuil@...all.nl>
To: Ezequiel Garcia <ezequiel@...labora.com>,
linux-media@...r.kernel.org
Cc: kernel@...labora.com,
Nicolas Dufresne <nicolas.dufresne@...labora.com>,
Tomasz Figa <tfiga@...omium.org>,
linux-rockchip@...ts.infradead.org,
Heiko Stuebner <heiko@...ech.de>,
Jonas Karlman <jonas@...boo.se>,
Philipp Zabel <p.zabel@...gutronix.de>,
Boris Brezillon <boris.brezillon@...labora.com>,
Paul Kocialkowski <paul.kocialkowski@...tlin.com>,
Alexandre Courbot <acourbot@...omium.org>,
fbuergisser@...omium.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH v4 05/11] media: uapi: h264: Get rid of the p0/b0/b1
ref-lists
On 8/8/19 12:34 PM, Ezequiel Garcia wrote:
> From: Boris Brezillon <boris.brezillon@...labora.com>
>
> 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 <boris.brezillon@...labora.com>
> Reviewed-by: Nicolas Dufresne <nicolas.dufresne@...labora.com>
> Reviewed-by: Ezequiel Garcia <ezequiel@...labora.com>
> Reviewed-by: Paul Kocialkowski <paul.kocialkowski@...tlin.com>
> ---
> Changes in v4:
> * Add R-b
???
Nothing was added, left-over from some old version?
Regards,
Hans
> * Remove the reflist from the struct
>
> Changes in v3:
> * None
>
> Changes in v2:
> * None
> ---
> Documentation/media/uapi/v4l/ext-ctrls-codec.rst | 9 ---------
> include/media/h264-ctrls.h | 3 ---
> 2 files changed, 12 deletions(-)
>
> diff --git a/Documentation/media/uapi/v4l/ext-ctrls-codec.rst b/Documentation/media/uapi/v4l/ext-ctrls-codec.rst
> index 94fd3a9b8b9e..d60909b2d1ca 100644
> --- a/Documentation/media/uapi/v4l/ext-ctrls-codec.rst
> +++ b/Documentation/media/uapi/v4l/ext-ctrls-codec.rst
> @@ -1942,15 +1942,6 @@ enum v4l2_mpeg_video_h264_hierarchical_coding_type -
> * - __u16
> - ``nal_ref_idc``
> - NAL reference ID value coming from the NAL Unit header
> - * - __u8
> - - ``ref_pic_list_p0[32]``
> - - Backward reference list used by P-frames in the original bitstream order
> - * - __u8
> - - ``ref_pic_list_b0[32]``
> - - Backward reference list used by B-frames in the original bitstream order
> - * - __u8
> - - ``ref_pic_list_b1[32]``
> - - Forward reference list used by B-frames in the original bitstream order
> * - __s32
> - ``top_field_order_cnt``
> - Picture Order Count for the coded top field
> diff --git a/include/media/h264-ctrls.h b/include/media/h264-ctrls.h
> index 31555c99f64a..2de438393025 100644
> --- a/include/media/h264-ctrls.h
> +++ b/include/media/h264-ctrls.h
> @@ -203,9 +203,6 @@ struct v4l2_ctrl_h264_decode_params {
> struct v4l2_h264_dpb_entry dpb[16];
> __u16 num_slices;
> __u16 nal_ref_idc;
> - __u8 ref_pic_list_p0[32];
> - __u8 ref_pic_list_b0[32];
> - __u8 ref_pic_list_b1[32];
> __s32 top_field_order_cnt;
> __s32 bottom_field_order_cnt;
> __u32 flags; /* V4L2_H264_DECODE_PARAM_FLAG_* */
>
Powered by blists - more mailing lists