From patchwork Fri Sep 1 01:50:35 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "Gustavo F. Padovan" X-Patchwork-Id: 43676 X-Patchwork-Delegate: hverkuil@xs4all.nl Received: from vger.kernel.org ([209.132.180.67]) by www.linuxtv.org with esmtp (Exim 4.84_2) (envelope-from ) id 1dnb76-0002eG-Gz; Fri, 01 Sep 2017 01:51:16 +0000 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751706AbdIABvO (ORCPT + 1 other); Thu, 31 Aug 2017 21:51:14 -0400 Received: from mail-qk0-f194.google.com ([209.85.220.194]:33149 "EHLO mail-qk0-f194.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751667AbdIABvO (ORCPT ); Thu, 31 Aug 2017 21:51:14 -0400 Received: by mail-qk0-f194.google.com with SMTP id k126so1008053qkb.0 for ; Thu, 31 Aug 2017 18:51:13 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references; bh=BbilZd8O2S6bm9VXjYBL8GJO3wyS8XlWzQtbw0R+8Ds=; b=QV9DBx9s+vbjHle8f9XZ2+y8iaNfDztPKWhK+WtghfTSpGB+QKWpOkXZGwOJ7YVE2D JUIuRFlSR0/1T6vLrd+emAUJFwvw6etRnmoPIxwQ5cCiNVwBfd0sz0HRYyDJo+Lv8kJn A5Ro/S97SF0v8Vsv6bvYK4hTkomdUSnfjaejxsvp39Oi4mpUiZ9qWrLKmdOs9P0KQ0Zu 4/672uIOwTGZp2f9r+t9Npl3QkcnNpqhVpDAClldzY2lzDlh86VmYJLzIivjnNPrbW5I 0lKGgzykmW7b4I2fr5+P7YLwENo2LWfoq5OCpNfIp7XzDBekjdpLzvMT1+QYDyJtdMi+ LW3w== X-Gm-Message-State: AHPjjUhy7esUNo1ODuAXB3U/3UpQRvZbfdv08Rb5Snn4EFnbqhdgvzOW xWaYFOoyTF/fFhZW7UA= X-Google-Smtp-Source: ADKCNb5u029B/XgYL5cA0fa0+ZKdL+0I//VCNsn4icEGDQpr3D7/WX8QIlgbYlPklJWE6rQEFxrNNQ== X-Received: by 10.55.212.4 with SMTP id l4mr553851qki.147.1504230673280; Thu, 31 Aug 2017 18:51:13 -0700 (PDT) Received: from localhost.localdomain ([187.10.21.246]) by smtp.gmail.com with ESMTPSA id o17sm6366232qkl.52.2017.08.31.18.51.10 (version=TLS1_2 cipher=ECDHE-RSA-CHACHA20-POLY1305 bits=256/256); Thu, 31 Aug 2017 18:51:12 -0700 (PDT) From: Gustavo Padovan To: linux-media@vger.kernel.org Cc: Hans Verkuil , Mauro Carvalho Chehab , Shuah Khan , Gustavo Padovan Subject: [PATCH v2 08/14] [media] v4l: add support to BUF_QUEUED event Date: Thu, 31 Aug 2017 22:50:35 -0300 Message-Id: <20170901015041.7757-9-gustavo@padovan.org> X-Mailer: git-send-email 2.13.5 In-Reply-To: <20170901015041.7757-1-gustavo@padovan.org> References: <20170901015041.7757-1-gustavo@padovan.org> Sender: linux-media-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-media@vger.kernel.org From: Gustavo Padovan Implement the needed pieces to let userspace subscribe for V4L2_EVENT_BUF_QUEUED events. Videobuf2 will queue the event for the DQEVENT ioctl. v3: - Do not call v4l2 event API from vb2 (Mauro) v2: - Use VIDEO_MAX_FRAME to allocate room for events at v4l2_event_subscribe() (Hans) Signed-off-by: Gustavo Padovan --- drivers/media/v4l2-core/v4l2-ctrls.c | 6 +++++- drivers/media/v4l2-core/videobuf2-core.c | 2 ++ drivers/media/v4l2-core/videobuf2-v4l2.c | 13 +++++++++++++ 3 files changed, 20 insertions(+), 1 deletion(-) diff --git a/drivers/media/v4l2-core/v4l2-ctrls.c b/drivers/media/v4l2-core/v4l2-ctrls.c index dd1db678718c..17d4b9e3eec6 100644 --- a/drivers/media/v4l2-core/v4l2-ctrls.c +++ b/drivers/media/v4l2-core/v4l2-ctrls.c @@ -3438,8 +3438,12 @@ EXPORT_SYMBOL(v4l2_ctrl_log_status); int v4l2_ctrl_subscribe_event(struct v4l2_fh *fh, const struct v4l2_event_subscription *sub) { - if (sub->type == V4L2_EVENT_CTRL) + switch (sub->type) { + case V4L2_EVENT_CTRL: return v4l2_event_subscribe(fh, sub, 0, &v4l2_ctrl_sub_ev_ops); + case V4L2_EVENT_BUF_QUEUED: + return v4l2_event_subscribe(fh, sub, VIDEO_MAX_FRAME, NULL); + } return -EINVAL; } EXPORT_SYMBOL(v4l2_ctrl_subscribe_event); diff --git a/drivers/media/v4l2-core/videobuf2-core.c b/drivers/media/v4l2-core/videobuf2-core.c index b19c1bc4b083..bbbae0eed567 100644 --- a/drivers/media/v4l2-core/videobuf2-core.c +++ b/drivers/media/v4l2-core/videobuf2-core.c @@ -1231,6 +1231,8 @@ static void __enqueue_in_driver(struct vb2_buffer *vb) trace_vb2_buf_queue(q, vb); call_void_vb_qop(vb, buf_queue, vb); + + call_void_bufop(q, buffer_queued, vb); } static int __buf_prepare(struct vb2_buffer *vb, const void *pb) diff --git a/drivers/media/v4l2-core/videobuf2-v4l2.c b/drivers/media/v4l2-core/videobuf2-v4l2.c index 8c322cd1b346..1c93bfedaffc 100644 --- a/drivers/media/v4l2-core/videobuf2-v4l2.c +++ b/drivers/media/v4l2-core/videobuf2-v4l2.c @@ -138,6 +138,18 @@ static void __copy_timestamp(struct vb2_buffer *vb, const void *pb) } }; +static void __buffer_queued(struct vb2_buffer *vb) +{ + struct video_device *vdev = to_video_device(vb->vb2_queue->dev); + struct v4l2_event event; + + memset(&event, 0, sizeof(event)); + event.type = V4L2_EVENT_BUF_QUEUED; + event.u.buf_queued.index = vb->index; + + v4l2_event_queue(vdev, &event); +} + static void vb2_warn_zero_bytesused(struct vb2_buffer *vb) { static bool check_once; @@ -455,6 +467,7 @@ static const struct vb2_buf_ops v4l2_buf_ops = { .fill_user_buffer = __fill_v4l2_buffer, .fill_vb2_buffer = __fill_vb2_buffer, .copy_timestamp = __copy_timestamp, + .buffer_queued = __buffer_queued, }; /**