Re: [PATCH 3/3] [media] adv7180: Add g_tvnorms operation

[Hans Verkuil <hverkuil@xxxxxxxxx>]

On 04/02/2016 10:42 AM, Niklas Söderlund wrote:
> The ADV7180 supports NTSC, PAL and SECAM.
>
> Signed-off-by: Niklas Söderlund <niklas.soderlund+renesas@xxxxxxxxxxxx>

Acked-by: Hans Verkuil <hans.verkuil@xxxxxxxxx>

Thanks!

	Hans


> ---
>   drivers/media/i2c/adv7180.c | 8 +++++++-
>   1 file changed, 7 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/media/i2c/adv7180.c b/drivers/media/i2c/adv7180.c
> index 80ded70..51a92b3 100644
> --- a/drivers/media/i2c/adv7180.c
> +++ b/drivers/media/i2c/adv7180.c
> @@ -741,6 +741,12 @@ static int adv7180_cropcap(struct v4l2_subdev *sd, struct v4l2_cropcap *cropcap)
>   	return 0;
>   }
>
> +static int adv7180_g_tvnorms(struct v4l2_subdev *sd, v4l2_std_id *norm)
> +{
> +	*norm = V4L2_STD_ALL;
> +	return 0;
> +}
> +
>   static const struct v4l2_subdev_video_ops adv7180_video_ops = {
>   	.s_std = adv7180_s_std,
>   	.g_std = adv7180_g_std,
> @@ -749,9 +755,9 @@ static const struct v4l2_subdev_video_ops adv7180_video_ops = {
>   	.s_routing = adv7180_s_routing,
>   	.g_mbus_config = adv7180_g_mbus_config,
>   	.cropcap = adv7180_cropcap,
> +	.g_tvnorms = adv7180_g_tvnorms,
>   };
>
> -
>   static const struct v4l2_subdev_core_ops adv7180_core_ops = {
>   	.s_power = adv7180_s_power,
>   };