Re: [PATCH v4 3/3] v4l: async: add subnotifier to subdevices

2017-07-18 Thread Niklas Söderlund
Hi Hans, On 2017-07-18 17:06:15 +0200, Hans Verkuil wrote: > On 18/07/17 16:47, Niklas Söderlund wrote: > >>> void v4l2_async_notifier_unregister(struct v4l2_async_notifier *notifier) > >>> { > >>> - struct v4l2_subdev *sd, *tmp; > >>> + struct v4l2_subdev *sd, *tmp, **subdev; > >>> unsigned i

Re: [PATCH v4 3/3] v4l: async: add subnotifier to subdevices

2017-07-18 Thread Hans Verkuil
On 18/07/17 16:47, Niklas Söderlund wrote: >>> void v4l2_async_notifier_unregister(struct v4l2_async_notifier *notifier) >>> { >>> - struct v4l2_subdev *sd, *tmp; >>> + struct v4l2_subdev *sd, *tmp, **subdev; >>> unsigned int notif_n_subdev = notifier->num_subdevs; >>> unsigned int n_

Re: [PATCH v4 3/3] v4l: async: add subnotifier to subdevices

2017-07-18 Thread Niklas Söderlund
Hi Hans, Thanks for your feedback. On 2017-07-18 16:22:43 +0200, Hans Verkuil wrote: > On 17/07/17 18:59, Niklas Söderlund wrote: > > Add a subdevice specific notifier which can be used by a subdevice > > driver to compliment the master device notifier to extend the subdevice > > compliment -> c

Re: [PATCH v4 3/3] v4l: async: add subnotifier to subdevices

2017-07-18 Thread Hans Verkuil
On 17/07/17 18:59, Niklas Söderlund wrote: > Add a subdevice specific notifier which can be used by a subdevice > driver to compliment the master device notifier to extend the subdevice compliment -> complement Just one character difference, but a wildly different meaning :-) Although it was ver

Re: [PATCH v4 3/3] v4l: async: add subnotifier to subdevices

2017-07-18 Thread Niklas Söderlund
Hi Geert, Thanks for your feedback. On 2017-07-18 09:11:19 +0200, Geert Uytterhoeven wrote: > Hi Niklas, > > On Mon, Jul 17, 2017 at 6:59 PM, Niklas Söderlund > wrote: > > Add a subdevice specific notifier which can be used by a subdevice > > driver to compliment the master device notifier to e

Re: [PATCH v4 3/3] v4l: async: add subnotifier to subdevices

2017-07-18 Thread Geert Uytterhoeven
Hi Niklas, On Mon, Jul 17, 2017 at 6:59 PM, Niklas Söderlund wrote: > Add a subdevice specific notifier which can be used by a subdevice > driver to compliment the master device notifier to extend the subdevice > discovery. > > The master device registers the subdevices closest to itself in its >

[PATCH v4 3/3] v4l: async: add subnotifier to subdevices

2017-07-17 Thread Niklas Söderlund
Add a subdevice specific notifier which can be used by a subdevice driver to compliment the master device notifier to extend the subdevice discovery. The master device registers the subdevices closest to itself in its notifier while the subdevice(s) register notifiers for their closest neighboring