Hi Tomasz and others, On Fri, Jan 12, 2018 at 05:19:04PM +0900, Tomasz Figa wrote: > On Thu, Jan 4, 2018 at 11:57 AM, Yong Zhi <yong.zhi@xxxxxxxxx> wrote: ... > > @@ -793,7 +794,7 @@ static void cio2_vb2_return_all_buffers(struct cio2_queue *q) > > if (q->bufs[i]) { > > atomic_dec(&q->bufs_queued); > > vb2_buffer_done(&q->bufs[i]->vbb.vb2_buf, > > - VB2_BUF_STATE_ERROR); > > + state); > > nit: Does it really exceed 80 characters after folding into previous line? > > With the nit fixed: > Reviewed-by: Tomasz Figa <tfiga@xxxxxxxxxxxx> The patches have been merged to media tree master; if there are matters to address, then please send more patches on top of the master branch. :-) Thanks. -- Cheers, Sakari Ailus sakari.ailus@xxxxxxxxxxxxxxx