[PATCH v2] [media] v4l2-async: Always unregister the subdev on failure

2016-08-24 Thread Alban Bedel
() to plug these holes. Signed-off-by: Alban Bedel <alban.be...@avionic-design.de> --- Changelog: v2: * Added the missing unbind() calls as suggested by Javier. --- drivers/media/v4l2-core/v4l2-async.c | 29 + 1 file changed, 17 insertions(+), 12 deletions(-) diff

Re: [PATCH] [media] v4l2-async: Always unregister the subdev on failure

2016-08-24 Thread Alban Bedel
On Fri, 1 Jul 2016 13:55:44 +0200 Hans Verkuil <hverk...@xs4all.nl> wrote: > On 05/11/2016 06:32 PM, Alban Bedel wrote: > > On Wed, 11 May 2016 12:22:44 -0400 > > Javier Martinez Canillas <jav...@osg.samsung.com> wrote: > > > >> Hello Alban, > >

Re: [PATCH] [media] v4l2-async: Always unregister the subdev on failure

2016-05-11 Thread Alban Bedel
On Wed, 11 May 2016 12:22:44 -0400 Javier Martinez Canillas <jav...@osg.samsung.com> wrote: > Hello Alban, > > On 05/11/2016 11:40 AM, Alban Bedel wrote: > > In v4l2_async_test_notify() if the registered_async callback or the > > complete notifier returns an error the

[PATCH] [media] v4l2-async: Always unregister the subdev on failure

2016-05-11 Thread Alban Bedel
() to plug these holes. Signed-off-by: Alban Bedel <alban.be...@avionic-design.de> --- drivers/media/v4l2-core/v4l2-async.c | 10 -- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/drivers/media/v4l2-core/v4l2-async.c b/drivers/media/v4l2-core/v4l2-async.c index ceb28d4..4

[PATCH v2] [media] v4l2-async: Pass the v4l2_async_subdev to the unbind callback

2016-05-11 Thread Alban Bedel
v4l2_async_cleanup() is always called before calling the unbind() callback. However v4l2_async_cleanup() clears the asd member, so when calling the unbind() callback the v4l2_async_subdev is always NULL. To fix this save the asd before calling v4l2_async_cleanup(). Signed-off-by: Alban Bedel

[PATCH] [media] v4l2-async: Pass the v4l2_async_subdev to the unbind callback

2016-05-10 Thread Alban Bedel
v4l2_async_cleanup() is always called before before calling the unbind() callback. However v4l2_async_cleanup() clear the asd member, so when calling the unbind() callback the v4l2_async_subdev is always NULL. To fix this save the asd before calling v4l2_async_cleanup(). Signed-off-by: Alban