Re: [PATCH 2/6] sta2x11_vip: fix s_std

[Federico Vaga <federico.vaga@gmail.com>]

Acked-by: Federico Vaga <federico.vaga@gmail.com>

It sounds fine to me (even the ADV7180 patch). Unfortunately I do not have the 
hardware to test it.

On Friday, April 22, 2016 03:03:38 PM Hans Verkuil wrote:
> From: Hans Verkuil <hans.verkuil@cisco.com>
> 
> The s_std ioctl was broken in this driver, partially due to the
> changes to the adv7180 driver (this affected the handling of
> V4L2_STD_ALL) and partially because the new standard was never
> stored in vip->std.
> 
> The handling of V4L2_STD_ALL has been rewritten to just call querystd
> and the new standard is now stored correctly.
> 
> Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
> Cc: Federico Vaga <federico.vaga@gmail.com>
> ---
>  drivers/media/pci/sta2x11/sta2x11_vip.c | 26 ++++++++++----------------
>  1 file changed, 10 insertions(+), 16 deletions(-)
> 
> diff --git a/drivers/media/pci/sta2x11/sta2x11_vip.c
> b/drivers/media/pci/sta2x11/sta2x11_vip.c index 753411c..c79623c 100644
> --- a/drivers/media/pci/sta2x11/sta2x11_vip.c
> +++ b/drivers/media/pci/sta2x11/sta2x11_vip.c
> @@ -444,27 +444,21 @@ static int vidioc_querycap(struct file *file, void
> *priv, static int vidioc_s_std(struct file *file, void *priv, v4l2_std_id
> std) {
>  	struct sta2x11_vip *vip = video_drvdata(file);
> -	v4l2_std_id oldstd = vip->std, newstd;
> +	v4l2_std_id oldstd = vip->std;
>  	int status;
> 
> -	if (V4L2_STD_ALL == std) {
> -		v4l2_subdev_call(vip->decoder, video, s_std, std);
> -		ssleep(2);
> -		v4l2_subdev_call(vip->decoder, video, querystd, &newstd);
> -		v4l2_subdev_call(vip->decoder, video, g_input_status, &status);
> -		if (status & V4L2_IN_ST_NO_SIGNAL)
> +	/*
> +	 * This is here for backwards compatibility only.
> +	 * The use of V4L2_STD_ALL to trigger a querystd is non-standard.
> +	 */
> +	if (std == V4L2_STD_ALL) {
> +		v4l2_subdev_call(vip->decoder, video, querystd, &std);
> +		if (std == V4L2_STD_UNKNOWN)
>  			return -EIO;
> -		std = vip->std = newstd;
> -		if (oldstd != std) {
> -			if (V4L2_STD_525_60 & std)
> -				vip->format = formats_60[0];
> -			else
> -				vip->format = formats_50[0];
> -		}
> -		return 0;
>  	}
> 
> -	if (oldstd != std) {
> +	if (vip->std != std) {
> +		vip->std = std;
>  		if (V4L2_STD_525_60 & std)
>  			vip->format = formats_60[0];
>  		else

-- 
Federico Vaga