From: Pavel Machek <pavel-+ZI9xUNit7I@public.gmane.org>
To: Sakari Ailus
<sakari.ailus-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>,
Hans Verkuil <hverkuil-qWit8jRvyhVmR6Xm/wNWPw@public.gmane.org>,
Sakari Ailus <sakari.ailus-X3B1VOXEql0@public.gmane.org>
Cc: linux-media-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
niklas.soderlund-1zkq55x86MTxsAP9Fp7wbw@public.gmane.org,
robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org,
laurent.pinchart-ryLnwIuWjnjg/C1BVhZhaw@public.gmane.org,
devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
sre-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org
Subject: Re: [PATCH v7 11/18] v4l: async: Register sub-devices before calling bound callback
Date: Fri, 8 Sep 2017 14:28:40 +0200 [thread overview]
Message-ID: <20170908122840.GL18365@amd> (raw)
In-Reply-To: <20170903201805.31998-1-sakari.ailus-VuQAYsv1563Yd54FQh9/CA@public.gmane.org> <20170903174958.27058-18-sakari.ailus-VuQAYsv1563Yd54FQh9/CA@public.gmane.org> <4900fc41-1b57-deb6-2041-26a6333f2033-qWit8jRvyhVmR6Xm/wNWPw@public.gmane.org> <20170903174958.27058-17-sakari.ailus-VuQAYsv1563Yd54FQh9/CA@public.gmane.org> <3921fde8-6192-87d7-9c5d-5dd2035a9565-qWit8jRvyhVmR6Xm/wNWPw@public.gmane.org> <20170904163400.z26qmxuejhgdcmrw-S+BSfZ9RZZmRSg0ZkenSGLdO1Tsj/99ntUK59QYPAWc@public.gmane.org> <0bb75f81-cc81-a4bf-f2af-41862c1d777a-qWit8jRvyhVmR6Xm/wNWPw@public.gmane.org> <20170903174958.27058-16-sakari.ailus-VuQAYsv1563Yd54FQh9/CA@public.gmane.org> <20170904162925.hxtzy5jagv5ylq4c-S+BSfZ9RZZmRSg0ZkenSGLdO1Tsj/99ntUK59QYPAWc@public.gmane.org> <96371574-0205-fd0e-452e-d001695bd69e-qWit8jRvyhVmR6Xm/wNWPw@public.gmane.org> <20170903174958.27058-15-sakari.ailus-VuQAYsv1563Yd54FQh9/CA@public.gmane.org> <20170904162705.thujzc7xw6hgjau3-S+BSfZ9RZZmRSg0ZkenSGLdO1Tsj/99ntUK59QYPAWc@public.gmane.org> <9a68a9a6-0949-f1c2-f029-8045d7d688b0-qWit8jRvyhVmR6Xm/wNWPw@public.gmane.org> <20170903174958.27058-14-sakari.ailus-VuQAYsv1563Yd54FQh9/CA@public.gmane.org> <31d63a76-adab-2a04-12dc-4717b1512eaa-VuQAYsv1563Yd54FQh9/CA@public.gmane.org> <6ad1c25a-e2a7-b73f-4d7c-6a5c071e6366-qWit8jRvyhVmR6Xm/wNWPw@public.gmane.org> <201709041
[-- Attachment #1: Type: text/plain, Size: 583 bytes --]
On Sun 2017-09-03 20:49:51, Sakari Ailus wrote:
> Register the sub-device before calling the notifier's bound callback.
> Doing this the other way around is problematic as the struct v4l2_device
> has not assigned for the sub-device yet and may be required by the bound
> callback.
>
> Signed-off-by: Sakari Ailus <sakari.ailus-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
Acked-by: Pavel Machek <pavel-+ZI9xUNit7I@public.gmane.org>
--
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 181 bytes --]
WARNING: multiple messages have this Message-ID (diff)
From: Pavel Machek <pavel@ucw.cz>
To: Sakari Ailus <sakari.ailus@linux.intel.com>,
Hans Verkuil <hverkuil@xs4all.nl>,
Sakari Ailus <sakari.ailus@iki.fi>
Cc: linux-media@vger.kernel.org, niklas.soderlund@ragnatech.se,
robh@kernel.org, laurent.pinchart@ideasonboard.com,
devicetree@vger.kernel.org, sre@kernel.org
Subject: Re: [PATCH v7 11/18] v4l: async: Register sub-devices before calling bound callback
Date: Fri, 8 Sep 2017 14:28:40 +0200 [thread overview]
Message-ID: <20170908122840.GL18365@amd> (raw)
In-Reply-To: <20170903201805.31998-1-sakari.ailus@linux.intel.com> <20170903174958.27058-18-sakari.ailus@linux.intel.com> <4900fc41-1b57-deb6-2041-26a6333f2033@xs4all.nl> <20170903174958.27058-17-sakari.ailus@linux.intel.com> <3921fde8-6192-87d7-9c5d-5dd2035a9565@xs4all.nl> <20170904163400.z26qmxuejhgdcmrw@valkosipuli.retiisi.org.uk> <0bb75f81-cc81-a4bf-f2af-41862c1d777a@xs4all.nl> <20170903174958.27058-16-sakari.ailus@linux.intel.com> <20170904162925.hxtzy5jagv5ylq4c@valkosipuli.retiisi.org.uk> <96371574-0205-fd0e-452e-d001695bd69e@xs4all.nl> <20170903174958.27058-15-sakari.ailus@linux.intel.com> <20170904162705.thujzc7xw6hgjau3@valkosipuli.retiisi.org.uk> <9a68a9a6-0949-f1c2-f029-8045d7d688b0@xs4all.nl> <20170903174958.27058-14-sakari.ailus@linux.intel.com> <31d63a76-adab-2a04-12dc-4717b1512eaa@linux.intel.com> <6ad1c25a-e2a7-b73f-4d7c-6a5c071e6366@xs4all.nl> <20170904164201.7rycyycvrukiusjz@valkosipuli.retiisi.org.uk> <60db0a4b-0560-9591-f41f-1d055a50ba12@xs4all.nl> <20170903174958.27058-13-sakari.ailus@linux.intel.com> <20170903174958.27058-12-sakari.ailus@linux.intel.com>
[-- Attachment #1: Type: text/plain, Size: 537 bytes --]
On Sun 2017-09-03 20:49:51, Sakari Ailus wrote:
> Register the sub-device before calling the notifier's bound callback.
> Doing this the other way around is problematic as the struct v4l2_device
> has not assigned for the sub-device yet and may be required by the bound
> callback.
>
> Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>
Acked-by: Pavel Machek <pavel@ucw.cz>
--
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 181 bytes --]
next prev parent reply other threads:[~2017-09-08 12:28 UTC|newest]
Thread overview: 69+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-09-03 17:49 [PATCH v7 00/18] Unified fwnode endpoint parser, async sub-device notifier support, N9 flash DTS Sakari Ailus
2017-09-03 17:49 ` Sakari Ailus
2017-09-03 17:49 ` [PATCH v7 01/18] v4l: fwnode: Move KernelDoc documentation to the header Sakari Ailus
[not found] ` <20170903174958.27058-2-sakari.ailus-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
2017-09-04 15:49 ` Pavel Machek
2017-09-04 15:49 ` Pavel Machek
2017-09-05 14:53 ` Sakari Ailus
2017-09-05 14:53 ` Sakari Ailus
2017-09-03 17:49 ` [PATCH v7 02/18] v4l: async: Add V4L2 async documentation to the documentation build Sakari Ailus
2017-09-03 17:49 ` [PATCH v7 03/18] docs-rst: v4l: Include Qualcomm CAMSS in " Sakari Ailus
2017-09-03 17:49 ` [PATCH v7 04/18] v4l: fwnode: Support generic parsing of graph endpoints in a device Sakari Ailus
[not found] ` <20170903174958.27058-5-sakari.ailus-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
2017-09-04 13:19 ` Hans Verkuil
2017-09-04 13:19 ` Hans Verkuil
[not found] ` <e07f9b4d-e8dc-5598-98ee-3c69e3100b81-qWit8jRvyhVmR6Xm/wNWPw@public.gmane.org>
2017-09-04 15:54 ` Sakari Ailus
2017-09-04 15:54 ` Sakari Ailus
[not found] ` <20170904155415.nak4dofd2k6ytupa-S+BSfZ9RZZmRSg0ZkenSGLdO1Tsj/99ntUK59QYPAWc@public.gmane.org>
2017-09-04 17:37 ` Hans Verkuil
2017-09-04 17:37 ` Hans Verkuil
2017-09-04 20:30 ` Sakari Ailus
2017-09-03 17:49 ` [PATCH v7 06/18] rcar-vin: Use generic parser for parsing fwnode endpoints Sakari Ailus
2017-09-03 17:49 ` [PATCH v7 07/18] omap3isp: Fix check for our own sub-devices Sakari Ailus
[not found] ` <20170903174958.27058-8-sakari.ailus-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
2017-09-04 13:28 ` Hans Verkuil
2017-09-04 13:28 ` Hans Verkuil
[not found] ` <f8a4ba8c-f749-26ab-0897-c929b3f23e9f-qWit8jRvyhVmR6Xm/wNWPw@public.gmane.org>
2017-09-04 16:07 ` Sakari Ailus
2017-09-04 16:07 ` Sakari Ailus
2017-09-03 17:49 ` [PATCH v7 08/18] omap3isp: Print the name of the entity where no source pads could be found Sakari Ailus
2017-09-03 17:49 ` [PATCH v7 09/18] v4l: async: Move async subdev notifier operations to a separate structure Sakari Ailus
2017-09-04 13:48 ` Hans Verkuil
2017-09-03 17:49 ` [PATCH v7 10/18] v4l: async: Introduce macros for calling async ops callbacks Sakari Ailus
2017-09-04 13:50 ` Hans Verkuil
2017-09-04 16:09 ` Sakari Ailus
2017-09-03 17:49 ` [PATCH v7 11/18] v4l: async: Register sub-devices before calling bound callback Sakari Ailus
2017-09-03 17:49 ` [PATCH v7 13/18] dt: bindings: Add a binding for flash devices associated to a sensor Sakari Ailus
2017-09-04 14:31 ` Hans Verkuil
2017-09-04 16:27 ` Sakari Ailus
2017-09-03 17:49 ` [PATCH v7 17/18] smiapp: Add support for flash and lens devices Sakari Ailus
[not found] ` <20170903174958.27058-18-sakari.ailus-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
2017-09-03 20:18 ` [PATCH v7 1/1] dt: bindings: smiapp: Document lens-focus and flash properties Sakari Ailus
2017-09-03 20:18 ` Sakari Ailus
[not found] ` <20170903201805.31998-1-sakari.ailus-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
2017-09-08 12:28 ` Pavel Machek
2017-09-08 12:28 ` Pavel Machek
[not found] ` <20170903174958.27058-1-sakari.ailus-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
2017-09-03 17:49 ` [PATCH v7 05/18] omap3isp: Use generic parser for parsing fwnode endpoints Sakari Ailus
2017-09-03 17:49 ` Sakari Ailus
2017-09-03 17:49 ` [PATCH v7 12/18] v4l: async: Allow binding notifiers to sub-devices Sakari Ailus
2017-09-03 17:49 ` Sakari Ailus
[not found] ` <20170903174958.27058-13-sakari.ailus-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
2017-09-04 14:56 ` Hans Verkuil
2017-09-04 14:56 ` Hans Verkuil
[not found] ` <60db0a4b-0560-9591-f41f-1d055a50ba12-qWit8jRvyhVmR6Xm/wNWPw@public.gmane.org>
2017-09-04 16:42 ` Sakari Ailus
2017-09-04 16:42 ` Sakari Ailus
2017-09-05 6:49 ` Hans Verkuil
2017-09-05 8:36 ` Sakari Ailus
2017-09-03 17:49 ` [PATCH v7 14/18] dt: bindings: Add lens-focus binding for image sensors Sakari Ailus
2017-09-03 17:49 ` Sakari Ailus
[not found] ` <20170903174958.27058-15-sakari.ailus-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
2017-09-04 14:37 ` Hans Verkuil
2017-09-04 14:37 ` Hans Verkuil
2017-09-04 16:29 ` Sakari Ailus
[not found] ` <20170904162705.thujzc7xw6hgjau3-S+BSfZ9RZZmRSg0ZkenSGLdO1Tsj/99ntUK59QYPAWc@public.gmane.org>
[not found] ` <9a68a9a6-0949-f1c2-f029-8045d7d688b0-qWit8jRvyhVmR6Xm/wNWPw@public.gmane.org>
[not found] ` <20170903174958.27058-14-sakari.ailus-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
[not found] ` <31d63a76-adab-2a04-12dc-4717b1512eaa-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
[not found] ` <6ad1c25a-e2a7-b73f-4d7c-6a5c071e6366-qWit8jRvyhVmR6Xm/wNWPw@public.gmane.org>
2017-09-08 12:28 ` Pavel Machek [this message]
2017-09-08 12:28 ` [PATCH v7 11/18] v4l: async: Register sub-devices before calling bound callback Pavel Machek
2017-09-03 17:49 ` [PATCH v7 15/18] v4l2-fwnode: Add convenience function for parsing generic references Sakari Ailus
2017-09-03 17:49 ` Sakari Ailus
2017-09-04 14:44 ` Hans Verkuil
[not found] ` <0bb75f81-cc81-a4bf-f2af-41862c1d777a-qWit8jRvyhVmR6Xm/wNWPw@public.gmane.org>
[not found] ` <20170903174958.27058-16-sakari.ailus-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
[not found] ` <20170904162925.hxtzy5jagv5ylq4c-S+BSfZ9RZZmRSg0ZkenSGLdO1Tsj/99ntUK59QYPAWc@public.gmane.org>
2017-09-04 16:34 ` Sakari Ailus
2017-09-04 16:34 ` Sakari Ailus
[not found] ` <20170904163400.z26qmxuejhgdcmrw-S+BSfZ9RZZmRSg0ZkenSGLdO1Tsj/99ntUK59QYPAWc@public.gmane.org>
2017-09-04 17:38 ` Hans Verkuil
2017-09-04 17:38 ` Hans Verkuil
2017-09-03 17:49 ` [PATCH v7 16/18] v4l2-fwnode: Add convenience function for parsing common external refs Sakari Ailus
2017-09-03 17:49 ` Sakari Ailus
[not found] ` <20170903174958.27058-17-sakari.ailus-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
2017-09-04 14:58 ` Hans Verkuil
2017-09-04 14:58 ` Hans Verkuil
2017-09-03 17:49 ` [PATCH v7 18/18] arm: dts: omap3: N9/N950: Add flash references to the camera Sakari Ailus
2017-09-03 17:49 ` Sakari Ailus
2017-09-04 7:47 ` [PATCH v7 00/18] Unified fwnode endpoint parser, async sub-device notifier support, N9 flash DTS 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=20170908122840.GL18365@amd \
--to=pavel-+zi9xunit7i@public.gmane.org \
--cc=devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=hverkuil-qWit8jRvyhVmR6Xm/wNWPw@public.gmane.org \
--cc=laurent.pinchart-ryLnwIuWjnjg/C1BVhZhaw@public.gmane.org \
--cc=linux-media-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=niklas.soderlund-1zkq55x86MTxsAP9Fp7wbw@public.gmane.org \
--cc=robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org \
--cc=sakari.ailus-VuQAYsv1563Yd54FQh9/CA@public.gmane.org \
--cc=sakari.ailus-X3B1VOXEql0@public.gmane.org \
--cc=sre-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.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.