Hi Guennadi,

On Thursday 01 September 2011 09:03:52 Guennadi Liakhovetski wrote:
> On Thu, 1 Sep 2011, Sakari Ailus wrote:
> > On Wed, Aug 31, 2011 at 08:02:41PM +0200, Guennadi Liakhovetski wrote:

[snip]

> > > +
> > > 
> > >  /*
> > >  
> > >   *       I O C T L   C O D E S   F O R   V I D E O   D E V I C E S
> > >   *
> > > 
> > > @@ -2182,6 +2194,9 @@ struct v4l2_dbg_chip_ident {
> > > 
> > >  #define  VIDIOC_SUBSCRIBE_EVENT   _IOW('V', 90, struct
> > >  v4l2_event_subscription) #define VIDIOC_UNSUBSCRIBE_EVENT _IOW('V',
> > >  91, struct v4l2_event_subscription)
> > > 
> > > +#define VIDIOC_CREATE_BUFS       _IOWR('V', 92, struct 
> > > v4l2_create_buffers)
> > > +#define VIDIOC_PREPARE_BUF        _IOW('V', 93, struct v4l2_buffer)
> > 
> > Does prepare_buf ever do anything that would need to return anything to
> > the user? I guess the answer is "no"?
> 
> Exactly, that's why it's an "_IOW" ioctl(), not an "_IOWR", or have I
> misunderstood you?

This caught my eyes as well. Do you think VIDIOC_PREPARE_BUF could need to 
return information to userspace in the future ?

-- 
Regards,

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

Reply via email to