Tested-by: Benoit Parrot <[email protected]>
Javier Martinez Canillas <[email protected]> wrote on Tue [2016-Feb-16 17:03:21 -0300]: > After sub-dev registration in v4l2_async_test_notify(), the v4l2-async > core calls the registered_async callback but if a sub-dev driver does > not implement it, v4l2_subdev_call() will return a -ENOIOCTLCMD which > should not be considered an error. > > Reported-by: Benoit Parrot <[email protected]> > Signed-off-by: Javier Martinez Canillas <[email protected]> > > --- > > Changes in v2: > - Check the return of v4l2_subdev_call (Benoit). > > drivers/media/v4l2-core/v4l2-async.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/media/v4l2-core/v4l2-async.c > b/drivers/media/v4l2-core/v4l2-async.c > index 716bfd47daab..a4b224d92572 100644 > --- a/drivers/media/v4l2-core/v4l2-async.c > +++ b/drivers/media/v4l2-core/v4l2-async.c > @@ -120,7 +120,7 @@ static int v4l2_async_test_notify(struct > v4l2_async_notifier *notifier, > } > > ret = v4l2_subdev_call(sd, core, registered_async); > - if (ret < 0) { > + if (ret < 0 && ret != -ENOIOCTLCMD) { > if (notifier->unbind) > notifier->unbind(notifier, sd, asd); > return ret; > -- > 2.5.0 >
