lkml.org 
[lkml]   [2020]   [Jul]   [24]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH] v4l2-ctrl: Add VP9 codec levels
From
Date


On 7/14/20 7:09 PM, Nicolas Dufresne wrote:
> Le lundi 13 juillet 2020 à 11:15 +0300, Stanimir Varbanov a écrit :
>> Add menu control for VP9 codec levels. A total of 14 levels are
>> defined for Profile 0 (8bit) and Profile 2 (10bit). Each level
>> is a set of constrained bitstreams coded with targeted resolutions,
>> frame rates, and bitrates.
>
> Looks good to me. Perhaps literally put the link in the message ?
>
> https://www.webmproject.org/vp9/levels/

sure, I will do that.

>
>>
>> The definition has been taken from webm project.
>>
>> Signed-off-by: Stanimir Varbanov <stanimir.varbanov@linaro.org>
>> ---
>> .../media/v4l/ext-ctrls-codec.rst | 42 +++++++++++++++++++
>> drivers/media/v4l2-core/v4l2-ctrls.c | 21 ++++++++++
>> include/uapi/linux/v4l2-controls.h | 17 ++++++++
>> 3 files changed, 80 insertions(+)
>>
>> diff --git a/Documentation/userspace-api/media/v4l/ext-ctrls-codec.rst b/Documentation/userspace-api/media/v4l/ext-ctrls-codec.rst
>> index d0d506a444b1..d49bdafa768a 100644
>> --- a/Documentation/userspace-api/media/v4l/ext-ctrls-codec.rst
>> +++ b/Documentation/userspace-api/media/v4l/ext-ctrls-codec.rst
>> @@ -3316,6 +3316,48 @@ enum v4l2_mpeg_video_vp9_profile -
>> * - ``V4L2_MPEG_VIDEO_VP9_PROFILE_3``
>> - Profile 3
>>
>> +.. _v4l2-mpeg-video-vp9-level:
>> +
>> +``V4L2_CID_MPEG_VIDEO_VP9_LEVEL (enum)``
>> +
>> +enum v4l2_mpeg_video_vp9_level -
>> + This control allows selecting the level for VP9 encoder.
>> + This is also used to enumerate supported levels by VP9 encoder or decoder.
>> + Possible values are:
>
> Shall we add some hint in the doc itself upon where these levels are
> from so one can figure-out what constraints they impose.

Fine by me. The same link as in the commit message?

>
> Also, as for other levels (H264/HEVC), all drivers are mostly dummy and
> don't validate any of this, shall we keep it this way ? Levels are in
> fact complex, because some of the constraints are in image size, and
> most of the other constraints are raw or compressed size over time, so
> requires timing information for let's say a CBR encoder to enforce it,
> for the S_FMT function for the raw constraints.

I think this should be a driver responsibility.

