summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJavier Martinez Canillas2015-09-03 13:46:06 +0200
committerMauro Carvalho Chehab2016-01-11 15:18:46 +0100
commitc5a98cac6ecd78ca647c41cb25e3ea1835579d70 (patch)
treed320097f9a8ba2ca65606f15ea41279471e8c42c
parent[media] v4l: vsp1: separate links creation from entities init (diff)
downloadkernel-qcow2-linux-c5a98cac6ecd78ca647c41cb25e3ea1835579d70.tar.gz
kernel-qcow2-linux-c5a98cac6ecd78ca647c41cb25e3ea1835579d70.tar.xz
kernel-qcow2-linux-c5a98cac6ecd78ca647c41cb25e3ea1835579d70.zip
[media] uvcvideo: create pad links after subdev registration
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> Signed-off-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com>
-rw-r--r--drivers/media/usb/uvc/uvc_entity.c23
1 files changed, 19 insertions, 4 deletions
diff --git a/drivers/media/usb/uvc/uvc_entity.c b/drivers/media/usb/uvc/uvc_entity.c
index 429e428ccd93..7f82b65b238e 100644
--- a/drivers/media/usb/uvc/uvc_entity.c
+++ b/drivers/media/usb/uvc/uvc_entity.c
@@ -26,6 +26,15 @@
static int uvc_mc_register_entity(struct uvc_video_chain *chain,
struct uvc_entity *entity)
{
+ if (UVC_ENTITY_TYPE(entity) == UVC_TT_STREAMING)
+ return 0;
+
+ return v4l2_device_register_subdev(&chain->dev->vdev, &entity->subdev);
+}
+
+static int uvc_mc_create_pads_links(struct uvc_video_chain *chain,
+ struct uvc_entity *entity)
+{
const u32 flags = MEDIA_LNK_FL_ENABLED | MEDIA_LNK_FL_IMMUTABLE;
struct media_entity *sink;
unsigned int i;
@@ -62,10 +71,7 @@ static int uvc_mc_register_entity(struct uvc_video_chain *chain,
return ret;
}
- if (UVC_ENTITY_TYPE(entity) == UVC_TT_STREAMING)
- return 0;
-
- return v4l2_device_register_subdev(&chain->dev->vdev, &entity->subdev);
+ return 0;
}
static struct v4l2_subdev_ops uvc_subdev_ops = {
@@ -124,5 +130,14 @@ int uvc_mc_register_entities(struct uvc_video_chain *chain)
}
}
+ list_for_each_entry(entity, &chain->entities, chain) {
+ ret = uvc_mc_create_pads_links(chain, entity);
+ if (ret < 0) {
+ uvc_printk(KERN_INFO, "Failed to create pads links for "
+ "entity %u\n", entity->id);
+ return ret;
+ }
+ }
+
return 0;
}