Re: [REVIEWv2 PATCH 10/13] vb2: set v4l2_buffer.bytesused to 0 for mp buffers

2014-04-09 Thread Pawel Osciak
Ah, alas, Sakari is right. This should not be needed, since we memcpy vb->v4l2_buf to this, also overwriting bytesused. On Thu, Apr 10, 2014 at 10:08 AM, Pawel Osciak wrote: > On Mon, Apr 7, 2014 at 10:11 PM, Hans Verkuil wrote: >> From: Hans Verkuil >> >> The bytesused field of struct v4l2_buf

Re: [REVIEWv2 PATCH 10/13] vb2: set v4l2_buffer.bytesused to 0 for mp buffers

2014-04-09 Thread Pawel Osciak
On Mon, Apr 7, 2014 at 10:11 PM, Hans Verkuil wrote: > From: Hans Verkuil > > The bytesused field of struct v4l2_buffer is not used for multiplanar > formats, so just zero it to prevent it from having some random value. > > Signed-off-by: Hans Verkuil Acked-by: Pawel Osciak > --- > drivers/m

[REVIEWv2 PATCH 10/13] vb2: set v4l2_buffer.bytesused to 0 for mp buffers

2014-04-07 Thread Hans Verkuil
From: Hans Verkuil The bytesused field of struct v4l2_buffer is not used for multiplanar formats, so just zero it to prevent it from having some random value. Signed-off-by: Hans Verkuil --- drivers/media/v4l2-core/videobuf2-core.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/me