linux-kernel - Re: [PATCH] v4l2-ctrls: Fix h264 hierarchical coding type menu ctrl

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <b2e3999a-0854-50e7-b3c6-a2048666ff08@xs4all.nl>
Date:   Sat, 20 Mar 2021 13:11:23 +0100
From:   Hans Verkuil <hverkuil-cisco@...all.nl>
To:     Stanimir Varbanov <stanimir.varbanov@...aro.org>,
        linux-media@...r.kernel.org, linux-kernel@...r.kernel.org
Cc:     Mauro Carvalho Chehab <mchehab@...nel.org>,
        Ezequiel Garcia <ezequiel@...labora.com>
Subject: Re: [PATCH] v4l2-ctrls: Fix h264 hierarchical coding type menu ctrl

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@...aro.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:
>>
>