diff mbox series

v4l2-ctrls: Fix h264 hierarchical coding type menu ctrl

Message ID 20210302224206.3680649-1-stanimir.varbanov@linaro.org
State Accepted
Commit a9e3863d0006ddad14e6b5745cb1a04e9801885f
Headers show
Series v4l2-ctrls: Fix h264 hierarchical coding type menu ctrl | expand

Commit Message

Stanimir Varbanov March 2, 2021, 10:42 p.m. UTC
Add a name of the menu and fill control type.

Signed-off-by: Stanimir Varbanov <stanimir.varbanov@linaro.org>
---
 drivers/media/v4l2-core/v4l2-ctrls.c | 8 ++++++++
 1 file changed, 8 insertions(+)

Comments

Stanimir Varbanov March 20, 2021, 11:53 a.m. UTC | #1
Kindly ping for review.

On 3/3/21 12:42 AM, Stanimir Varbanov wrote:
> Add a name of the menu and fill control type.

> 

> Signed-off-by: Stanimir Varbanov <stanimir.varbanov@linaro.org>

> ---

>  drivers/media/v4l2-core/v4l2-ctrls.c | 8 ++++++++

>  1 file changed, 8 insertions(+)

> 

> diff --git a/drivers/media/v4l2-core/v4l2-ctrls.c b/drivers/media/v4l2-core/v4l2-ctrls.c

> index 016cf6204cbb..5d99e2294335 100644

> --- a/drivers/media/v4l2-core/v4l2-ctrls.c

> +++ b/drivers/media/v4l2-core/v4l2-ctrls.c

> @@ -421,6 +421,11 @@ const char * const *v4l2_ctrl_get_menu(u32 id)

>  		"Annex B Start Code",

>  		NULL,

>  	};

> +	static const char * const h264_hierarchical_coding_type[] = {

> +		"Hier Coding B",

> +		"Hier Coding P",

> +		NULL,

> +	};

