On Thu, Dec 28, 2017 at 02:29:35PM -0200, Mauro Carvalho Chehab wrote: > From: Satendra Singh Thakur <satendr...@samsung.com> > > Currently, there's a logic with checks if *count is non-zero, > q->num_buffers is zero and q->memory is different than memory. > > That's flawed when the device is initialized, or after the > queues are freed, as it does, unnecessary calls to > __vb2_queue_cancel() and __vb2_queue_free(). > > That can be avoided by making sure that q->memory is set to > VB2_MEMORY_UNKNOWN at vb2_core_queue_init(), and adding such > check at the loop. > > [mche...@s-opensource.com: fix checkpatch issues and improve the > patch, by setting q->memory to zero at vb2_core_queue_init] > Signed-off-by: Satendra Singh Thakur <satendr...@samsung.com> > Signed-off-by: Mauro Carvalho Chehab <mche...@s-opensource.com>
Acked-by: Sakari Ailus <sakari.ai...@linux.intel.com> -- Sakari Ailus sakari.ai...@linux.intel.com
