From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from galahad.ideasonboard.com ([185.26.127.97]:33462 "EHLO galahad.ideasonboard.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751854AbaHDKjl (ORCPT ); Mon, 4 Aug 2014 06:39:41 -0400 From: Laurent Pinchart To: Hans Verkuil Cc: linux-media@vger.kernel.org, Hans Verkuil Subject: Re: [PATCH for v3.17 1/2] videobuf2-core.h: fix comment Date: Mon, 04 Aug 2014 12:40:08 +0200 Message-ID: <1677917.ZvIcNnnNOx@avalon> In-Reply-To: <1407148032-41607-2-git-send-email-hverkuil@xs4all.nl> References: <1407148032-41607-1-git-send-email-hverkuil@xs4all.nl> <1407148032-41607-2-git-send-email-hverkuil@xs4all.nl> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" Sender: linux-media-owner@vger.kernel.org List-ID: Hi Hans, Thank you for the patch. On Monday 04 August 2014 12:27:11 Hans Verkuil wrote: > From: Hans Verkuil > > The comment for start_streaming that tells the developer with which vb2 > state buffers should be returned to vb2 gave the wrong state. Very > confusing. > > Signed-off-by: Hans Verkuil Acked-by: Laurent Pinchart I wonder whether we couldn't simplify drivers by moving this into vb2 though. A failed start_streaming requires drivers to dequeue all buffers internally, but the call to vb2_buffer_done() could be handled inside vb2. On the other hand it would make the vb2 warning go away, and drivers that fail to dequeue buffers internally would not be caught as easily, so I won't push for that change. > --- > include/media/videobuf2-core.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/include/media/videobuf2-core.h b/include/media/videobuf2-core.h > index fc910a6..80fa725 100644 > --- a/include/media/videobuf2-core.h > +++ b/include/media/videobuf2-core.h > @@ -295,7 +295,7 @@ struct vb2_buffer { > * can return an error if hardware fails, in that case all > * buffers that have been already given by the @buf_queue > * callback are to be returned by the driver by calling > - * @vb2_buffer_done(VB2_BUF_STATE_DEQUEUED). > + * @vb2_buffer_done(VB2_BUF_STATE_QUEUED). > * If you need a minimum number of buffers before you can > * start streaming, then set @min_buffers_needed in the > * vb2_queue structure. If that is non-zero then -- Regards, Laurent Pinchart