>  	static const char * const mpeg_mpeg2_level[] = {

>  		"Low",

>  		"Main",

> @@ -697,6 +702,8 @@ const char * const *v4l2_ctrl_get_menu(u32 id)

>  		return h264_decode_mode;

>  	case V4L2_CID_STATELESS_H264_START_CODE:

>  		return h264_start_code;

> +	case V4L2_CID_MPEG_VIDEO_H264_HIERARCHICAL_CODING_TYPE:

> +		return h264_hierarchical_coding_type;

>  	case V4L2_CID_MPEG_VIDEO_MPEG2_LEVEL:

>  		return mpeg_mpeg2_level;

>  	case V4L2_CID_MPEG_VIDEO_MPEG2_PROFILE:

> @@ -1326,6 +1333,7 @@ void v4l2_ctrl_fill(u32 id, const char **name, enum v4l2_ctrl_type *type,

>  	case V4L2_CID_MPEG_VIDEO_H264_VUI_SAR_IDC:

>  	case V4L2_CID_MPEG_VIDEO_H264_SEI_FP_ARRANGEMENT_TYPE:

>  	case V4L2_CID_MPEG_VIDEO_H264_FMO_MAP_TYPE:

> +	case V4L2_CID_MPEG_VIDEO_H264_HIERARCHICAL_CODING_TYPE:

>  	case V4L2_CID_MPEG_VIDEO_MPEG2_LEVEL:

>  	case V4L2_CID_MPEG_VIDEO_MPEG2_PROFILE:

>  	case V4L2_CID_MPEG_VIDEO_MPEG4_LEVEL:

> 


-- 
regards,
Stan
Hans Verkuil March 20, 2021, 12:11 p.m. UTC | #2
On 20/03/2021 12:53, Stanimir Varbanov wrote:
> Kindly ping for review.


Weird, it was marked as 'Under Review' in patchwork, but it wasn't delegated
to anyone.

I've changed it to 'New' and delegated it to myself.

It looks good and I'll pick it up next time I collect 'various fixes' in a PR.
Probably next week.

Regards,

	Hans

> 

> On 3/3/21 12:42 AM, Stanimir Varbanov wrote:

>> Add a name of the menu and fill control type.

>>

>> Signed-off-by: Stanimir Varbanov <stanimir.varbanov@linaro.org>

>> ---

>>  drivers/media/v4l2-core/v4l2-ctrls.c | 8 ++++++++

>>  1 file changed, 8 insertions(+)

>>

>> diff --git a/drivers/media/v4l2-core/v4l2-ctrls.c b/drivers/media/v4l2-core/v4l2-ctrls.c

>> index 016cf6204cbb..5d99e2294335 100644

>> --- a/drivers/media/v4l2-core/v4l2-ctrls.c

>> +++ b/drivers/media/v4l2-core/v4l2-ctrls.c

>> @@ -421,6 +421,11 @@ const char * const *v4l2_ctrl_get_menu(u32 id)

>>  		"Annex B Start Code",

>>  		NULL,

>>  	};

>> +	static const char * const h264_hierarchical_coding_type[] = {

>> +		"Hier Coding B",

>> +		"Hier Coding P",

>> +		NULL,

>> +	};

>>  	static const char * const mpeg_mpeg2_level[] = {

>>  		"Low",

>>  		"Main",

>> @@ -697,6 +702,8 @@ const char * const *v4l2_ctrl_get_menu(u32 id)

>>  		return h264_decode_mode;

>>  	case V4L2_CID_STATELESS_H264_START_CODE:

>>  		return h264_start_code;

>> +	case V4L2_CID_MPEG_VIDEO_H264_HIERARCHICAL_CODING_TYPE:

>> +		return h264_hierarchical_coding_type;

>>  	case V4L2_CID_MPEG_VIDEO_MPEG2_LEVEL:

>>  		return mpeg_mpeg2_level;

>>  	case V4L2_CID_MPEG_VIDEO_MPEG2_PROFILE:

>> @@ -1326,6 +1333,7 @@ void v4l2_ctrl_fill(u32 id, const char **name, enum v4l2_ctrl_type *type,

>>  	case V4L2_CID_MPEG_VIDEO_H264_VUI_SAR_IDC:

>>  	case V4L2_CID_MPEG_VIDEO_H264_SEI_FP_ARRANGEMENT_TYPE:

>>  	case V4L2_CID_MPEG_VIDEO_H264_FMO_MAP_TYPE:

>> +	case V4L2_CID_MPEG_VIDEO_H264_HIERARCHICAL_CODING_TYPE:

>>  	case V4L2_CID_MPEG_VIDEO_MPEG2_LEVEL:

>>  	case V4L2_CID_MPEG_VIDEO_MPEG2_PROFILE:

>>  	case V4L2_CID_MPEG_VIDEO_MPEG4_LEVEL:

>>

>
diff mbox series

Patch

diff --git a/drivers/media/v4l2-core/v4l2-ctrls.c b/drivers/media/v4l2-core/v4l2-ctrls.c
index 016cf6204cbb..5d99e2294335 100644
--- a/drivers/media/v4l2-core/v4l2-ctrls.c
+++ b/drivers/media/v4l2-core/v4l2-ctrls.c
@@ -421,6 +421,11 @@  const char * const *v4l2_ctrl_get_menu(u32 id)
 		"Annex B Start Code",
 		NULL,
 	};
+	static const char * const h264_hierarchical_coding_type[] = {
+		"Hier Coding B",
+		"Hier Coding P",
+		NULL,
+	};
 	static const char * const mpeg_mpeg2_level[] = {
 		"Low",
 		"Main",
@@ -697,6 +702,8 @@  const char * const *v4l2_ctrl_get_menu(u32 id)
 		return h264_decode_mode;
 	case V4L2_CID_STATELESS_H264_START_CODE:
 		return h264_start_code;
+	case V4L2_CID_MPEG_VIDEO_H264_HIERARCHICAL_CODING_TYPE:
+		return h264_hierarchical_coding_type;
 	case V4L2_CID_MPEG_VIDEO_MPEG2_LEVEL:
 		return mpeg_mpeg2_level;
 	case V4L2_CID_MPEG_VIDEO_MPEG2_PROFILE:
@@ -1326,6 +1333,7 @@  void v4l2_ctrl_fill(u32 id, const char **name, enum v4l2_ctrl_type *type,
 	case V4L2_CID_MPEG_VIDEO_H264_VUI_SAR_IDC:
 	case V4L2_CID_MPEG_VIDEO_H264_SEI_FP_ARRANGEMENT_TYPE:
 	case V4L2_CID_MPEG_VIDEO_H264_FMO_MAP_TYPE:
+	case V4L2_CID_MPEG_VIDEO_H264_HIERARCHICAL_CODING_TYPE:
 	case V4L2_CID_MPEG_VIDEO_MPEG2_LEVEL:
 	case V4L2_CID_MPEG_VIDEO_MPEG2_PROFILE:
 	case V4L2_CID_MPEG_VIDEO_MPEG4_LEVEL: