RE: [PATCH] v4l/s5p-mfc: corrected encoder v4l control definitions

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Andrzej, 

Thank you for the patch.

Best wishes,
--
Kamil Debski
Linux Platform Group
Samsung Poland R&D Center

> -----Original Message-----
> From: Andrzej Hajda [mailto:a.hajda@xxxxxxxxxxx]
> Sent: 14 June 2012 09:04
> To: linux-media@xxxxxxxxxxxxxxx
> Cc: Hans Verkuil; Marek Szyprowski; Kamil Debski; Andrzej Hajda; Kyungmin
> Park
> Subject: [PATCH] v4l/s5p-mfc: corrected encoder v4l control definitions
> 
> Patch corrects definition of H264 level control and
> changes bare numbers to enums in two other cases.
> 
> Signed-off-by: Andrzej Hajda <a.hajda@xxxxxxxxxxx>
> Signed-off-by: Kyungmin Park <kyungmin.park@xxxxxxxxxxx>

Acked-by: Kamil Debski <k.debski@xxxxxxxxxxx>

> ---
>  drivers/media/video/s5p-mfc/s5p_mfc_enc.c |   10 ++--------
>  1 files changed, 2 insertions(+), 8 deletions(-)
> 
> diff --git a/drivers/media/video/s5p-mfc/s5p_mfc_enc.c
> b/drivers/media/video/s5p-mfc/s5p_mfc_enc.c
> index acedb20..9c19aa8 100644
> --- a/drivers/media/video/s5p-mfc/s5p_mfc_enc.c
> +++ b/drivers/media/video/s5p-mfc/s5p_mfc_enc.c
> @@ -243,12 +243,6 @@ static struct mfc_control controls[] = {
>  		.minimum = V4L2_MPEG_VIDEO_H264_LEVEL_1_0,
>  		.maximum = V4L2_MPEG_VIDEO_H264_LEVEL_4_0,
>  		.default_value = V4L2_MPEG_VIDEO_H264_LEVEL_1_0,
> -		.menu_skip_mask = ~(
> -				(1 << V4L2_MPEG_VIDEO_H264_LEVEL_4_1) |
> -				(1 << V4L2_MPEG_VIDEO_H264_LEVEL_4_2) |
> -				(1 << V4L2_MPEG_VIDEO_H264_LEVEL_5_0) |
> -				(1 << V4L2_MPEG_VIDEO_H264_LEVEL_5_1)
> -				),
>  	},
>  	{
>  		.id = V4L2_CID_MPEG_VIDEO_MPEG4_LEVEL,
> @@ -494,7 +488,7 @@ static struct mfc_control controls[] = {
>  		.type = V4L2_CTRL_TYPE_MENU,
>  		.minimum = V4L2_MPEG_VIDEO_H264_VUI_SAR_IDC_UNSPECIFIED,
>  		.maximum = V4L2_MPEG_VIDEO_H264_VUI_SAR_IDC_EXTENDED,
> -		.default_value = 0,
> +		.default_value = V4L2_MPEG_VIDEO_H264_VUI_SAR_IDC_UNSPECIFIED,
>  		.menu_skip_mask = 0,
>  	},
>  	{
> @@ -534,7 +528,7 @@ static struct mfc_control controls[] = {
>  		.type = V4L2_CTRL_TYPE_MENU,
>  		.minimum = V4L2_MPEG_VIDEO_MPEG4_PROFILE_SIMPLE,
>  		.maximum = V4L2_MPEG_VIDEO_MPEG4_PROFILE_ADVANCED_SIMPLE,
> -		.default_value = 0,
> +		.default_value = V4L2_MPEG_VIDEO_MPEG4_PROFILE_SIMPLE,
>  		.menu_skip_mask = 0,
>  	},
>  	{
> --
> 1.7.0.4

--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux