All of lore.kernel.org
 help / color / mirror / Atom feed
From: Sakari Ailus <sakari.ailus@linux.intel.com>
To: linux-media@vger.kernel.org
Cc: niklas.soderlund@ragnatech.se, maxime.ripard@free-electrons.com,
	robh@kernel.org, hverkuil@xs4all.nl,
	laurent.pinchart@ideasonboard.com, devicetree@vger.kernel.org,
	pavel@ucw.cz, sre@kernel.org
Subject: [PATCH v13 13/25] v4l: async: Allow async notifier register call succeed with no subdevs
Date: Fri, 15 Sep 2017 17:17:12 +0300	[thread overview]
Message-ID: <20170915141724.23124-14-sakari.ailus@linux.intel.com> (raw)
In-Reply-To: <20170915141724.23124-1-sakari.ailus@linux.intel.com>

The information on how many async sub-devices would be bindable to a
notifier is typically dependent on information from platform firmware and
it's not driver's business to be aware of that.

Many V4L2 main drivers are perfectly usable (and useful) without async
sub-devices and so if there aren't any around, just proceed call the
notifier's complete callback immediately without registering the notifier
itself.

If a driver needs to check whether there are async sub-devices available,
it can be done by inspecting the notifier's num_subdevs field which tells
the number of async sub-devices.

Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>
Acked-by: Hans Verkuil <hans.verkuil@cisco.com>
---
 drivers/media/v4l2-core/v4l2-async.c | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/drivers/media/v4l2-core/v4l2-async.c b/drivers/media/v4l2-core/v4l2-async.c
