Re: [PATCH v15 04/32] v4l: async: Fix notifier complete callback error handling

2017-10-10 Thread Sakari Ailus
Hi Hans, On Tue, Oct 10, 2017 at 03:18:13PM +0200, Hans Verkuil wrote: > On 10/10/2017 02:57 PM, Sakari Ailus wrote: > > Hi Hans, > > > > On Mon, Oct 09, 2017 at 01:45:25PM +0200, Hans Verkuil wrote: > >> On 04/10/17 23:50, Sakari Ailus wrote: > >>> The notifier complete callback may return an er

Re: [PATCH v15 04/32] v4l: async: Fix notifier complete callback error handling

2017-10-10 Thread Hans Verkuil
On 10/10/2017 02:57 PM, Sakari Ailus wrote: > Hi Hans, > > On Mon, Oct 09, 2017 at 01:45:25PM +0200, Hans Verkuil wrote: >> On 04/10/17 23:50, Sakari Ailus wrote: >>> The notifier complete callback may return an error. This error code was >>> simply returned to the caller but never handled properl

Re: [PATCH v15 04/32] v4l: async: Fix notifier complete callback error handling

2017-10-10 Thread Sakari Ailus
Hi Hans, On Mon, Oct 09, 2017 at 01:45:25PM +0200, Hans Verkuil wrote: > On 04/10/17 23:50, Sakari Ailus wrote: > > The notifier complete callback may return an error. This error code was > > simply returned to the caller but never handled properly. > > > > Move calling the complete callback func

Re: [PATCH v15 04/32] v4l: async: Fix notifier complete callback error handling

2017-10-09 Thread Hans Verkuil
On 04/10/17 23:50, Sakari Ailus wrote: > The notifier complete callback may return an error. This error code was > simply returned to the caller but never handled properly. > > Move calling the complete callback function to the caller from > v4l2_async_test_notify and undo the work that was done e

[PATCH v15 04/32] v4l: async: Fix notifier complete callback error handling

2017-10-04 Thread Sakari Ailus
The notifier complete callback may return an error. This error code was simply returned to the caller but never handled properly. Move calling the complete callback function to the caller from v4l2_async_test_notify and undo the work that was done either in async sub-device or async notifier regis