* [PATCH v3 0/3] Add controls for VP8/VP9 profile
@ 2018-06-14 7:46 Keiichi Watanabe
2018-06-14 7:46 ` [PATCH v3 1/3] media: v4l2-ctrl: Change control for VP8 profile to menu control Keiichi Watanabe
` (2 more replies)
0 siblings, 3 replies; 9+ messages in thread
From: Keiichi Watanabe @ 2018-06-14 7:46 UTC (permalink / raw)
To: linux-arm-kernel
Cc: Mauro Carvalho Chehab, Tiffany Lin, Andrew-CT Chen,
Matthias Brugger, Stanimir Varbanov, Kyungmin Park, Kamil Debski,
Jeongtae Park, Andrzej Hajda, Hans Verkuil, Keiichi Watanabe,
Jonathan Corbet, Sakari Ailus, Smitha T Murthy,
Ricardo Ribalda Delgado, Tomasz Figa, Andy Shevchenko,
linux-media, linux-kernel, linux-mediatek, linux-arm-msm,
s.nawrocki
This patch series add new menu controls V4L2_CID_MPEG_VIDEO_VP8_PROFILE and
V4L2_CID_MPEG_VIDEO_VP9_PROFILE for VP8/VP9 profiles. These controls can be used
to select a desired profile for VP8/VP9 encoders. In addition, they are also
used to query for supported profiles by an encoder or a decoder.
Patch 1 adds a control V4L2_CID_MPEG_VIDEO_VP8_PROFILE for VP8 profile. Though
V4L2_CID_MPEG_VIDEO_VPX_PROFILE is originally used for VP8 profile, this control
is not good by the following reasons:
(i) Despite the name contains 'VPX', it cannot be used for VP9 because supported
profiles differ between VP8 and VP9.
(ii) Unlike other controls for profiles (e.g. H264), this is not a menu control
but an integer control, which cannot be used to query for supported profiles.
Thus, V4L2_CID_MPEG_VIDEO_VPX_PROFILE is deprecated and become an alias of
V4L2_CID_MPEG_VIDEO_VP8_PROFILE. In addition, Patch 1 fixes the use of
V4L2_CID_MPEG_VIDEO_VPX_PROFILE in existing venus/s5p_mfc drivers.
Patch 2 adds a control V4L2_CID_MPEG_VIDEO_VP9_PROFILE for VP9 profile.
By patch 3, this control is supported in MediaTek decoder's driver, which
supports VP9 profile 0.
Version 3 changes:
- Add V4L2_CID_MPEG_VIDEO_VP8_PROFILE in v4l2-controls.
- Make V4L2_CID_MPEG_VIDEO_VPX_PROFILE to be an alias of
V4L2_CID_MPEG_VIDEO_VP8_PROFILE.
- Fix the use of V4L2_CID_MPEG_VIDEO_VPX_PROFILE in venus/s5p_mfc drivers.
- Small fix in mtk_vcodec_dec.
Version 2 changes:
- Support V4L2_CID_MPEG_VIDEO_VP9_PROFILE in MediaTek decoder's driver.
Version 1 changes:
- Add V4L2_CID_MPEG_VIDEO_VP9_PROFILE in v4l2-controls.
Keiichi Watanabe (3):
media: v4l2-ctrl: Change control for VP8 profile to menu control
media: v4l2-ctrl: Add control for VP9 profile
media: mtk-vcodec: Support VP9 profile in decoder
.../media/uapi/v4l/extended-controls.rst | 52 +++++++++++++++++--
.../platform/mtk-vcodec/mtk_vcodec_dec.c | 5 ++
drivers/media/platform/qcom/venus/core.h | 2 +-
.../media/platform/qcom/venus/hfi_helper.h | 12 ++---
.../media/platform/qcom/venus/vdec_ctrls.c | 10 ++--
drivers/media/platform/qcom/venus/venc.c | 14 ++---
.../media/platform/qcom/venus/venc_ctrls.c | 12 +++--
drivers/media/platform/s5p-mfc/s5p_mfc_enc.c | 15 +++---
drivers/media/v4l2-core/v4l2-ctrls.c | 23 +++++++-
include/uapi/linux/v4l2-controls.h | 18 ++++++-
10 files changed, 127 insertions(+), 36 deletions(-)
--
2.18.0.rc1.242.g61856ae69a-goog
^ permalink raw reply [flat|nested] 9+ messages in thread* [PATCH v3 1/3] media: v4l2-ctrl: Change control for VP8 profile to menu control 2018-06-14 7:46 [PATCH v3 0/3] Add controls for VP8/VP9 profile Keiichi Watanabe @ 2018-06-14 7:46 ` Keiichi Watanabe 2018-06-14 8:04 ` Stanimir Varbanov 2018-06-15 8:51 ` Hans Verkuil 2018-06-14 7:46 ` [PATCH v3 2/3] media: v4l2-ctrl: Add control for VP9 profile Keiichi Watanabe 2018-06-14 7:46 ` [PATCH v3 3/3] media: mtk-vcodec: Support VP9 profile in decoder Keiichi Watanabe 2 siblings, 2 replies; 9+ messages in thread From: Keiichi Watanabe @ 2018-06-14 7:46 UTC (permalink / raw) To: linux-arm-kernel Cc: Mauro Carvalho Chehab, Tiffany Lin, Andrew-CT Chen, Matthias Brugger, Stanimir Varbanov, Kyungmin Park, Kamil Debski, Jeongtae Park, Andrzej Hajda, Hans Verkuil, Keiichi Watanabe, Jonathan Corbet, Sakari Ailus, Smitha T Murthy, Ricardo Ribalda Delgado, Tomasz Figa, Andy Shevchenko, linux-media, linux-kernel, linux-mediatek, linux-arm-msm, s.nawrocki Add a menu control V4L2_CID_MPEG_VIDEO_VP8_PROFILE for VP8 profile and make V4L2_CID_MPEG_VIDEO_VPX_PROFILE an alias of it. This new control is used to select a desired profile for VP8 encoder, and query for supported profiles by VP8 encoder/decoder. Though we have originally a control V4L2_CID_MPEG_VIDEO_VPX_PROFILE and its name contains 'VPX', it works only for VP8 because supported profiles usually differ between VP8 and VP9. In addition, this contorol cannot be used for querying since it is not a menu control but an integer control, which cannot return an arbitrary set of supported profiles. The new control V4L2_CID_MPEG_VIDEO_VP8_PROFILE is a menu control as with controls for other codec profiles. (e.g. H264) In addition, this patch also fixes the use of V4L2_CID_MPEG_VIDEO_VPX_PROFILE in drivers of Qualcomm's venus and Samsung's s5p-mfc. Signed-off-by: Keiichi Watanabe <keiichiw@chromium.org> --- .../media/uapi/v4l/extended-controls.rst | 27 ++++++++++++++++--- drivers/media/platform/qcom/venus/core.h | 2 +- .../media/platform/qcom/venus/hfi_helper.h | 12 ++++----- .../media/platform/qcom/venus/vdec_ctrls.c | 10 ++++--- drivers/media/platform/qcom/venus/venc.c | 14 +++++----- .../media/platform/qcom/venus/venc_ctrls.c | 12 +++++---- drivers/media/platform/s5p-mfc/s5p_mfc_enc.c | 15 +++++------ drivers/media/v4l2-core/v4l2-ctrls.c | 12 ++++++++- include/uapi/linux/v4l2-controls.h | 11 +++++++- 9 files changed, 79 insertions(+), 36 deletions(-) diff --git a/Documentation/media/uapi/v4l/extended-controls.rst b/Documentation/media/uapi/v4l/extended-controls.rst index 03931f9b1285..de99eafb0872 100644 --- a/Documentation/media/uapi/v4l/extended-controls.rst +++ b/Documentation/media/uapi/v4l/extended-controls.rst @@ -1955,9 +1955,30 @@ enum v4l2_vp8_golden_frame_sel - ``V4L2_CID_MPEG_VIDEO_VPX_P_FRAME_QP (integer)`` Quantization parameter for a P frame for VP8. -``V4L2_CID_MPEG_VIDEO_VPX_PROFILE (integer)`` - Select the desired profile for VPx encoder. Acceptable values are 0, - 1, 2 and 3 corresponding to encoder profiles 0, 1, 2 and 3. +.. _v4l2-mpeg-video-vp8-profile: + +``V4L2_CID_MPEG_VIDEO_VP8_PROFILE`` + (enum) + +enum v4l2_mpeg_video_vp8_profile - + This control allows to select the profile for VP8 encoder. + This is also used to enumerate supported profiles by VP8 encoder or decoder. + Possible values are: + + + +.. flat-table:: + :header-rows: 0 + :stub-columns: 0 + + * - ``V4L2_MPEG_VIDEO_VP8_PROFILE_0`` + - Profile 0 + * - ``V4L2_MPEG_VIDEO_VP8_PROFILE_1`` + - Profile 1 + * - ``V4L2_MPEG_VIDEO_VP8_PROFILE_2`` + - Profile 2 + * - ``V4L2_MPEG_VIDEO_VP8_PROFILE_3`` + - Profile 3 High Efficiency Video Coding (HEVC/H.265) Control Reference diff --git a/drivers/media/platform/qcom/venus/core.h b/drivers/media/platform/qcom/venus/core.h index 0360d295f4c8..f242e7f9f6a2 100644 --- a/drivers/media/platform/qcom/venus/core.h +++ b/drivers/media/platform/qcom/venus/core.h @@ -159,7 +159,7 @@ struct venc_controls { struct { u32 mpeg4; u32 h264; - u32 vpx; + u32 vp8; } profile; struct { u32 mpeg4; diff --git a/drivers/media/platform/qcom/venus/hfi_helper.h b/drivers/media/platform/qcom/venus/hfi_helper.h index 55d8eb21403a..07bf49dd2ec6 100644 --- a/drivers/media/platform/qcom/venus/hfi_helper.h +++ b/drivers/media/platform/qcom/venus/hfi_helper.h @@ -333,12 +333,12 @@ #define HFI_VC1_LEVEL_3 0x00000040 #define HFI_VC1_LEVEL_4 0x00000080 -#define HFI_VPX_PROFILE_SIMPLE 0x00000001 -#define HFI_VPX_PROFILE_ADVANCED 0x00000002 -#define HFI_VPX_PROFILE_VERSION_0 0x00000004 -#define HFI_VPX_PROFILE_VERSION_1 0x00000008 -#define HFI_VPX_PROFILE_VERSION_2 0x00000010 -#define HFI_VPX_PROFILE_VERSION_3 0x00000020 +#define HFI_VP8_PROFILE_SIMPLE 0x00000001 +#define HFI_VP8_PROFILE_ADVANCED 0x00000002 +#define HFI_VP8_PROFILE_VERSION_0 0x00000004 +#define HFI_VP8_PROFILE_VERSION_1 0x00000008 +#define HFI_VP8_PROFILE_VERSION_2 0x00000010 +#define HFI_VP8_PROFILE_VERSION_3 0x00000020 #define HFI_DIVX_FORMAT_4 0x1 #define HFI_DIVX_FORMAT_5 0x2 diff --git a/drivers/media/platform/qcom/venus/vdec_ctrls.c b/drivers/media/platform/qcom/venus/vdec_ctrls.c index 032839bbc967..f4604b0cd57e 100644 --- a/drivers/media/platform/qcom/venus/vdec_ctrls.c +++ b/drivers/media/platform/qcom/venus/vdec_ctrls.c @@ -29,7 +29,7 @@ static int vdec_op_s_ctrl(struct v4l2_ctrl *ctrl) break; case V4L2_CID_MPEG_VIDEO_H264_PROFILE: case V4L2_CID_MPEG_VIDEO_MPEG4_PROFILE: - case V4L2_CID_MPEG_VIDEO_VPX_PROFILE: + case V4L2_CID_MPEG_VIDEO_VP8_PROFILE: ctr->profile = ctrl->val; break; case V4L2_CID_MPEG_VIDEO_H264_LEVEL: @@ -54,7 +54,7 @@ static int vdec_op_g_volatile_ctrl(struct v4l2_ctrl *ctrl) switch (ctrl->id) { case V4L2_CID_MPEG_VIDEO_H264_PROFILE: case V4L2_CID_MPEG_VIDEO_MPEG4_PROFILE: - case V4L2_CID_MPEG_VIDEO_VPX_PROFILE: + case V4L2_CID_MPEG_VIDEO_VP8_PROFILE: ret = hfi_session_get_property(inst, ptype, &hprop); if (!ret) ctr->profile = hprop.profile_level.profile; @@ -130,8 +130,10 @@ int vdec_ctrl_init(struct venus_inst *inst) if (ctrl) ctrl->flags |= V4L2_CTRL_FLAG_VOLATILE; - ctrl = v4l2_ctrl_new_std(&inst->ctrl_handler, &vdec_ctrl_ops, - V4L2_CID_MPEG_VIDEO_VPX_PROFILE, 0, 3, 1, 0); + ctrl = v4l2_ctrl_new_std_menu(&inst->ctrl_handler, &vdec_ctrl_ops, + V4L2_CID_MPEG_VIDEO_VP8_PROFILE, + V4L2_MPEG_VIDEO_VP8_PROFILE_3, + 0, V4L2_MPEG_VIDEO_VP8_PROFILE_0); if (ctrl) ctrl->flags |= V4L2_CTRL_FLAG_VOLATILE; diff --git a/drivers/media/platform/qcom/venus/venc.c b/drivers/media/platform/qcom/venus/venc.c index 6b2ce479584e..aa54dd005c3e 100644 --- a/drivers/media/platform/qcom/venus/venc.c +++ b/drivers/media/platform/qcom/venus/venc.c @@ -223,17 +223,17 @@ static int venc_v4l2_to_hfi(int id, int value) case V4L2_MPEG_VIDEO_H264_ENTROPY_MODE_CABAC: return HFI_H264_ENTROPY_CABAC; } - case V4L2_CID_MPEG_VIDEO_VPX_PROFILE: + case V4L2_CID_MPEG_VIDEO_VP8_PROFILE: switch (value) { case 0: default: - return HFI_VPX_PROFILE_VERSION_0; + return HFI_VP8_PROFILE_VERSION_0; case 1: - return HFI_VPX_PROFILE_VERSION_1; + return HFI_VP8_PROFILE_VERSION_1; case 2: - return HFI_VPX_PROFILE_VERSION_2; + return HFI_VP8_PROFILE_VERSION_2; case 3: - return HFI_VPX_PROFILE_VERSION_3; + return HFI_VP8_PROFILE_VERSION_3; } case V4L2_CID_MPEG_VIDEO_H264_LOOP_FILTER_MODE: switch (value) { @@ -756,8 +756,8 @@ static int venc_set_properties(struct venus_inst *inst) level = venc_v4l2_to_hfi(V4L2_CID_MPEG_VIDEO_H264_LEVEL, ctr->level.h264); } else if (inst->fmt_cap->pixfmt == V4L2_PIX_FMT_VP8) { - profile = venc_v4l2_to_hfi(V4L2_CID_MPEG_VIDEO_VPX_PROFILE, - ctr->profile.vpx); + profile = venc_v4l2_to_hfi(V4L2_CID_MPEG_VIDEO_VP8_PROFILE, + ctr->profile.vp8); level = 0; } else if (inst->fmt_cap->pixfmt == V4L2_PIX_FMT_MPEG4) { profile = venc_v4l2_to_hfi(V4L2_CID_MPEG_VIDEO_MPEG4_PROFILE, diff --git a/drivers/media/platform/qcom/venus/venc_ctrls.c b/drivers/media/platform/qcom/venus/venc_ctrls.c index 21e938a28662..e5162b78609d 100644 --- a/drivers/media/platform/qcom/venus/venc_ctrls.c +++ b/drivers/media/platform/qcom/venus/venc_ctrls.c @@ -101,8 +101,8 @@ static int venc_op_s_ctrl(struct v4l2_ctrl *ctrl) case V4L2_CID_MPEG_VIDEO_H264_PROFILE: ctr->profile.h264 = ctrl->val; break; - case V4L2_CID_MPEG_VIDEO_VPX_PROFILE: - ctr->profile.vpx = ctrl->val; + case V4L2_CID_MPEG_VIDEO_VP8_PROFILE: + ctr->profile.vp8 = ctrl->val; break; case V4L2_CID_MPEG_VIDEO_MPEG4_LEVEL: ctr->level.mpeg4 = ctrl->val; @@ -248,6 +248,11 @@ int venc_ctrl_init(struct venus_inst *inst) V4L2_MPEG_VIDEO_MULTI_SICE_MODE_MAX_BYTES, 0, V4L2_MPEG_VIDEO_MULTI_SLICE_MODE_SINGLE); + v4l2_ctrl_new_std_menu(&inst->ctrl_handler, &venc_ctrl_ops, + V4L2_CID_MPEG_VIDEO_VP8_PROFILE, + V4L2_MPEG_VIDEO_VP8_PROFILE_3, + 0, V4L2_MPEG_VIDEO_VP8_PROFILE_0); + v4l2_ctrl_new_std(&inst->ctrl_handler, &venc_ctrl_ops, V4L2_CID_MPEG_VIDEO_BITRATE, BITRATE_MIN, BITRATE_MAX, BITRATE_STEP, BITRATE_DEFAULT); @@ -256,9 +261,6 @@ int venc_ctrl_init(struct venus_inst *inst) V4L2_CID_MPEG_VIDEO_BITRATE_PEAK, BITRATE_MIN, BITRATE_MAX, BITRATE_STEP, BITRATE_DEFAULT_PEAK); - v4l2_ctrl_new_std(&inst->ctrl_handler, &venc_ctrl_ops, - V4L2_CID_MPEG_VIDEO_VPX_PROFILE, 0, 3, 1, 0); - v4l2_ctrl_new_std(&inst->ctrl_handler, &venc_ctrl_ops, V4L2_CID_MPEG_VIDEO_H264_I_FRAME_QP, 1, 51, 1, 26); diff --git a/drivers/media/platform/s5p-mfc/s5p_mfc_enc.c b/drivers/media/platform/s5p-mfc/s5p_mfc_enc.c index 570f391f2cfd..3ad4f5073002 100644 --- a/drivers/media/platform/s5p-mfc/s5p_mfc_enc.c +++ b/drivers/media/platform/s5p-mfc/s5p_mfc_enc.c @@ -692,12 +692,12 @@ static struct mfc_control controls[] = { .default_value = 10, }, { - .id = V4L2_CID_MPEG_VIDEO_VPX_PROFILE, - .type = V4L2_CTRL_TYPE_INTEGER, - .minimum = 0, - .maximum = 3, - .step = 1, - .default_value = 0, + .id = V4L2_CID_MPEG_VIDEO_VP8_PROFILE, + .type = V4L2_CTRL_TYPE_MENU, + .minimum = V4L2_MPEG_VIDEO_VP8_PROFILE_0, + .maximum = V4L2_MPEG_VIDEO_VP8_PROFILE_3, + .default_value = V4L2_MPEG_VIDEO_VP8_PROFILE_0, + .menu_skip_mask = 0, }, { .id = V4L2_CID_MPEG_VIDEO_HEVC_I_FRAME_QP, @@ -2057,7 +2057,7 @@ static int s5p_mfc_enc_s_ctrl(struct v4l2_ctrl *ctrl) case V4L2_CID_MPEG_VIDEO_VPX_P_FRAME_QP: p->codec.vp8.rc_p_frame_qp = ctrl->val; break; - case V4L2_CID_MPEG_VIDEO_VPX_PROFILE: + case V4L2_CID_MPEG_VIDEO_VP8_PROFILE: p->codec.vp8.profile = ctrl->val; break; case V4L2_CID_MPEG_VIDEO_HEVC_I_FRAME_QP: @@ -2711,4 +2711,3 @@ void s5p_mfc_enc_init(struct s5p_mfc_ctx *ctx) f.fmt.pix_mp.pixelformat = DEF_DST_FMT_ENC; ctx->dst_fmt = find_format(&f, MFC_FMT_ENC); } - diff --git a/drivers/media/v4l2-core/v4l2-ctrls.c b/drivers/media/v4l2-core/v4l2-ctrls.c index d29e45516eb7..e7e6340b395e 100644 --- a/drivers/media/v4l2-core/v4l2-ctrls.c +++ b/drivers/media/v4l2-core/v4l2-ctrls.c @@ -431,6 +431,13 @@ const char * const *v4l2_ctrl_get_menu(u32 id) "Use Previous Specific Frame", NULL, }; + static const char * const vp8_profile[] = { + "0", + "1", + "2", + "3", + NULL, + }; static const char * const flash_led_mode[] = { "Off", @@ -614,6 +621,8 @@ const char * const *v4l2_ctrl_get_menu(u32 id) return mpeg4_profile; case V4L2_CID_MPEG_VIDEO_VPX_GOLDEN_FRAME_SEL: return vpx_golden_frame_sel; + case V4L2_CID_MPEG_VIDEO_VP8_PROFILE: + return vp8_profile; case V4L2_CID_JPEG_CHROMA_SUBSAMPLING: return jpeg_chroma_subsampling; case V4L2_CID_DV_TX_MODE: @@ -839,7 +848,7 @@ const char *v4l2_ctrl_get_name(u32 id) case V4L2_CID_MPEG_VIDEO_VPX_MAX_QP: return "VPX Maximum QP Value"; case V4L2_CID_MPEG_VIDEO_VPX_I_FRAME_QP: return "VPX I-Frame QP Value"; case V4L2_CID_MPEG_VIDEO_VPX_P_FRAME_QP: return "VPX P-Frame QP Value"; - case V4L2_CID_MPEG_VIDEO_VPX_PROFILE: return "VPX Profile"; + case V4L2_CID_MPEG_VIDEO_VP8_PROFILE: return "VP8 Profile"; /* HEVC controls */ case V4L2_CID_MPEG_VIDEO_HEVC_I_FRAME_QP: return "HEVC I-Frame QP Value"; @@ -1180,6 +1189,7 @@ void v4l2_ctrl_fill(u32 id, const char **name, enum v4l2_ctrl_type *type, case V4L2_CID_DEINTERLACING_MODE: case V4L2_CID_TUNE_DEEMPHASIS: case V4L2_CID_MPEG_VIDEO_VPX_GOLDEN_FRAME_SEL: + case V4L2_CID_MPEG_VIDEO_VP8_PROFILE: 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 8d473c979b61..2001823c3072 100644 --- a/include/uapi/linux/v4l2-controls.h +++ b/include/uapi/linux/v4l2-controls.h @@ -587,7 +587,16 @@ enum v4l2_vp8_golden_frame_sel { #define V4L2_CID_MPEG_VIDEO_VPX_MAX_QP (V4L2_CID_MPEG_BASE+508) #define V4L2_CID_MPEG_VIDEO_VPX_I_FRAME_QP (V4L2_CID_MPEG_BASE+509) #define V4L2_CID_MPEG_VIDEO_VPX_P_FRAME_QP (V4L2_CID_MPEG_BASE+510) -#define V4L2_CID_MPEG_VIDEO_VPX_PROFILE (V4L2_CID_MPEG_BASE+511) + +#define V4L2_CID_MPEG_VIDEO_VP8_PROFILE (V4L2_CID_MPEG_BASE+511) +enum v4l2_mpeg_video_vp8_profile { + V4L2_MPEG_VIDEO_VP8_PROFILE_0 = 0, + V4L2_MPEG_VIDEO_VP8_PROFILE_1 = 1, + V4L2_MPEG_VIDEO_VP8_PROFILE_2 = 2, + V4L2_MPEG_VIDEO_VP8_PROFILE_3 = 3, +}; +/* Deprecated alias for compatibility reasons. */ +#define V4L2_CID_MPEG_VIDEO_VPX_PROFILE V4L2_CID_MPEG_VIDEO_VP8_PROFILE /* CIDs for HEVC encoding. */ -- 2.18.0.rc1.242.g61856ae69a-goog ^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH v3 1/3] media: v4l2-ctrl: Change control for VP8 profile to menu control 2018-06-14 7:46 ` [PATCH v3 1/3] media: v4l2-ctrl: Change control for VP8 profile to menu control Keiichi Watanabe @ 2018-06-14 8:04 ` Stanimir Varbanov 2018-06-14 8:13 ` Keiichi Watanabe 2018-06-15 8:51 ` Hans Verkuil 1 sibling, 1 reply; 9+ messages in thread From: Stanimir Varbanov @ 2018-06-14 8:04 UTC (permalink / raw) To: Keiichi Watanabe, linux-arm-kernel Cc: Mauro Carvalho Chehab, Tiffany Lin, Andrew-CT Chen, Matthias Brugger, Kyungmin Park, Kamil Debski, Jeongtae Park, Andrzej Hajda, Hans Verkuil, Jonathan Corbet, Sakari Ailus, Smitha T Murthy, Ricardo Ribalda Delgado, Tomasz Figa, Andy Shevchenko, linux-media, linux-kernel, linux-mediatek, linux-arm-msm, s.nawrocki Hi Keiichi, On 06/14/2018 10:46 AM, Keiichi Watanabe wrote: > Add a menu control V4L2_CID_MPEG_VIDEO_VP8_PROFILE for VP8 profile and > make V4L2_CID_MPEG_VIDEO_VPX_PROFILE an alias of it. This new control > is used to select a desired profile for VP8 encoder, and query for > supported profiles by VP8 encoder/decoder. > > Though we have originally a control V4L2_CID_MPEG_VIDEO_VPX_PROFILE and its name > contains 'VPX', it works only for VP8 because supported profiles usually differ > between VP8 and VP9. In addition, this contorol cannot be used for querying > since it is not a menu control but an integer control, which cannot return an > arbitrary set of supported profiles. > > The new control V4L2_CID_MPEG_VIDEO_VP8_PROFILE is a menu control as with > controls for other codec profiles. (e.g. H264) > > In addition, this patch also fixes the use of > V4L2_CID_MPEG_VIDEO_VPX_PROFILE in drivers of Qualcomm's venus and > Samsung's s5p-mfc. > > Signed-off-by: Keiichi Watanabe <keiichiw@chromium.org> > --- > .../media/uapi/v4l/extended-controls.rst | 27 ++++++++++++++++--- > drivers/media/platform/qcom/venus/core.h | 2 +- > .../media/platform/qcom/venus/hfi_helper.h | 12 ++++----- > .../media/platform/qcom/venus/vdec_ctrls.c | 10 ++++--- > drivers/media/platform/qcom/venus/venc.c | 14 +++++----- > .../media/platform/qcom/venus/venc_ctrls.c | 12 +++++---- > drivers/media/platform/s5p-mfc/s5p_mfc_enc.c | 15 +++++------ > drivers/media/v4l2-core/v4l2-ctrls.c | 12 ++++++++- > include/uapi/linux/v4l2-controls.h | 11 +++++++- > 9 files changed, 79 insertions(+), 36 deletions(-) > > diff --git a/Documentation/media/uapi/v4l/extended-controls.rst b/Documentation/media/uapi/v4l/extended-controls.rst > index 03931f9b1285..de99eafb0872 100644 > --- a/Documentation/media/uapi/v4l/extended-controls.rst > +++ b/Documentation/media/uapi/v4l/extended-controls.rst > @@ -1955,9 +1955,30 @@ enum v4l2_vp8_golden_frame_sel - > ``V4L2_CID_MPEG_VIDEO_VPX_P_FRAME_QP (integer)`` > Quantization parameter for a P frame for VP8. > > -``V4L2_CID_MPEG_VIDEO_VPX_PROFILE (integer)`` > - Select the desired profile for VPx encoder. Acceptable values are 0, > - 1, 2 and 3 corresponding to encoder profiles 0, 1, 2 and 3. > +.. _v4l2-mpeg-video-vp8-profile: > + > +``V4L2_CID_MPEG_VIDEO_VP8_PROFILE`` > + (enum) > + > +enum v4l2_mpeg_video_vp8_profile - > + This control allows to select the profile for VP8 encoder. > + This is also used to enumerate supported profiles by VP8 encoder or decoder. > + Possible values are: > + > + > + > +.. flat-table:: > + :header-rows: 0 > + :stub-columns: 0 > + > + * - ``V4L2_MPEG_VIDEO_VP8_PROFILE_0`` > + - Profile 0 > + * - ``V4L2_MPEG_VIDEO_VP8_PROFILE_1`` > + - Profile 1 > + * - ``V4L2_MPEG_VIDEO_VP8_PROFILE_2`` > + - Profile 2 > + * - ``V4L2_MPEG_VIDEO_VP8_PROFILE_3`` > + - Profile 3 > > > High Efficiency Video Coding (HEVC/H.265) Control Reference > diff --git a/drivers/media/platform/qcom/venus/core.h b/drivers/media/platform/qcom/venus/core.h > index 0360d295f4c8..f242e7f9f6a2 100644 > --- a/drivers/media/platform/qcom/venus/core.h > +++ b/drivers/media/platform/qcom/venus/core.h > @@ -159,7 +159,7 @@ struct venc_controls { > struct { > u32 mpeg4; > u32 h264; > - u32 vpx; > + u32 vp8; > } profile; > struct { > u32 mpeg4; > diff --git a/drivers/media/platform/qcom/venus/hfi_helper.h b/drivers/media/platform/qcom/venus/hfi_helper.h > index 55d8eb21403a..07bf49dd2ec6 100644 > --- a/drivers/media/platform/qcom/venus/hfi_helper.h > +++ b/drivers/media/platform/qcom/venus/hfi_helper.h > @@ -333,12 +333,12 @@ > #define HFI_VC1_LEVEL_3 0x00000040 > #define HFI_VC1_LEVEL_4 0x00000080 > > -#define HFI_VPX_PROFILE_SIMPLE 0x00000001 > -#define HFI_VPX_PROFILE_ADVANCED 0x00000002 > -#define HFI_VPX_PROFILE_VERSION_0 0x00000004 > -#define HFI_VPX_PROFILE_VERSION_1 0x00000008 > -#define HFI_VPX_PROFILE_VERSION_2 0x00000010 > -#define HFI_VPX_PROFILE_VERSION_3 0x00000020 > +#define HFI_VP8_PROFILE_SIMPLE 0x00000001 > +#define HFI_VP8_PROFILE_ADVANCED 0x00000002 > +#define HFI_VP8_PROFILE_VERSION_0 0x00000004 > +#define HFI_VP8_PROFILE_VERSION_1 0x00000008 > +#define HFI_VP8_PROFILE_VERSION_2 0x00000010 > +#define HFI_VP8_PROFILE_VERSION_3 0x00000020 Please do not rename these driver internal defines, just leave VPX as it is now. -- regards, Stan ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH v3 1/3] media: v4l2-ctrl: Change control for VP8 profile to menu control 2018-06-14 8:04 ` Stanimir Varbanov @ 2018-06-14 8:13 ` Keiichi Watanabe 0 siblings, 0 replies; 9+ messages in thread From: Keiichi Watanabe @ 2018-06-14 8:13 UTC (permalink / raw) To: Stanimir Varbanov Cc: list@263.net:IOMMU DRIVERS <iommu@lists.linux-foundation.org>, Joerg Roedel <joro@8bytes.org>,, Mauro Carvalho Chehab, Tiffany Lin, Andrew-CT Chen, Matthias Brugger, Kyungmin Park, Kamil Debski, Jeongtae Park, Andrzej Hajda, Hans Verkuil, Jonathan Corbet, Sakari Ailus, Smitha T Murthy, Ricardo Ribalda Delgado, Tomasz Figa, Andy Shevchenko, Linux Media Mailing List, Linux Kernel Mailing List, linux-mediatek, linux-arm-msm, Sylwester Nawrocki Thanks, Stanimir On Thu, Jun 14, 2018 at 5:04 PM, Stanimir Varbanov <stanimir.varbanov@linaro.org> wrote: > Hi Keiichi, > > On 06/14/2018 10:46 AM, Keiichi Watanabe wrote: >> Add a menu control V4L2_CID_MPEG_VIDEO_VP8_PROFILE for VP8 profile and >> make V4L2_CID_MPEG_VIDEO_VPX_PROFILE an alias of it. This new control >> is used to select a desired profile for VP8 encoder, and query for >> supported profiles by VP8 encoder/decoder. >> >> Though we have originally a control V4L2_CID_MPEG_VIDEO_VPX_PROFILE and its name >> contains 'VPX', it works only for VP8 because supported profiles usually differ >> between VP8 and VP9. In addition, this contorol cannot be used for querying >> since it is not a menu control but an integer control, which cannot return an >> arbitrary set of supported profiles. >> >> The new control V4L2_CID_MPEG_VIDEO_VP8_PROFILE is a menu control as with >> controls for other codec profiles. (e.g. H264) >> >> In addition, this patch also fixes the use of >> V4L2_CID_MPEG_VIDEO_VPX_PROFILE in drivers of Qualcomm's venus and >> Samsung's s5p-mfc. >> >> Signed-off-by: Keiichi Watanabe <keiichiw@chromium.org> >> --- >> .../media/uapi/v4l/extended-controls.rst | 27 ++++++++++++++++--- >> drivers/media/platform/qcom/venus/core.h | 2 +- >> .../media/platform/qcom/venus/hfi_helper.h | 12 ++++----- >> .../media/platform/qcom/venus/vdec_ctrls.c | 10 ++++--- >> drivers/media/platform/qcom/venus/venc.c | 14 +++++----- >> .../media/platform/qcom/venus/venc_ctrls.c | 12 +++++---- >> drivers/media/platform/s5p-mfc/s5p_mfc_enc.c | 15 +++++------ >> drivers/media/v4l2-core/v4l2-ctrls.c | 12 ++++++++- >> include/uapi/linux/v4l2-controls.h | 11 +++++++- >> 9 files changed, 79 insertions(+), 36 deletions(-) >> >> diff --git a/Documentation/media/uapi/v4l/extended-controls.rst b/Documentation/media/uapi/v4l/extended-controls.rst >> index 03931f9b1285..de99eafb0872 100644 >> --- a/Documentation/media/uapi/v4l/extended-controls.rst >> +++ b/Documentation/media/uapi/v4l/extended-controls.rst >> @@ -1955,9 +1955,30 @@ enum v4l2_vp8_golden_frame_sel - >> ``V4L2_CID_MPEG_VIDEO_VPX_P_FRAME_QP (integer)`` >> Quantization parameter for a P frame for VP8. >> >> -``V4L2_CID_MPEG_VIDEO_VPX_PROFILE (integer)`` >> - Select the desired profile for VPx encoder. Acceptable values are 0, >> - 1, 2 and 3 corresponding to encoder profiles 0, 1, 2 and 3. >> +.. _v4l2-mpeg-video-vp8-profile: >> + >> +``V4L2_CID_MPEG_VIDEO_VP8_PROFILE`` >> + (enum) >> + >> +enum v4l2_mpeg_video_vp8_profile - >> + This control allows to select the profile for VP8 encoder. >> + This is also used to enumerate supported profiles by VP8 encoder or decoder. >> + Possible values are: >> + >> + >> + >> +.. flat-table:: >> + :header-rows: 0 >> + :stub-columns: 0 >> + >> + * - ``V4L2_MPEG_VIDEO_VP8_PROFILE_0`` >> + - Profile 0 >> + * - ``V4L2_MPEG_VIDEO_VP8_PROFILE_1`` >> + - Profile 1 >> + * - ``V4L2_MPEG_VIDEO_VP8_PROFILE_2`` >> + - Profile 2 >> + * - ``V4L2_MPEG_VIDEO_VP8_PROFILE_3`` >> + - Profile 3 >> >> >> High Efficiency Video Coding (HEVC/H.265) Control Reference >> diff --git a/drivers/media/platform/qcom/venus/core.h b/drivers/media/platform/qcom/venus/core.h >> index 0360d295f4c8..f242e7f9f6a2 100644 >> --- a/drivers/media/platform/qcom/venus/core.h >> +++ b/drivers/media/platform/qcom/venus/core.h >> @@ -159,7 +159,7 @@ struct venc_controls { >> struct { >> u32 mpeg4; >> u32 h264; >> - u32 vpx; >> + u32 vp8; >> } profile; >> struct { >> u32 mpeg4; >> diff --git a/drivers/media/platform/qcom/venus/hfi_helper.h b/drivers/media/platform/qcom/venus/hfi_helper.h >> index 55d8eb21403a..07bf49dd2ec6 100644 >> --- a/drivers/media/platform/qcom/venus/hfi_helper.h >> +++ b/drivers/media/platform/qcom/venus/hfi_helper.h >> @@ -333,12 +333,12 @@ >> #define HFI_VC1_LEVEL_3 0x00000040 >> #define HFI_VC1_LEVEL_4 0x00000080 >> >> -#define HFI_VPX_PROFILE_SIMPLE 0x00000001 >> -#define HFI_VPX_PROFILE_ADVANCED 0x00000002 >> -#define HFI_VPX_PROFILE_VERSION_0 0x00000004 >> -#define HFI_VPX_PROFILE_VERSION_1 0x00000008 >> -#define HFI_VPX_PROFILE_VERSION_2 0x00000010 >> -#define HFI_VPX_PROFILE_VERSION_3 0x00000020 >> +#define HFI_VP8_PROFILE_SIMPLE 0x00000001 >> +#define HFI_VP8_PROFILE_ADVANCED 0x00000002 >> +#define HFI_VP8_PROFILE_VERSION_0 0x00000004 >> +#define HFI_VP8_PROFILE_VERSION_1 0x00000008 >> +#define HFI_VP8_PROFILE_VERSION_2 0x00000010 >> +#define HFI_VP8_PROFILE_VERSION_3 0x00000020 > > Please do not rename these driver internal defines, just leave VPX as it > is now. > I got it. I'll fix it in the next revision of the patch. Best regards, Keiichi > > -- > regards, > Stan ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH v3 1/3] media: v4l2-ctrl: Change control for VP8 profile to menu control 2018-06-14 7:46 ` [PATCH v3 1/3] media: v4l2-ctrl: Change control for VP8 profile to menu control Keiichi Watanabe 2018-06-14 8:04 ` Stanimir Varbanov @ 2018-06-15 8:51 ` Hans Verkuil 1 sibling, 0 replies; 9+ messages in thread From: Hans Verkuil @ 2018-06-15 8:51 UTC (permalink / raw) To: Keiichi Watanabe, linux-arm-kernel Cc: Mauro Carvalho Chehab, Tiffany Lin, Andrew-CT Chen, Matthias Brugger, Stanimir Varbanov, Kyungmin Park, Kamil Debski, Jeongtae Park, Andrzej Hajda, Jonathan Corbet, Sakari Ailus, Smitha T Murthy, Ricardo Ribalda Delgado, Tomasz Figa, Andy Shevchenko, linux-media, linux-kernel, linux-mediatek, linux-arm-msm, s.nawrocki On 14/06/18 09:46, Keiichi Watanabe wrote: > Add a menu control V4L2_CID_MPEG_VIDEO_VP8_PROFILE for VP8 profile and > make V4L2_CID_MPEG_VIDEO_VPX_PROFILE an alias of it. This new control > is used to select a desired profile for VP8 encoder, and query for > supported profiles by VP8 encoder/decoder. > > Though we have originally a control V4L2_CID_MPEG_VIDEO_VPX_PROFILE and its name > contains 'VPX', it works only for VP8 because supported profiles usually differ > between VP8 and VP9. In addition, this contorol cannot be used for querying typo: contorol -> control > since it is not a menu control but an integer control, which cannot return an > arbitrary set of supported profiles. > > The new control V4L2_CID_MPEG_VIDEO_VP8_PROFILE is a menu control as with > controls for other codec profiles. (e.g. H264) > > In addition, this patch also fixes the use of > V4L2_CID_MPEG_VIDEO_VPX_PROFILE in drivers of Qualcomm's venus and > Samsung's s5p-mfc. > > Signed-off-by: Keiichi Watanabe <keiichiw@chromium.org> > --- > .../media/uapi/v4l/extended-controls.rst | 27 ++++++++++++++++--- > drivers/media/platform/qcom/venus/core.h | 2 +- > .../media/platform/qcom/venus/hfi_helper.h | 12 ++++----- > .../media/platform/qcom/venus/vdec_ctrls.c | 10 ++++--- > drivers/media/platform/qcom/venus/venc.c | 14 +++++----- > .../media/platform/qcom/venus/venc_ctrls.c | 12 +++++---- > drivers/media/platform/s5p-mfc/s5p_mfc_enc.c | 15 +++++------ > drivers/media/v4l2-core/v4l2-ctrls.c | 12 ++++++++- > include/uapi/linux/v4l2-controls.h | 11 +++++++- > 9 files changed, 79 insertions(+), 36 deletions(-) > > diff --git a/Documentation/media/uapi/v4l/extended-controls.rst b/Documentation/media/uapi/v4l/extended-controls.rst > index 03931f9b1285..de99eafb0872 100644 > --- a/Documentation/media/uapi/v4l/extended-controls.rst > +++ b/Documentation/media/uapi/v4l/extended-controls.rst > @@ -1955,9 +1955,30 @@ enum v4l2_vp8_golden_frame_sel - > ``V4L2_CID_MPEG_VIDEO_VPX_P_FRAME_QP (integer)`` > Quantization parameter for a P frame for VP8. > > -``V4L2_CID_MPEG_VIDEO_VPX_PROFILE (integer)`` > - Select the desired profile for VPx encoder. Acceptable values are 0, > - 1, 2 and 3 corresponding to encoder profiles 0, 1, 2 and 3. > +.. _v4l2-mpeg-video-vp8-profile: > + > +``V4L2_CID_MPEG_VIDEO_VP8_PROFILE`` > + (enum) > + > +enum v4l2_mpeg_video_vp8_profile - > + This control allows to select the profile for VP8 encoder. > + This is also used to enumerate supported profiles by VP8 encoder or decoder. > + Possible values are: > + > + > + One empty line is enough. > +.. flat-table:: > + :header-rows: 0 > + :stub-columns: 0 > + > + * - ``V4L2_MPEG_VIDEO_VP8_PROFILE_0`` > + - Profile 0 > + * - ``V4L2_MPEG_VIDEO_VP8_PROFILE_1`` > + - Profile 1 > + * - ``V4L2_MPEG_VIDEO_VP8_PROFILE_2`` > + - Profile 2 > + * - ``V4L2_MPEG_VIDEO_VP8_PROFILE_3`` > + - Profile 3 > > > High Efficiency Video Coding (HEVC/H.265) Control Reference > diff --git a/drivers/media/platform/qcom/venus/core.h b/drivers/media/platform/qcom/venus/core.h > index 0360d295f4c8..f242e7f9f6a2 100644 > --- a/drivers/media/platform/qcom/venus/core.h > +++ b/drivers/media/platform/qcom/venus/core.h > @@ -159,7 +159,7 @@ struct venc_controls { > struct { > u32 mpeg4; > u32 h264; > - u32 vpx; > + u32 vp8; > } profile; > struct { > u32 mpeg4; > diff --git a/drivers/media/platform/qcom/venus/hfi_helper.h b/drivers/media/platform/qcom/venus/hfi_helper.h > index 55d8eb21403a..07bf49dd2ec6 100644 > --- a/drivers/media/platform/qcom/venus/hfi_helper.h > +++ b/drivers/media/platform/qcom/venus/hfi_helper.h > @@ -333,12 +333,12 @@ > #define HFI_VC1_LEVEL_3 0x00000040 > #define HFI_VC1_LEVEL_4 0x00000080 > > -#define HFI_VPX_PROFILE_SIMPLE 0x00000001 > -#define HFI_VPX_PROFILE_ADVANCED 0x00000002 > -#define HFI_VPX_PROFILE_VERSION_0 0x00000004 > -#define HFI_VPX_PROFILE_VERSION_1 0x00000008 > -#define HFI_VPX_PROFILE_VERSION_2 0x00000010 > -#define HFI_VPX_PROFILE_VERSION_3 0x00000020 > +#define HFI_VP8_PROFILE_SIMPLE 0x00000001 > +#define HFI_VP8_PROFILE_ADVANCED 0x00000002 > +#define HFI_VP8_PROFILE_VERSION_0 0x00000004 > +#define HFI_VP8_PROFILE_VERSION_1 0x00000008 > +#define HFI_VP8_PROFILE_VERSION_2 0x00000010 > +#define HFI_VP8_PROFILE_VERSION_3 0x00000020 > > #define HFI_DIVX_FORMAT_4 0x1 > #define HFI_DIVX_FORMAT_5 0x2 > diff --git a/drivers/media/platform/qcom/venus/vdec_ctrls.c b/drivers/media/platform/qcom/venus/vdec_ctrls.c > index 032839bbc967..f4604b0cd57e 100644 > --- a/drivers/media/platform/qcom/venus/vdec_ctrls.c > +++ b/drivers/media/platform/qcom/venus/vdec_ctrls.c > @@ -29,7 +29,7 @@ static int vdec_op_s_ctrl(struct v4l2_ctrl *ctrl) > break; > case V4L2_CID_MPEG_VIDEO_H264_PROFILE: > case V4L2_CID_MPEG_VIDEO_MPEG4_PROFILE: > - case V4L2_CID_MPEG_VIDEO_VPX_PROFILE: > + case V4L2_CID_MPEG_VIDEO_VP8_PROFILE: > ctr->profile = ctrl->val; > break; > case V4L2_CID_MPEG_VIDEO_H264_LEVEL: > @@ -54,7 +54,7 @@ static int vdec_op_g_volatile_ctrl(struct v4l2_ctrl *ctrl) > switch (ctrl->id) { > case V4L2_CID_MPEG_VIDEO_H264_PROFILE: > case V4L2_CID_MPEG_VIDEO_MPEG4_PROFILE: > - case V4L2_CID_MPEG_VIDEO_VPX_PROFILE: > + case V4L2_CID_MPEG_VIDEO_VP8_PROFILE: > ret = hfi_session_get_property(inst, ptype, &hprop); > if (!ret) > ctr->profile = hprop.profile_level.profile; > @@ -130,8 +130,10 @@ int vdec_ctrl_init(struct venus_inst *inst) > if (ctrl) > ctrl->flags |= V4L2_CTRL_FLAG_VOLATILE; > > - ctrl = v4l2_ctrl_new_std(&inst->ctrl_handler, &vdec_ctrl_ops, > - V4L2_CID_MPEG_VIDEO_VPX_PROFILE, 0, 3, 1, 0); > + ctrl = v4l2_ctrl_new_std_menu(&inst->ctrl_handler, &vdec_ctrl_ops, > + V4L2_CID_MPEG_VIDEO_VP8_PROFILE, > + V4L2_MPEG_VIDEO_VP8_PROFILE_3, > + 0, V4L2_MPEG_VIDEO_VP8_PROFILE_0); > if (ctrl) > ctrl->flags |= V4L2_CTRL_FLAG_VOLATILE; > > diff --git a/drivers/media/platform/qcom/venus/venc.c b/drivers/media/platform/qcom/venus/venc.c > index 6b2ce479584e..aa54dd005c3e 100644 > --- a/drivers/media/platform/qcom/venus/venc.c > +++ b/drivers/media/platform/qcom/venus/venc.c > @@ -223,17 +223,17 @@ static int venc_v4l2_to_hfi(int id, int value) > case V4L2_MPEG_VIDEO_H264_ENTROPY_MODE_CABAC: > return HFI_H264_ENTROPY_CABAC; > } > - case V4L2_CID_MPEG_VIDEO_VPX_PROFILE: > + case V4L2_CID_MPEG_VIDEO_VP8_PROFILE: > switch (value) { > case 0: > default: > - return HFI_VPX_PROFILE_VERSION_0; > + return HFI_VP8_PROFILE_VERSION_0; > case 1: > - return HFI_VPX_PROFILE_VERSION_1; > + return HFI_VP8_PROFILE_VERSION_1; > case 2: > - return HFI_VPX_PROFILE_VERSION_2; > + return HFI_VP8_PROFILE_VERSION_2; > case 3: > - return HFI_VPX_PROFILE_VERSION_3; > + return HFI_VP8_PROFILE_VERSION_3; > } > case V4L2_CID_MPEG_VIDEO_H264_LOOP_FILTER_MODE: > switch (value) { > @@ -756,8 +756,8 @@ static int venc_set_properties(struct venus_inst *inst) > level = venc_v4l2_to_hfi(V4L2_CID_MPEG_VIDEO_H264_LEVEL, > ctr->level.h264); > } else if (inst->fmt_cap->pixfmt == V4L2_PIX_FMT_VP8) { > - profile = venc_v4l2_to_hfi(V4L2_CID_MPEG_VIDEO_VPX_PROFILE, > - ctr->profile.vpx); > + profile = venc_v4l2_to_hfi(V4L2_CID_MPEG_VIDEO_VP8_PROFILE, > + ctr->profile.vp8); > level = 0; > } else if (inst->fmt_cap->pixfmt == V4L2_PIX_FMT_MPEG4) { > profile = venc_v4l2_to_hfi(V4L2_CID_MPEG_VIDEO_MPEG4_PROFILE, > diff --git a/drivers/media/platform/qcom/venus/venc_ctrls.c b/drivers/media/platform/qcom/venus/venc_ctrls.c > index 21e938a28662..e5162b78609d 100644 > --- a/drivers/media/platform/qcom/venus/venc_ctrls.c > +++ b/drivers/media/platform/qcom/venus/venc_ctrls.c > @@ -101,8 +101,8 @@ static int venc_op_s_ctrl(struct v4l2_ctrl *ctrl) > case V4L2_CID_MPEG_VIDEO_H264_PROFILE: > ctr->profile.h264 = ctrl->val; > break; > - case V4L2_CID_MPEG_VIDEO_VPX_PROFILE: > - ctr->profile.vpx = ctrl->val; > + case V4L2_CID_MPEG_VIDEO_VP8_PROFILE: > + ctr->profile.vp8 = ctrl->val; > break; > case V4L2_CID_MPEG_VIDEO_MPEG4_LEVEL: > ctr->level.mpeg4 = ctrl->val; > @@ -248,6 +248,11 @@ int venc_ctrl_init(struct venus_inst *inst) > V4L2_MPEG_VIDEO_MULTI_SICE_MODE_MAX_BYTES, > 0, V4L2_MPEG_VIDEO_MULTI_SLICE_MODE_SINGLE); > > + v4l2_ctrl_new_std_menu(&inst->ctrl_handler, &venc_ctrl_ops, > + V4L2_CID_MPEG_VIDEO_VP8_PROFILE, > + V4L2_MPEG_VIDEO_VP8_PROFILE_3, > + 0, V4L2_MPEG_VIDEO_VP8_PROFILE_0); > + > v4l2_ctrl_new_std(&inst->ctrl_handler, &venc_ctrl_ops, > V4L2_CID_MPEG_VIDEO_BITRATE, BITRATE_MIN, BITRATE_MAX, > BITRATE_STEP, BITRATE_DEFAULT); > @@ -256,9 +261,6 @@ int venc_ctrl_init(struct venus_inst *inst) > V4L2_CID_MPEG_VIDEO_BITRATE_PEAK, BITRATE_MIN, BITRATE_MAX, > BITRATE_STEP, BITRATE_DEFAULT_PEAK); > > - v4l2_ctrl_new_std(&inst->ctrl_handler, &venc_ctrl_ops, > - V4L2_CID_MPEG_VIDEO_VPX_PROFILE, 0, 3, 1, 0); > - > v4l2_ctrl_new_std(&inst->ctrl_handler, &venc_ctrl_ops, > V4L2_CID_MPEG_VIDEO_H264_I_FRAME_QP, 1, 51, 1, 26); > > diff --git a/drivers/media/platform/s5p-mfc/s5p_mfc_enc.c b/drivers/media/platform/s5p-mfc/s5p_mfc_enc.c > index 570f391f2cfd..3ad4f5073002 100644 > --- a/drivers/media/platform/s5p-mfc/s5p_mfc_enc.c > +++ b/drivers/media/platform/s5p-mfc/s5p_mfc_enc.c > @@ -692,12 +692,12 @@ static struct mfc_control controls[] = { > .default_value = 10, > }, > { > - .id = V4L2_CID_MPEG_VIDEO_VPX_PROFILE, > - .type = V4L2_CTRL_TYPE_INTEGER, > - .minimum = 0, > - .maximum = 3, > - .step = 1, > - .default_value = 0, > + .id = V4L2_CID_MPEG_VIDEO_VP8_PROFILE, > + .type = V4L2_CTRL_TYPE_MENU, > + .minimum = V4L2_MPEG_VIDEO_VP8_PROFILE_0, > + .maximum = V4L2_MPEG_VIDEO_VP8_PROFILE_3, > + .default_value = V4L2_MPEG_VIDEO_VP8_PROFILE_0, > + .menu_skip_mask = 0, > }, > { > .id = V4L2_CID_MPEG_VIDEO_HEVC_I_FRAME_QP, > @@ -2057,7 +2057,7 @@ static int s5p_mfc_enc_s_ctrl(struct v4l2_ctrl *ctrl) > case V4L2_CID_MPEG_VIDEO_VPX_P_FRAME_QP: > p->codec.vp8.rc_p_frame_qp = ctrl->val; > break; > - case V4L2_CID_MPEG_VIDEO_VPX_PROFILE: > + case V4L2_CID_MPEG_VIDEO_VP8_PROFILE: > p->codec.vp8.profile = ctrl->val; > break; > case V4L2_CID_MPEG_VIDEO_HEVC_I_FRAME_QP: > @@ -2711,4 +2711,3 @@ void s5p_mfc_enc_init(struct s5p_mfc_ctx *ctx) > f.fmt.pix_mp.pixelformat = DEF_DST_FMT_ENC; > ctx->dst_fmt = find_format(&f, MFC_FMT_ENC); > } > - > diff --git a/drivers/media/v4l2-core/v4l2-ctrls.c b/drivers/media/v4l2-core/v4l2-ctrls.c > index d29e45516eb7..e7e6340b395e 100644 > --- a/drivers/media/v4l2-core/v4l2-ctrls.c > +++ b/drivers/media/v4l2-core/v4l2-ctrls.c > @@ -431,6 +431,13 @@ const char * const *v4l2_ctrl_get_menu(u32 id) > "Use Previous Specific Frame", > NULL, > }; > + static const char * const vp8_profile[] = { > + "0", > + "1", > + "2", > + "3", > + NULL, > + }; > > static const char * const flash_led_mode[] = { > "Off", > @@ -614,6 +621,8 @@ const char * const *v4l2_ctrl_get_menu(u32 id) > return mpeg4_profile; > case V4L2_CID_MPEG_VIDEO_VPX_GOLDEN_FRAME_SEL: > return vpx_golden_frame_sel; > + case V4L2_CID_MPEG_VIDEO_VP8_PROFILE: > + return vp8_profile; > case V4L2_CID_JPEG_CHROMA_SUBSAMPLING: > return jpeg_chroma_subsampling; > case V4L2_CID_DV_TX_MODE: > @@ -839,7 +848,7 @@ const char *v4l2_ctrl_get_name(u32 id) > case V4L2_CID_MPEG_VIDEO_VPX_MAX_QP: return "VPX Maximum QP Value"; > case V4L2_CID_MPEG_VIDEO_VPX_I_FRAME_QP: return "VPX I-Frame QP Value"; > case V4L2_CID_MPEG_VIDEO_VPX_P_FRAME_QP: return "VPX P-Frame QP Value"; > - case V4L2_CID_MPEG_VIDEO_VPX_PROFILE: return "VPX Profile"; > + case V4L2_CID_MPEG_VIDEO_VP8_PROFILE: return "VP8 Profile"; > > /* HEVC controls */ > case V4L2_CID_MPEG_VIDEO_HEVC_I_FRAME_QP: return "HEVC I-Frame QP Value"; > @@ -1180,6 +1189,7 @@ void v4l2_ctrl_fill(u32 id, const char **name, enum v4l2_ctrl_type *type, > case V4L2_CID_DEINTERLACING_MODE: > case V4L2_CID_TUNE_DEEMPHASIS: > case V4L2_CID_MPEG_VIDEO_VPX_GOLDEN_FRAME_SEL: > + case V4L2_CID_MPEG_VIDEO_VP8_PROFILE: > 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 8d473c979b61..2001823c3072 100644 > --- a/include/uapi/linux/v4l2-controls.h > +++ b/include/uapi/linux/v4l2-controls.h > @@ -587,7 +587,16 @@ enum v4l2_vp8_golden_frame_sel { > #define V4L2_CID_MPEG_VIDEO_VPX_MAX_QP (V4L2_CID_MPEG_BASE+508) > #define V4L2_CID_MPEG_VIDEO_VPX_I_FRAME_QP (V4L2_CID_MPEG_BASE+509) > #define V4L2_CID_MPEG_VIDEO_VPX_P_FRAME_QP (V4L2_CID_MPEG_BASE+510) > -#define V4L2_CID_MPEG_VIDEO_VPX_PROFILE (V4L2_CID_MPEG_BASE+511) > + > +#define V4L2_CID_MPEG_VIDEO_VP8_PROFILE (V4L2_CID_MPEG_BASE+511) > +enum v4l2_mpeg_video_vp8_profile { > + V4L2_MPEG_VIDEO_VP8_PROFILE_0 = 0, > + V4L2_MPEG_VIDEO_VP8_PROFILE_1 = 1, > + V4L2_MPEG_VIDEO_VP8_PROFILE_2 = 2, > + V4L2_MPEG_VIDEO_VP8_PROFILE_3 = 3, > +}; > +/* Deprecated alias for compatibility reasons. */ > +#define V4L2_CID_MPEG_VIDEO_VPX_PROFILE V4L2_CID_MPEG_VIDEO_VP8_PROFILE > > /* CIDs for HEVC encoding. */ > > -- > 2.18.0.rc1.242.g61856ae69a-goog > Regards, Hans ^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH v3 2/3] media: v4l2-ctrl: Add control for VP9 profile 2018-06-14 7:46 [PATCH v3 0/3] Add controls for VP8/VP9 profile Keiichi Watanabe 2018-06-14 7:46 ` [PATCH v3 1/3] media: v4l2-ctrl: Change control for VP8 profile to menu control Keiichi Watanabe @ 2018-06-14 7:46 ` Keiichi Watanabe 2018-06-14 8:06 ` Stanimir Varbanov 2018-06-14 7:46 ` [PATCH v3 3/3] media: mtk-vcodec: Support VP9 profile in decoder Keiichi Watanabe 2 siblings, 1 reply; 9+ messages in thread From: Keiichi Watanabe @ 2018-06-14 7:46 UTC (permalink / raw) To: linux-arm-kernel Cc: Mauro Carvalho Chehab, Tiffany Lin, Andrew-CT Chen, Matthias Brugger, Stanimir Varbanov, Kyungmin Park, Kamil Debski, Jeongtae Park, Andrzej Hajda, Hans Verkuil, Keiichi Watanabe, Jonathan Corbet, Sakari Ailus, Smitha T Murthy, Ricardo Ribalda Delgado, Tomasz Figa, Andy Shevchenko, linux-media, linux-kernel, linux-mediatek, linux-arm-msm, s.nawrocki Add a new control V4L2_CID_MPEG_VIDEO_VP9_PROFILE for VP9 profiles. This control allows to select a desired profile for VP9 encoder and query for supported profiles by VP9 encoder/decoder. Though this control is similar to V4L2_CID_MPEG_VIDEO_VP8_PROFILE, we need to separate this control from it because supported profiles usually differ between VP8 and VP9. Signed-off-by: Keiichi Watanabe <keiichiw@chromium.org> --- .../media/uapi/v4l/extended-controls.rst | 25 +++++++++++++++++++ drivers/media/v4l2-core/v4l2-ctrls.c | 11 ++++++++ include/uapi/linux/v4l2-controls.h | 7 ++++++ 3 files changed, 43 insertions(+) diff --git a/Documentation/media/uapi/v4l/extended-controls.rst b/Documentation/media/uapi/v4l/extended-controls.rst index de99eafb0872..095b42e9d6fe 100644 --- a/Documentation/media/uapi/v4l/extended-controls.rst +++ b/Documentation/media/uapi/v4l/extended-controls.rst @@ -1980,6 +1980,31 @@ enum v4l2_mpeg_video_vp8_profile - * - ``V4L2_MPEG_VIDEO_VP8_PROFILE_3`` - Profile 3 +.. _v4l2-mpeg-video-vp9-profile: + +``V4L2_CID_MPEG_VIDEO_VP9_PROFILE`` + (enum) + +enum v4l2_mpeg_video_vp9_profile - + This control allows to select the profile for VP9 encoder. + This is also used to enumerate supported profiles by VP9 encoder or decoder. + Possible values are: + + + +.. flat-table:: + :header-rows: 0 + :stub-columns: 0 + + * - ``V4L2_MPEG_VIDEO_VP9_PROFILE_0`` + - Profile 0 + * - ``V4L2_MPEG_VIDEO_VP9_PROFILE_1`` + - Profile 1 + * - ``V4L2_MPEG_VIDEO_VP9_PROFILE_2`` + - Profile 2 + * - ``V4L2_MPEG_VIDEO_VP9_PROFILE_3`` + - Profile 3 + 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 e7e6340b395e..eacfab7574dc 100644 --- a/drivers/media/v4l2-core/v4l2-ctrls.c +++ b/drivers/media/v4l2-core/v4l2-ctrls.c @@ -438,6 +438,13 @@ const char * const *v4l2_ctrl_get_menu(u32 id) "3", NULL, }; + static const char * const vp9_profile[] = { + "0", + "1", + "2", + "3", + NULL, + }; static const char * const flash_led_mode[] = { "Off", @@ -623,6 +630,8 @@ const char * const *v4l2_ctrl_get_menu(u32 id) return vpx_golden_frame_sel; case V4L2_CID_MPEG_VIDEO_VP8_PROFILE: return vp8_profile; + case V4L2_CID_MPEG_VIDEO_VP9_PROFILE: + return vp9_profile; case V4L2_CID_JPEG_CHROMA_SUBSAMPLING: return jpeg_chroma_subsampling; case V4L2_CID_DV_TX_MODE: @@ -849,6 +858,7 @@ const char *v4l2_ctrl_get_name(u32 id) case V4L2_CID_MPEG_VIDEO_VPX_I_FRAME_QP: return "VPX I-Frame QP Value"; 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"; /* HEVC controls */ case V4L2_CID_MPEG_VIDEO_HEVC_I_FRAME_QP: return "HEVC I-Frame QP Value"; @@ -1190,6 +1200,7 @@ void v4l2_ctrl_fill(u32 id, const char **name, enum v4l2_ctrl_type *type, case V4L2_CID_TUNE_DEEMPHASIS: 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_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 2001823c3072..f03d3214eec2 100644 --- a/include/uapi/linux/v4l2-controls.h +++ b/include/uapi/linux/v4l2-controls.h @@ -597,6 +597,13 @@ enum v4l2_mpeg_video_vp8_profile { }; /* Deprecated alias for compatibility reasons. */ #define V4L2_CID_MPEG_VIDEO_VPX_PROFILE V4L2_CID_MPEG_VIDEO_VP8_PROFILE +#define V4L2_CID_MPEG_VIDEO_VP9_PROFILE (V4L2_CID_MPEG_BASE+512) +enum v4l2_mpeg_video_vp9_profile { + V4L2_MPEG_VIDEO_VP9_PROFILE_0 = 0, + V4L2_MPEG_VIDEO_VP9_PROFILE_1 = 1, + V4L2_MPEG_VIDEO_VP9_PROFILE_2 = 2, + V4L2_MPEG_VIDEO_VP9_PROFILE_3 = 3, +}; /* CIDs for HEVC encoding. */ -- 2.18.0.rc1.242.g61856ae69a-goog ^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH v3 2/3] media: v4l2-ctrl: Add control for VP9 profile 2018-06-14 7:46 ` [PATCH v3 2/3] media: v4l2-ctrl: Add control for VP9 profile Keiichi Watanabe @ 2018-06-14 8:06 ` Stanimir Varbanov 2018-06-14 9:12 ` Keiichi Watanabe 0 siblings, 1 reply; 9+ messages in thread From: Stanimir Varbanov @ 2018-06-14 8:06 UTC (permalink / raw) To: Keiichi Watanabe, linux-arm-kernel Cc: Mauro Carvalho Chehab, Tiffany Lin, Andrew-CT Chen, Matthias Brugger, Kyungmin Park, Kamil Debski, Jeongtae Park, Andrzej Hajda, Hans Verkuil, Jonathan Corbet, Sakari Ailus, Smitha T Murthy, Ricardo Ribalda Delgado, Tomasz Figa, Andy Shevchenko, linux-media, linux-kernel, linux-mediatek, linux-arm-msm, s.nawrocki Hi Keiichi, On 06/14/2018 10:46 AM, Keiichi Watanabe wrote: > Add a new control V4L2_CID_MPEG_VIDEO_VP9_PROFILE for VP9 > profiles. This control allows to select a desired profile for VP9 > encoder and query for supported profiles by VP9 encoder/decoder. > > Though this control is similar to V4L2_CID_MPEG_VIDEO_VP8_PROFILE, we need to > separate this control from it because supported profiles usually differ between > VP8 and VP9. > > Signed-off-by: Keiichi Watanabe <keiichiw@chromium.org> > --- > .../media/uapi/v4l/extended-controls.rst | 25 +++++++++++++++++++ > drivers/media/v4l2-core/v4l2-ctrls.c | 11 ++++++++ > include/uapi/linux/v4l2-controls.h | 7 ++++++ > 3 files changed, 43 insertions(+) > > diff --git a/Documentation/media/uapi/v4l/extended-controls.rst b/Documentation/media/uapi/v4l/extended-controls.rst > index de99eafb0872..095b42e9d6fe 100644 > --- a/Documentation/media/uapi/v4l/extended-controls.rst > +++ b/Documentation/media/uapi/v4l/extended-controls.rst > @@ -1980,6 +1980,31 @@ enum v4l2_mpeg_video_vp8_profile - > * - ``V4L2_MPEG_VIDEO_VP8_PROFILE_3`` > - Profile 3 > > +.. _v4l2-mpeg-video-vp9-profile: > + > +``V4L2_CID_MPEG_VIDEO_VP9_PROFILE`` > + (enum) > + > +enum v4l2_mpeg_video_vp9_profile - what about vp9 levels, shouldn't we add them too? Or we will add it when there is a user. -- regards, Stan ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH v3 2/3] media: v4l2-ctrl: Add control for VP9 profile 2018-06-14 8:06 ` Stanimir Varbanov @ 2018-06-14 9:12 ` Keiichi Watanabe 0 siblings, 0 replies; 9+ messages in thread From: Keiichi Watanabe @ 2018-06-14 9:12 UTC (permalink / raw) To: Stanimir Varbanov Cc: list@263.net:IOMMU DRIVERS <iommu@lists.linux-foundation.org>, Joerg Roedel <joro@8bytes.org>,, Mauro Carvalho Chehab, Tiffany Lin, Andrew-CT Chen, Matthias Brugger, Kyungmin Park, Kamil Debski, Jeongtae Park, Andrzej Hajda, Hans Verkuil, Jonathan Corbet, Sakari Ailus, Smitha T Murthy, Ricardo Ribalda Delgado, Tomasz Figa, Andy Shevchenko, Linux Media Mailing List, Linux Kernel Mailing List, linux-mediatek, linux-arm-msm, Sylwester Nawrocki Hi Stanimir, On Thu, Jun 14, 2018 at 5:06 PM, Stanimir Varbanov <stanimir.varbanov@linaro.org> wrote: > Hi Keiichi, > > On 06/14/2018 10:46 AM, Keiichi Watanabe wrote: >> Add a new control V4L2_CID_MPEG_VIDEO_VP9_PROFILE for VP9 >> profiles. This control allows to select a desired profile for VP9 >> encoder and query for supported profiles by VP9 encoder/decoder. >> >> Though this control is similar to V4L2_CID_MPEG_VIDEO_VP8_PROFILE, we need to >> separate this control from it because supported profiles usually differ between >> VP8 and VP9. >> >> Signed-off-by: Keiichi Watanabe <keiichiw@chromium.org> >> --- >> .../media/uapi/v4l/extended-controls.rst | 25 +++++++++++++++++++ >> drivers/media/v4l2-core/v4l2-ctrls.c | 11 ++++++++ >> include/uapi/linux/v4l2-controls.h | 7 ++++++ >> 3 files changed, 43 insertions(+) >> >> diff --git a/Documentation/media/uapi/v4l/extended-controls.rst b/Documentation/media/uapi/v4l/extended-controls.rst >> index de99eafb0872..095b42e9d6fe 100644 >> --- a/Documentation/media/uapi/v4l/extended-controls.rst >> +++ b/Documentation/media/uapi/v4l/extended-controls.rst >> @@ -1980,6 +1980,31 @@ enum v4l2_mpeg_video_vp8_profile - >> * - ``V4L2_MPEG_VIDEO_VP8_PROFILE_3`` >> - Profile 3 >> >> +.. _v4l2-mpeg-video-vp9-profile: >> + >> +``V4L2_CID_MPEG_VIDEO_VP9_PROFILE`` >> + (enum) >> + >> +enum v4l2_mpeg_video_vp9_profile - > > what about vp9 levels, shouldn't we add them too? Or we will add it when > there is a user. > It sounds good, but I don't think we should include them in this series of patches now. As you said, we can add them when someone wants to use them. Best regards, Keiichi > > -- > regards, > Stan ^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH v3 3/3] media: mtk-vcodec: Support VP9 profile in decoder 2018-06-14 7:46 [PATCH v3 0/3] Add controls for VP8/VP9 profile Keiichi Watanabe 2018-06-14 7:46 ` [PATCH v3 1/3] media: v4l2-ctrl: Change control for VP8 profile to menu control Keiichi Watanabe 2018-06-14 7:46 ` [PATCH v3 2/3] media: v4l2-ctrl: Add control for VP9 profile Keiichi Watanabe @ 2018-06-14 7:46 ` Keiichi Watanabe 2 siblings, 0 replies; 9+ messages in thread From: Keiichi Watanabe @ 2018-06-14 7:46 UTC (permalink / raw) To: linux-arm-kernel Cc: Mauro Carvalho Chehab, Tiffany Lin, Andrew-CT Chen, Matthias Brugger, Stanimir Varbanov, Kyungmin Park, Kamil Debski, Jeongtae Park, Andrzej Hajda, Hans Verkuil, Keiichi Watanabe, Jonathan Corbet, Sakari Ailus, Smitha T Murthy, Ricardo Ribalda Delgado, Tomasz Figa, Andy Shevchenko, linux-media, linux-kernel, linux-mediatek, linux-arm-msm, s.nawrocki Add V4L2_CID_MPEG_VIDEO_VP9_PROFILE control in MediaTek decoder's driver. MediaTek decoder only supports profile 0 for now. Signed-off-by: Keiichi Watanabe <keiichiw@chromium.org> --- drivers/media/platform/mtk-vcodec/mtk_vcodec_dec.c | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/drivers/media/platform/mtk-vcodec/mtk_vcodec_dec.c b/drivers/media/platform/mtk-vcodec/mtk_vcodec_dec.c index 86f0a7134365..ba986232b953 100644 --- a/drivers/media/platform/mtk-vcodec/mtk_vcodec_dec.c +++ b/drivers/media/platform/mtk-vcodec/mtk_vcodec_dec.c @@ -1400,6 +1400,11 @@ int mtk_vcodec_dec_ctrls_setup(struct mtk_vcodec_ctx *ctx) V4L2_CID_MIN_BUFFERS_FOR_CAPTURE, 0, 32, 1, 1); ctrl->flags |= V4L2_CTRL_FLAG_VOLATILE; + v4l2_ctrl_new_std_menu(&ctx->ctrl_hdl, + &mtk_vcodec_dec_ctrl_ops, + V4L2_CID_MPEG_VIDEO_VP9_PROFILE, + V4L2_MPEG_VIDEO_VP9_PROFILE_0, + 0, V4L2_MPEG_VIDEO_VP9_PROFILE_0); if (ctx->ctrl_hdl.error) { mtk_v4l2_err("Adding control failed %d", -- 2.18.0.rc1.242.g61856ae69a-goog ^ permalink raw reply related [flat|nested] 9+ messages in thread
end of thread, other threads:[~2018-06-15 8:51 UTC | newest] Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2018-06-14 7:46 [PATCH v3 0/3] Add controls for VP8/VP9 profile Keiichi Watanabe 2018-06-14 7:46 ` [PATCH v3 1/3] media: v4l2-ctrl: Change control for VP8 profile to menu control Keiichi Watanabe 2018-06-14 8:04 ` Stanimir Varbanov 2018-06-14 8:13 ` Keiichi Watanabe 2018-06-15 8:51 ` Hans Verkuil 2018-06-14 7:46 ` [PATCH v3 2/3] media: v4l2-ctrl: Add control for VP9 profile Keiichi Watanabe 2018-06-14 8:06 ` Stanimir Varbanov 2018-06-14 9:12 ` Keiichi Watanabe 2018-06-14 7:46 ` [PATCH v3 3/3] media: mtk-vcodec: Support VP9 profile in decoder Keiichi Watanabe
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).