@@ -37,6 +37,10 @@ static int uvc_mc_register_entity(struct uvc_video_chain *chain,
if (sink == NULL)
return 0;
+ ret = v4l2_device_register_subdev(&chain->dev->vdev, &entity->subdev);
+ if (ret < 0)
+ return ret;
+
for (i = 0; i < entity->num_pads; ++i) {
struct media_entity *source;
struct uvc_entity *remote;
@@ -46,8 +50,10 @@ static int uvc_mc_register_entity(struct uvc_video_chain *chain,
continue;
remote = uvc_entity_by_id(chain->dev, entity->baSourceID[i]);
- if (remote == NULL)
- return -EINVAL;
+ if (remote == NULL) {
+ ret = -EINVAL;
+ goto error;
+ }
source = (UVC_ENTITY_TYPE(remote) == UVC_TT_STREAMING)
? (remote->vdev ? &remote->vdev->entity : NULL)
@@ -59,13 +65,15 @@ static int uvc_mc_register_entity(struct uvc_video_chain *chain,
ret = media_create_pad_link(source, remote_pad,
sink, i, flags);
if (ret < 0)
- return ret;
+ goto error;
}
if (UVC_ENTITY_TYPE(entity) == UVC_TT_STREAMING)
return 0;
- return v4l2_device_register_subdev(&chain->dev->vdev, &entity->subdev);
+error:
+ v4l2_device_unregister_subdev(&entity->subdev);
+ return ret;
}
static struct v4l2_subdev_ops uvc_subdev_ops = {
The uvc driver creates the pads links before the media entity is registered with the media device. This doesn't work now that obj IDs are used to create links so the media_device has to be set. Move entities registration logic before pads links creation. Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com> --- drivers/media/usb/uvc/uvc_entity.c | 16 ++++++++++++---- 1 file changed, 12 insertions(+), 4 deletions(-)