Re: [PATCH v13 11/25] v4l: async: Introduce helpers for calling async ops callbacks

2017-09-20 Thread Sakari Ailus
Hi Laurent, On Tue, Sep 19, 2017 at 07:27:17PM +0300, Laurent Pinchart wrote: > Hi Sakari, > > On Tuesday, 19 September 2017 17:50:49 EEST Sakari Ailus wrote: > > On Tue, Sep 19, 2017 at 03:43:48PM +0300, Laurent Pinchart wrote: > > > On Tuesday, 19 September 2017 15:13:11 EEST Sakari Ailus wrote

Re: [PATCH v13 11/25] v4l: async: Introduce helpers for calling async ops callbacks

2017-09-19 Thread Laurent Pinchart
Hi Sakari, On Tuesday, 19 September 2017 17:50:49 EEST Sakari Ailus wrote: > On Tue, Sep 19, 2017 at 03:43:48PM +0300, Laurent Pinchart wrote: > > On Tuesday, 19 September 2017 15:13:11 EEST Sakari Ailus wrote: > >> On Tue, Sep 19, 2017 at 03:01:14PM +0300, Laurent Pinchart wrote: > >>> On Friday,

Re: [PATCH v13 11/25] v4l: async: Introduce helpers for calling async ops callbacks

2017-09-19 Thread Sakari Ailus
Hi Laurent, On Tue, Sep 19, 2017 at 03:43:48PM +0300, Laurent Pinchart wrote: > Hi Sakari, > > On Tuesday, 19 September 2017 15:13:11 EEST Sakari Ailus wrote: > > On Tue, Sep 19, 2017 at 03:01:14PM +0300, Laurent Pinchart wrote: > > > On Friday, 15 September 2017 17:17:10 EEST Sakari Ailus wrote:

Re: [PATCH v13 11/25] v4l: async: Introduce helpers for calling async ops callbacks

2017-09-19 Thread Laurent Pinchart
Hi Sakari, On Tuesday, 19 September 2017 15:13:11 EEST Sakari Ailus wrote: > On Tue, Sep 19, 2017 at 03:01:14PM +0300, Laurent Pinchart wrote: > > On Friday, 15 September 2017 17:17:10 EEST Sakari Ailus wrote: > >> Add three helper functions to call async operations callbacks. Besides > >> simplif

Re: [PATCH v13 11/25] v4l: async: Introduce helpers for calling async ops callbacks

2017-09-19 Thread Sakari Ailus
Hi Laurent, On Tue, Sep 19, 2017 at 03:01:14PM +0300, Laurent Pinchart wrote: > Hi Sakari, > > Thank you for the patch. > > On Friday, 15 September 2017 17:17:10 EEST Sakari Ailus wrote: > > Add three helper functions to call async operations callbacks. Besides > > simplifying callbacks, this al

Re: [PATCH v13 11/25] v4l: async: Introduce helpers for calling async ops callbacks

2017-09-19 Thread Laurent Pinchart
Hi Sakari, Thank you for the patch. On Friday, 15 September 2017 17:17:10 EEST Sakari Ailus wrote: > Add three helper functions to call async operations callbacks. Besides > simplifying callbacks, this allows async notifiers to have no ops set, > i.e. it can be left NULL. > > Signed-off-by: Saka

Re: [PATCH v13 11/25] v4l: async: Introduce helpers for calling async ops callbacks

2017-09-16 Thread Pavel Machek
On Fri 2017-09-15 17:17:10, Sakari Ailus wrote: > Add three helper functions to call async operations callbacks. Besides > simplifying callbacks, this allows async notifiers to have no ops set, > i.e. it can be left NULL. > > Signed-off-by: Sakari Ailus > Acked-by: Hans Verkuil I'd remove "_cal

[PATCH v13 11/25] v4l: async: Introduce helpers for calling async ops callbacks

2017-09-15 Thread Sakari Ailus
Add three helper functions to call async operations callbacks. Besides simplifying callbacks, this allows async notifiers to have no ops set, i.e. it can be left NULL. Signed-off-by: Sakari Ailus Acked-by: Hans Verkuil --- drivers/media/v4l2-core/v4l2-async.c | 49 ++