Re: [PATCH] v4l2-async: add subnotifier registration for subdevices

2017-05-03 Thread Niklas Söderlund
Hej Sakari, Tack för dina kommentarer. On 2017-05-03 22:51:46 +0300, Sakari Ailus wrote: > Hejssan! > > On Fri, Apr 28, 2017 at 01:47:48PM +0200, Niklas Söderlund wrote: > > On 2017-04-28 13:28:17 +0300, Sakari Ailus wrote: > > > Hi Niklas, > > > > > > Thank you for the patch. > > > > > > Do

Re: [PATCH] v4l2-async: add subnotifier registration for subdevices

2017-05-03 Thread Niklas Söderlund
Hi Kieran, Thanks for your feedback. On 2017-05-03 18:07:47 +0100, Kieran Bingham wrote: > Hi Niklas, > > Small thing to ponder discovered below: > > On 27/04/17 23:30, Niklas Söderlund wrote: > > When registered() of v4l2_subdev_internal_ops is called the subdevice > > have access to the

Re: [PATCH] v4l2-async: add subnotifier registration for subdevices

2017-05-03 Thread Sakari Ailus
Hejssan! On Fri, Apr 28, 2017 at 01:47:48PM +0200, Niklas Söderlund wrote: > On 2017-04-28 13:28:17 +0300, Sakari Ailus wrote: > > Hi Niklas, > > > > Thank you for the patch. > > > > Do you happen to have a driver that would use this, to see some example of > > how the code is to be used? > >

Re: [PATCH] v4l2-async: add subnotifier registration for subdevices

2017-05-03 Thread Kieran Bingham
Hi Niklas, Small thing to ponder discovered below: On 27/04/17 23:30, Niklas Söderlund wrote: > When registered() of v4l2_subdev_internal_ops is called the subdevice > have access to the master devices v4l2_dev and it's called with the > async frameworks list_lock held. In this context the

Re: [PATCH] v4l2-async: add subnotifier registration for subdevices

2017-04-28 Thread Niklas Söderlund
Hi Sakari, Thanks for your feedback. On 2017-04-28 13:28:17 +0300, Sakari Ailus wrote: > Hi Niklas, > > Thank you for the patch. > > Do you happen to have a driver that would use this, to see some example of > how the code is to be used? Yes, the latest R-Car CSI-2 series make use of this,

Re: [PATCH] v4l2-async: add subnotifier registration for subdevices

2017-04-28 Thread Sakari Ailus
Hi Niklas, Thank you for the patch. Do you happen to have a driver that would use this, to see some example of how the code is to be used? Could you update the documentation in Documentation/media/kapi/v4l2-subdev.rst, too? On Fri, Apr 28, 2017 at 12:30:35AM +0200, Niklas Söderlund wrote: >

[PATCH] v4l2-async: add subnotifier registration for subdevices

2017-04-27 Thread Niklas Söderlund
When registered() of v4l2_subdev_internal_ops is called the subdevice have access to the master devices v4l2_dev and it's called with the async frameworks list_lock held. In this context the subdevice can register its own notifiers to allow for incremental discovery of subdevices. The master