Re: [PATCH v5 2/4] vb2: add allow_zero_bytesused flag to the vb2_queue struct

2015-02-23 Thread Hans Verkuil
On 02/23/2015 12:58 PM, Kamil Debski wrote: > Hi, > > Thank you for the review :) > >> From: Hans Verkuil [mailto:hverk...@xs4all.nl] >> Sent: Friday, February 20, 2015 5:52 PM >> >> Hi Kamil, >> >> One question and one typo below... >> >> On 02/20/2015 05:38 PM, Kamil Debski wrote: >>> The vb2:

RE: [PATCH v5 2/4] vb2: add allow_zero_bytesused flag to the vb2_queue struct

2015-02-23 Thread Kamil Debski
Hi, Thank you for the review :) > From: Hans Verkuil [mailto:hverk...@xs4all.nl] > Sent: Friday, February 20, 2015 5:52 PM > > Hi Kamil, > > One question and one typo below... > > On 02/20/2015 05:38 PM, Kamil Debski wrote: > > The vb2: fix bytesused == 0 handling (8a75ffb) patch changed the >

Re: [PATCH v5 2/4] vb2: add allow_zero_bytesused flag to the vb2_queue struct

2015-02-20 Thread Hans Verkuil
Hi Kamil, One question and one typo below... On 02/20/2015 05:38 PM, Kamil Debski wrote: > The vb2: fix bytesused == 0 handling (8a75ffb) patch changed the behavior > of __fill_vb2_buffer function, so that if bytesused is 0 it is set to the > size of the buffer. However, bytesused set to 0 is use