On Thu, 29 Sep 2011, Sakari Ailus wrote: > Guennadi Liakhovetski wrote: > > On Wed, 28 Sep 2011, Sakari Ailus wrote: > > > > > Hi Guennadi, > > > > > > On Wed, Sep 28, 2011 at 04:56:11PM +0200, Guennadi Liakhovetski wrote: > > > > @@ -2099,6 +2103,15 @@ struct v4l2_dbg_chip_ident { > > > > __u32 revision; /* chip revision, chip specific */ > > > > } __attribute__ ((packed)); > > > > > > > > +/* VIDIOC_CREATE_BUFS */ > > > > +struct v4l2_create_buffers { > > > > + __u32 index; /* output: buffers > > > > index...index + count - 1 have been created */ > > > > + __u32 count; > > > > + enum v4l2_memory memory; > > > > + struct v4l2_format format; /* "type" is used always, the > > > > rest if sizeimage == 0 */ > > > > + __u32 reserved[8]; > > > > +}; > > > > > > What about the kerneldoc comments you wrote right after v6 on 1st > > > September > > > for v4l2_create_buffers and the same for the compat32 version? > > > > Looks like someone is trying very hard to cause me a heart failure;-) They > > are in a separate patch: > > No, I don't! :-) I'm just interested all the appropriate changes are in the > patchset. My understanding was you intended to add these changes to the > original patches. > > Are you planning to put them to the same patchset still, or a different one? The same. Thanks Guennadi --- 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