Just checking for number of buffers and memory access method isn't enough because the format might have changed since the buffers were allocated and the new format might need bigger ones. This reverts commit 31901a078af29c33c736dcbf815656920e904632. Signed-off-by: Uwe Kleine-KÃnig <u.kleine-koenig@xxxxxxxxxxxxxx> --- Hello, I resend, because the original post didn't made it into patchwork. (The web interface showed a database error around that time.) And please note that my name is still wrong in patchwork. Best regards Uwe drivers/media/video/videobuf2-core.c | 7 ------- 1 files changed, 0 insertions(+), 7 deletions(-) diff --git a/drivers/media/video/videobuf2-core.c b/drivers/media/video/videobuf2-core.c index 6ba1461..6489aa2 100644 --- a/drivers/media/video/videobuf2-core.c +++ b/drivers/media/video/videobuf2-core.c @@ -492,13 +492,6 @@ int vb2_reqbufs(struct vb2_queue *q, struct v4l2_requestbuffers *req) return -EINVAL; } - /* - * If the same number of buffers and memory access method is requested - * then return immediately. - */ - if (q->memory == req->memory && req->count == q->num_buffers) - return 0; - if (req->count == 0 || q->num_buffers != 0 || q->memory != req->memory) { /* * We already have buffers allocated, so first check if they -- 1.7.5.3 -- 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