Hi Philipp, Thank you for the patch. On Wednesday 04 June 2014 18:57:02 Philipp Zabel wrote: > Add support for registering the sensor subdevice using the v4l2-async API. > > Signed-off-by: Philipp Zabel <p.zabel@xxxxxxxxxxxxxx> Acked-by; Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx> and applied to my tree. > --- > Changes since v1: > - Simplified error path, call media_entity_cleanup even > if media_entity_init failed. > --- > drivers/media/i2c/mt9v032.c | 12 +++++++++++- > 1 file changed, 11 insertions(+), 1 deletion(-) > > diff --git a/drivers/media/i2c/mt9v032.c b/drivers/media/i2c/mt9v032.c > index 6c97dc1..cbd3546 100644 > --- a/drivers/media/i2c/mt9v032.c > +++ b/drivers/media/i2c/mt9v032.c > @@ -1010,10 +1010,19 @@ static int mt9v032_probe(struct i2c_client *client, > > mt9v032->pad.flags = MEDIA_PAD_FL_SOURCE; > ret = media_entity_init(&mt9v032->subdev.entity, 1, &mt9v032->pad, 0); > + if (ret < 0) > + goto err; > > + mt9v032->subdev.dev = &client->dev; > + ret = v4l2_async_register_subdev(&mt9v032->subdev); > if (ret < 0) > - v4l2_ctrl_handler_free(&mt9v032->ctrls); > + goto err; > + > + return 0; > > +err: > + media_entity_cleanup(&mt9v032->subdev.entity); > + v4l2_ctrl_handler_free(&mt9v032->ctrls); > return ret; > } > > @@ -1022,6 +1031,7 @@ static int mt9v032_remove(struct i2c_client *client) > struct v4l2_subdev *subdev = i2c_get_clientdata(client); > struct mt9v032 *mt9v032 = to_mt9v032(subdev); > > + v4l2_async_unregister_subdev(subdev); > v4l2_ctrl_handler_free(&mt9v032->ctrls); > v4l2_device_unregister_subdev(subdev); > media_entity_cleanup(&subdev->entity); -- Regards, Laurent Pinchart -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