>
>> +
>> +.. flat-table::
>> + :header-rows: 0
>> + :stub-columns: 0
>> +
>> + * - ``V4L2_MPEG_VIDEO_VP9_LEVEL_1_0``
>> + - Level 1
>> + * - ``V4L2_MPEG_VIDEO_VP9_LEVEL_1_1``
>> + - Level 1.1
>> + * - ``V4L2_MPEG_VIDEO_VP9_LEVEL_2_0``
>> + - Level 2
>> + * - ``V4L2_MPEG_VIDEO_VP9_LEVEL_2_1``
>> + - Level 2.1
>> + * - ``V4L2_MPEG_VIDEO_VP9_LEVEL_3_0``
>> + - Level 3
>> + * - ``V4L2_MPEG_VIDEO_VP9_LEVEL_3_1``
>> + - Level 3.1
>> + * - ``V4L2_MPEG_VIDEO_VP9_LEVEL_4_0``
>> + - Level 4
>> + * - ``V4L2_MPEG_VIDEO_VP9_LEVEL_4_1``
>> + - Level 4.1
>> + * - ``V4L2_MPEG_VIDEO_VP9_LEVEL_5_0``
>> + - Level 5
>> + * - ``V4L2_MPEG_VIDEO_VP9_LEVEL_5_1``
>> + - Level 5.1
>> + * - ``V4L2_MPEG_VIDEO_VP9_LEVEL_5_2``
>> + - Level 5.2
>> + * - ``V4L2_MPEG_VIDEO_VP9_LEVEL_6_0``
>> + - Level 6
>> + * - ``V4L2_MPEG_VIDEO_VP9_LEVEL_6_1``
>> + - Level 6.1
>> + * - ``V4L2_MPEG_VIDEO_VP9_LEVEL_6_2``
>> + - Level 6.2
>> +
>>
>> High Efficiency Video Coding (HEVC/H.265) Control Reference
>> ===========================================================
>> diff --git a/drivers/media/v4l2-core/v4l2-ctrls.c b/drivers/media/v4l2-core/v4l2-ctrls.c
>> index 3f3fbcd60cc6..359dc737053d 100644
>> --- a/drivers/media/v4l2-core/v4l2-ctrls.c
>> +++ b/drivers/media/v4l2-core/v4l2-ctrls.c
>> @@ -474,6 +474,23 @@ const char * const *v4l2_ctrl_get_menu(u32 id)
>> "3",
>> NULL,
>> };
>> + static const char * const vp9_level[] = {
>> + "1",
>> + "1.1",
>> + "2",
>> + "2.1",
>> + "3",
>> + "3.1",
>> + "4",
>> + "4.1",
>> + "5",
>> + "5.1",
>> + "5.2",
>> + "6",
>> + "6.1",
>> + "6.2",
>> + NULL,
>> + };
>>
>> static const char * const flash_led_mode[] = {
>> "Off",
>> @@ -685,6 +702,8 @@ const char * const *v4l2_ctrl_get_menu(u32 id)
>> return vp8_profile;
>> case V4L2_CID_MPEG_VIDEO_VP9_PROFILE:
>> return vp9_profile;
>> + case V4L2_CID_MPEG_VIDEO_VP9_LEVEL:
>> + return vp9_level;
>> case V4L2_CID_JPEG_CHROMA_SUBSAMPLING:
>> return jpeg_chroma_subsampling;
>> case V4L2_CID_DV_TX_MODE:
>> @@ -938,6 +957,7 @@ const char *v4l2_ctrl_get_name(u32 id)
>> case V4L2_CID_MPEG_VIDEO_VPX_P_FRAME_QP: return "VPX P-Frame QP Value";
>> case V4L2_CID_MPEG_VIDEO_VP8_PROFILE: return "VP8 Profile";
>> case V4L2_CID_MPEG_VIDEO_VP9_PROFILE: return "VP9 Profile";
>> + case V4L2_CID_MPEG_VIDEO_VP9_LEVEL: return "VP9 Level";
>> case V4L2_CID_MPEG_VIDEO_VP8_FRAME_HEADER: return "VP8 Frame Header";
>>
>> /* HEVC controls */
>> @@ -1294,6 +1314,7 @@ void v4l2_ctrl_fill(u32 id, const char **name, enum v4l2_ctrl_type *type,
>> case V4L2_CID_MPEG_VIDEO_VPX_GOLDEN_FRAME_SEL:
>> case V4L2_CID_MPEG_VIDEO_VP8_PROFILE:
>> case V4L2_CID_MPEG_VIDEO_VP9_PROFILE:
>> + case V4L2_CID_MPEG_VIDEO_VP9_LEVEL:
>> case V4L2_CID_DETECT_MD_MODE:
>> case V4L2_CID_MPEG_VIDEO_HEVC_PROFILE:
>> case V4L2_CID_MPEG_VIDEO_HEVC_LEVEL:
>> diff --git a/include/uapi/linux/v4l2-controls.h b/include/uapi/linux/v4l2-controls.h
>> index 62271418c1be..1b0bc79c1bc3 100644
>> --- a/include/uapi/linux/v4l2-controls.h
>> +++ b/include/uapi/linux/v4l2-controls.h
>> @@ -650,6 +650,23 @@ enum v4l2_mpeg_video_vp9_profile {
>> V4L2_MPEG_VIDEO_VP9_PROFILE_2 = 2,
>> V4L2_MPEG_VIDEO_VP9_PROFILE_3 = 3,
>> };
>> +#define V4L2_CID_MPEG_VIDEO_VP9_LEVEL (V4L2_CID_MPEG_BASE+513)
>> +enum v4l2_mpeg_video_vp9_level {
>> + V4L2_MPEG_VIDEO_VP9_LEVEL_1_0 = 0,
>> + V4L2_MPEG_VIDEO_VP9_LEVEL_1_1 = 1,
>> + V4L2_MPEG_VIDEO_VP9_LEVEL_2_0 = 2,
>> + V4L2_MPEG_VIDEO_VP9_LEVEL_2_1 = 3,
>> + V4L2_MPEG_VIDEO_VP9_LEVEL_3_0 = 4,
>> + V4L2_MPEG_VIDEO_VP9_LEVEL_3_1 = 5,
>> + V4L2_MPEG_VIDEO_VP9_LEVEL_4_0 = 6,
>> + V4L2_MPEG_VIDEO_VP9_LEVEL_4_1 = 7,
>> + V4L2_MPEG_VIDEO_VP9_LEVEL_5_0 = 8,
>> + V4L2_MPEG_VIDEO_VP9_LEVEL_5_1 = 9,
>> + V4L2_MPEG_VIDEO_VP9_LEVEL_5_2 = 10,
>> + V4L2_MPEG_VIDEO_VP9_LEVEL_6_0 = 11,
>> + V4L2_MPEG_VIDEO_VP9_LEVEL_6_1 = 12,
>> + V4L2_MPEG_VIDEO_VP9_LEVEL_6_2 = 13,
>> +};
>>
>> /* CIDs for HEVC encoding. */
>>
>

--
regards,
Stan

\
 
 \ /
  Last update: 2020-07-24 22:30    [W:0.534 / U:0.004 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site