Re: [RFC PATCH 04/13] soc_camera: remove use of current_norm.

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

 



On Fri, 7 Jun 2013, Hans Verkuil wrote:

> On Fri June 7 2013 11:36:57 Guennadi Liakhovetski wrote:
> > Hi Hans
> > 
> > On Mon, 3 Jun 2013, Hans Verkuil wrote:
> > 
> > > From: Hans Verkuil <hans.verkuil@xxxxxxxxx>
> > > 
> > > The current_norm field is deprecated, so don't set it. Since it is set to
> > > V4L2_STD_UNKNOWN which is 0 it didn't do anything anyway.
> > > 
> > > Also remove a few other unnecessary uses of V4L2_STD_UNKNOWN.
> > > 
> > > Signed-off-by: Hans Verkuil <hans.verkuil@xxxxxxxxx>
> > > Cc: Guennadi Liakhovetski <g.liakhovetski@xxxxxx>
> > 
> > I'd rather take this via my tree to avoid any conflicts, is this ok with 
> > you?
> 
> Ideally I would like to take this in myself since the last patch of the series
> removes current_norm support entirely. If you take it, then I have to leave off
> the last patch until both our trees are merged.
> 
> If you still prefer to take it yourself, then let me know and I'll leave this
> patch and the last patch out when I merge it.

Ok, take it via your tree, we'll resolve any conflicts afterwards.

Acked-by: Guennadi Liakhovetski <g.liakhovetski@xxxxxx>

Thanks
Guennadi

> 
> Regards,
> 
> 	Hans
> 
> > 
> > Thanks
> > Guennadi
> > 
> > > ---
> > >  drivers/media/platform/soc_camera/soc_camera.c |    3 ---
> > >  1 file changed, 3 deletions(-)
> > > 
> > > diff --git a/drivers/media/platform/soc_camera/soc_camera.c b/drivers/media/platform/soc_camera/soc_camera.c
> > > index eea832c..96645e9 100644
> > > --- a/drivers/media/platform/soc_camera/soc_camera.c
> > > +++ b/drivers/media/platform/soc_camera/soc_camera.c
> > > @@ -235,7 +235,6 @@ static int soc_camera_enum_input(struct file *file, void *priv,
> > >  
> > >  	/* default is camera */
> > >  	inp->type = V4L2_INPUT_TYPE_CAMERA;
> > > -	inp->std  = V4L2_STD_UNKNOWN;
> > >  	strcpy(inp->name, "Camera");
> > >  
> > >  	return 0;
> > > @@ -1513,11 +1512,9 @@ static int video_dev_create(struct soc_camera_device *icd)
> > >  	strlcpy(vdev->name, ici->drv_name, sizeof(vdev->name));
> > >  
> > >  	vdev->parent		= icd->pdev;
> > > -	vdev->current_norm	= V4L2_STD_UNKNOWN;
> > >  	vdev->fops		= &soc_camera_fops;
> > >  	vdev->ioctl_ops		= &soc_camera_ioctl_ops;
> > >  	vdev->release		= video_device_release;
> > > -	vdev->tvnorms		= V4L2_STD_UNKNOWN;
> > >  	vdev->ctrl_handler	= &icd->ctrl_handler;
> > >  	vdev->lock		= &ici->host_lock;
> > >  
> > 
> > ---
> > Guennadi Liakhovetski, Ph.D.
> > Freelance Open-Source Software Developer
> > http://www.open-technology.de/
> > 
> 

---
Guennadi Liakhovetski, Ph.D.
Freelance Open-Source Software Developer
http://www.open-technology.de/
--
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