index 9895b610e2a0..4be2f16af051 100644
--- a/drivers/media/v4l2-core/v4l2-async.c
+++ b/drivers/media/v4l2-core/v4l2-async.c
@@ -170,14 +170,16 @@ int v4l2_async_notifier_register(struct v4l2_device *v4l2_dev,
 	struct v4l2_async_subdev *asd;
 	int i;
 
-	if (!v4l2_dev || !notifier->num_subdevs ||
-	    notifier->num_subdevs > V4L2_MAX_SUBDEVS)
+	if (!v4l2_dev || notifier->num_subdevs > V4L2_MAX_SUBDEVS)
 		return -EINVAL;
 
 	notifier->v4l2_dev = v4l2_dev;
 	INIT_LIST_HEAD(&notifier->waiting);
 	INIT_LIST_HEAD(&notifier->done);
 
+	if (!notifier->num_subdevs)
+		return v4l2_async_notifier_call_complete(notifier);
+
 	for (i = 0; i < notifier->num_subdevs; i++) {
 		asd = notifier->subdevs[i];
 
-- 
2.11.0

  parent reply	other threads:[~2017-09-15 14:17 UTC|newest]

Thread overview: 137+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-09-15 14:16 [PATCH v13 00/25] Unified fwnode endpoint parser, async sub-device notifier support, N9 flash DTS Sakari Ailus
2017-09-15 14:16 ` Sakari Ailus
2017-09-15 14:17 ` [PATCH v13 02/25] v4l: async: Remove re-probing support Sakari Ailus
     [not found]   ` <20170915141724.23124-3-sakari.ailus-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
2017-09-19  9:21     ` Laurent Pinchart
2017-09-19  9:21       ` Laurent Pinchart
2017-09-15 14:17 ` [PATCH v13 04/25] v4l: async: Add V4L2 async documentation to the documentation build Sakari Ailus
     [not found] ` <20170915141724.23124-1-sakari.ailus-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
2017-09-15 14:17   ` [PATCH v13 01/25] v4l: fwnode: Move KernelDoc documentation to the header Sakari Ailus
2017-09-15 14:17     ` Sakari Ailus
     [not found]     ` <20170915141724.23124-2-sakari.ailus-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
2017-09-19 10:48       ` Laurent Pinchart
2017-09-19 10:48         ` Laurent Pinchart
2017-09-19 11:04         ` Hans Verkuil
2017-09-19 11:04           ` Hans Verkuil
     [not found]           ` <29354478-ec46-278b-c457-4e6f3cc6848c-qWit8jRvyhVmR6Xm/wNWPw@public.gmane.org>
2017-09-19 11:07             ` Laurent Pinchart
2017-09-19 11:07               ` Laurent Pinchart
2017-09-19 11:22             ` Sakari Ailus
2017-09-19 11:22               ` Sakari Ailus
2017-09-19 11:10         ` Sakari Ailus
2017-09-19 11:10           ` Sakari Ailus
     [not found]           ` <20170919111036.5va2unwqh2vymojr-S+BSfZ9RZZmRSg0ZkenSGLdO1Tsj/99ntUK59QYPAWc@public.gmane.org>
2017-09-19 11:14             ` Laurent Pinchart
2017-09-19 11:14               ` Laurent Pinchart
2017-09-19 11:25               ` Sakari Ailus
2017-09-19 11:25                 ` Sakari Ailus
2017-09-15 14:17   ` [PATCH v13 03/25] v4l: async: Use more intuitive names for internal functions Sakari Ailus
2017-09-15 14:17     ` Sakari Ailus
2017-09-15 14:17   ` [PATCH v13 05/25] v4l: fwnode: Support generic parsing of graph endpoints in a device Sakari Ailus
2017-09-15 14:17     ` Sakari Ailus
2017-09-19  8:03     ` Hans Verkuil
2017-09-19  8:20       ` Sakari Ailus
     [not found]         ` <20170919082015.vt6olgirnvmpcrpa-z7MJbOB4PBP+e+fPlCVrcFDQ4js95KgL@public.gmane.org>
2017-09-19  8:40           ` Hans Verkuil
2017-09-19  8:40             ` Hans Verkuil
     [not found]             ` <af99e12c-6fb8-a633-eec2-c1eb9d82226a-qWit8jRvyhVmR6Xm/wNWPw@public.gmane.org>
2017-09-19  9:30               ` Laurent Pinchart
2017-09-19  9:30                 ` Laurent Pinchart
2017-09-19 11:37                 ` Sakari Ailus
2017-09-19 10:00               ` Sakari Ailus
2017-09-19 10:00                 ` Sakari Ailus
2017-09-19 10:10                 ` Hans Verkuil
2017-09-19 11:35     ` Laurent Pinchart
2017-09-19 12:11       ` Sakari Ailus
     [not found]         ` <20170919121131.6m4cf4ftzhq7vpnc-z7MJbOB4PBP+e+fPlCVrcFDQ4js95KgL@public.gmane.org>
2017-09-19 12:34           ` Laurent Pinchart
2017-09-19 12:34             ` Laurent Pinchart
2017-09-26 20:56             ` Sakari Ailus
2017-09-15 14:17   ` [PATCH v13 07/25] rcar-vin: Use generic parser for parsing fwnode endpoints Sakari Ailus
2017-09-15 14:17     ` Sakari Ailus
     [not found]     ` <20170915141724.23124-8-sakari.ailus-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
2017-09-19 11:53       ` Laurent Pinchart
2017-09-19 11:53         ` Laurent Pinchart
2017-09-19 12:39         ` Sakari Ailus
2017-09-15 14:17   ` [PATCH v13 09/25] omap3isp: Print the name of the entity where no source pads could be found Sakari Ailus
2017-09-15 14:17     ` Sakari Ailus
2017-09-19 11:56     ` Laurent Pinchart
2017-09-15 14:17   ` [PATCH v13 11/25] v4l: async: Introduce helpers for calling async ops callbacks Sakari Ailus
2017-09-15 14:17     ` Sakari Ailus
     [not found]     ` <20170915141724.23124-12-sakari.ailus-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
2017-09-16  7:13       ` Pavel Machek
2017-09-16  7:13         ` Pavel Machek
2017-09-19 12:01       ` Laurent Pinchart
2017-09-19 12:01         ` Laurent Pinchart
2017-09-19 12:13         ` Sakari Ailus
2017-09-19 12:43           ` Laurent Pinchart
2017-09-19 14:50             ` Sakari Ailus
2017-09-19 14:50               ` Sakari Ailus
2017-09-19 16:27               ` Laurent Pinchart
2017-09-20 15:54                 ` Sakari Ailus
2017-09-20 15:54                   ` Sakari Ailus
2017-09-15 14:17   ` [PATCH v13 14/25] v4l: async: Allow binding notifiers to sub-devices Sakari Ailus
2017-09-15 14:17     ` Sakari Ailus
     [not found]     ` <20170915141724.23124-15-sakari.ailus-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
2017-09-19  8:06       ` Hans Verkuil
2017-09-19  8:06         ` Hans Verkuil
2017-09-19 13:52       ` Laurent Pinchart
2017-09-19 13:52         ` Laurent Pinchart
2017-09-19 15:17         ` Sakari Ailus
     [not found]           ` <20170919151732.4yafxfcxrreizd7r-S+BSfZ9RZZmRSg0ZkenSGLdO1Tsj/99ntUK59QYPAWc@public.gmane.org>
2017-09-19 17:52             ` Laurent Pinchart
2017-09-19 17:52               ` Laurent Pinchart
2017-09-20  9:42               ` Sakari Ailus
2017-09-15 14:17   ` [PATCH v13 19/25] v4l: fwnode: Add convenience function for parsing common external refs Sakari Ailus
2017-09-15 14:17     ` Sakari Ailus
2017-09-15 14:17   ` [PATCH v13 21/25] smiapp: Add support for flash and lens devices Sakari Ailus
2017-09-15 14:17     ` Sakari Ailus
     [not found]     ` <20170915141724.23124-22-sakari.ailus-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
2017-09-19 12:08       ` Laurent Pinchart
2017-09-19 12:08         ` Laurent Pinchart
2017-09-19 12:20         ` Sakari Ailus
2017-09-19 12:38           ` Laurent Pinchart
2017-09-15 14:17   ` [PATCH v13 23/25] ov5670: " Sakari Ailus
2017-09-15 14:17     ` Sakari Ailus
2017-09-15 14:17   ` [PATCH v13 24/25] ov13858: " Sakari Ailus
2017-09-15 14:17     ` Sakari Ailus
2017-09-15 14:17 ` [PATCH v13 06/25] omap3isp: Use generic parser for parsing fwnode endpoints Sakari Ailus
2017-09-16  7:04   ` Pavel Machek
2017-09-16  7:18     ` Pavel Machek
2017-09-16  7:18       ` Pavel Machek
2017-09-19 11:40   ` Laurent Pinchart
2017-09-19 12:41     ` Sakari Ailus
2017-09-19 12:43     ` Sakari Ailus
2017-09-19 12:46       ` Laurent Pinchart
2017-09-19 14:47         ` Sakari Ailus
2017-09-19 14:47           ` Sakari Ailus
2017-09-19 16:12           ` Laurent Pinchart
2017-09-21  8:46             ` Sakari Ailus
2017-09-21  8:46               ` Sakari Ailus
2017-09-20 15:50         ` Sakari Ailus
2017-09-20 15:50           ` Sakari Ailus
2017-09-15 14:17 ` [PATCH v13 08/25] omap3isp: Fix check for our own sub-devices Sakari Ailus
     [not found]   ` <20170915141724.23124-9-sakari.ailus-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
2017-09-19 11:55     ` Laurent Pinchart
2017-09-19 11:55       ` Laurent Pinchart
2017-09-15 14:17 ` [PATCH v13 10/25] v4l: async: Move async subdev notifier operations to a separate structure Sakari Ailus
2017-09-15 14:17 ` [PATCH v13 12/25] v4l: async: Register sub-devices before calling bound callback Sakari Ailus
2017-09-19 12:01   ` Laurent Pinchart
2017-09-15 14:17 ` Sakari Ailus [this message]
2017-09-19 12:04   ` [PATCH v13 13/25] v4l: async: Allow async notifier register call succeed with no subdevs Laurent Pinchart
2017-09-19 12:52     ` Laurent Pinchart
2017-09-19 14:58       ` Sakari Ailus
     [not found]         ` <20170919145831.uztphjdtd3fdxzvr-S+BSfZ9RZZmRSg0ZkenSGLdO1Tsj/99ntUK59QYPAWc@public.gmane.org>
2017-09-19 15:03           ` Sakari Ailus
2017-09-19 15:03             ` Sakari Ailus
2017-09-19 17:54             ` Laurent Pinchart
2017-09-20 10:51               ` Sakari Ailus
2017-09-20 10:51                 ` Sakari Ailus
2017-09-15 14:17 ` [PATCH v13 15/25] dt: bindings: Add a binding for flash LED devices associated to a sensor Sakari Ailus
     [not found]   ` <20170915141724.23124-16-sakari.ailus-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
2017-09-19 12:12     ` Laurent Pinchart
2017-09-19 12:12       ` Laurent Pinchart
2017-09-19 12:16       ` Sakari Ailus
2017-09-19 12:16         ` Sakari Ailus
2017-09-15 14:17 ` [PATCH v13 16/25] dt: bindings: Add lens-focus binding for image sensors Sakari Ailus
2017-09-15 14:17 ` [PATCH v13 17/25] v4l: fwnode: Add a helper function for parsing generic references Sakari Ailus
     [not found]   ` <20170915141724.23124-18-sakari.ailus-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
2017-09-19  8:14     ` Hans Verkuil
2017-09-19  8:14       ` Hans Verkuil
2017-09-19 12:19     ` Laurent Pinchart
2017-09-19 12:19       ` Laurent Pinchart
2017-09-19 13:04       ` Sakari Ailus
     [not found]         ` <20170919130453.ii5kz54qxlot4of2-z7MJbOB4PBP+e+fPlCVrcFDQ4js95KgL@public.gmane.org>
2017-09-19 14:01           ` Laurent Pinchart
2017-09-19 14:01             ` Laurent Pinchart
2017-09-15 14:17 ` [PATCH v13 18/25] v4l: fwnode: Add a helper function to obtain device / integer references Sakari Ailus
2017-09-19  8:31   ` Hans Verkuil
2017-09-19  8:45     ` Sakari Ailus
2017-09-19  9:21       ` Hans Verkuil
     [not found]         ` <09f8ce37-c6e0-e448-c773-e1f3510d1024-qWit8jRvyhVmR6Xm/wNWPw@public.gmane.org>
2017-09-19 10:16           ` Sakari Ailus
2017-09-19 10:16             ` Sakari Ailus
2017-09-15 14:17 ` [PATCH v13 20/25] dt: bindings: smiapp: Document lens-focus and flash-leds properties Sakari Ailus
2017-09-15 14:17 ` [PATCH v13 22/25] et8ek8: Add support for flash and lens devices Sakari Ailus
2017-09-15 14:17 ` [PATCH v13 25/25] arm: dts: omap3: N9/N950: Add flash references to the camera Sakari Ailus

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20170915141724.23124-14-sakari.ailus@linux.intel.com \
    --to=sakari.ailus@linux.intel.com \
    --cc=devicetree@vger.kernel.org \
    --cc=hverkuil@xs4all.nl \
    --cc=laurent.pinchart@ideasonboard.com \
    --cc=linux-media@vger.kernel.org \
    --cc=maxime.ripard@free-electrons.com \
    --cc=niklas.soderlund@ragnatech.se \
    --cc=pavel@ucw.cz \
    --cc=robh@kernel.org \
    --cc=sre@kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.