On 11/14/2014 04:35 PM, Sakari Ailus wrote:
> Hi Hans,
>
> One more comment...
>
> On Sun, Sep 21, 2014 at 04:48:21PM +0200, Hans Verkuil wrote:
>> diff --git a/include/uapi/linux/videodev2.h b/include/uapi/linux/videodev2.h
>> index 83ef28a..2ca44ed 100644
>> --- a/include/uapi/linux/videodev2.h
>> +++ b/include/uapi/linux/videodev2.h
>> @@ -672,6 +672,7 @@ struct v4l2_plane {
>> * @length: size in bytes of the buffer (NOT its payload) for single-plane
>> * buffers (when type != *_MPLANE); number of elements in the
>> * planes array for multi-plane buffers
>> + * @config_store: this buffer should use this configuration store
>> *
>> * Contains data exchanged by application and driver using one of the Streaming
>> * I/O methods.
>> @@ -695,7 +696,7 @@ struct v4l2_buffer {
>> __s32 fd;
>> } m;
>> __u32 length;
>> - __u32 reserved2;
>> + __u32 config_store;
>> __u32 reserved;
>> };
>>
>
> I would use __u16 instead since the value is 16-bit on the control
> interface.
>
Good point. Will do.
Hans
--
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
