[media] media-device: move media entity register/unregister functions
authorMauro Carvalho Chehab <mchehab@osg.samsung.com>
Tue, 15 Dec 2015 10:26:52 +0000 (08:26 -0200)
committerMauro Carvalho Chehab <mchehab@osg.samsung.com>
Mon, 11 Jan 2016 14:19:14 +0000 (12:19 -0200)
media entity register and unregister functions are called by media
device register/unregister. Move them to occur earlier, as we'll need
an unlocked version of media_device_entity_unregister() and we don't
want to add a function prototype without needing it.

No functional changes.

Signed-off-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com>
drivers/media/media-device.c

index da4126863ecc9430d15b52aa5fdc40ba630168b5..1222fa642ad843560494eb3619f3907997f614ce 100644 (file)
@@ -528,6 +528,86 @@ static void media_device_release(struct media_devnode *mdev)
        dev_dbg(mdev->parent, "Media device released\n");
 }
 
+/**
+ * media_device_register_entity - Register an entity with a media device
+ * @mdev:      The media device
+ * @entity:    The entity
+ */
+int __must_check media_device_register_entity(struct media_device *mdev,
+                                             struct media_entity *entity)
+{
+       unsigned int i;
+
+       if (entity->function == MEDIA_ENT_F_V4L2_SUBDEV_UNKNOWN ||
+           entity->function == MEDIA_ENT_F_UNKNOWN)
+               dev_warn(mdev->dev,
+                        "Entity type for entity %s was not initialized!\n",
+                        entity->name);
+
+       /* Warn if we apparently re-register an entity */
+       WARN_ON(entity->graph_obj.mdev != NULL);
+       entity->graph_obj.mdev = mdev;
+       INIT_LIST_HEAD(&entity->links);
+       entity->num_links = 0;
+       entity->num_backlinks = 0;
+
+       spin_lock(&mdev->lock);
+       /* Initialize media_gobj embedded at the entity */
+       media_gobj_create(mdev, MEDIA_GRAPH_ENTITY, &entity->graph_obj);
+
+       /* Initialize objects at the pads */
+       for (i = 0; i < entity->num_pads; i++)
+               media_gobj_create(mdev, MEDIA_GRAPH_PAD,
+                              &entity->pads[i].graph_obj);
+
+       spin_unlock(&mdev->lock);
+
+       return 0;
+}
+EXPORT_SYMBOL_GPL(media_device_register_entity);
+
+/**
+ * media_device_unregister_entity - Unregister an entity
+ * @entity:    The entity
+ *
+ * If the entity has never been registered this function will return
+ * immediately.
+ */
+void media_device_unregister_entity(struct media_entity *entity)
+{
+       struct media_device *mdev = entity->graph_obj.mdev;
+       struct media_link *link, *tmp;
+       struct media_interface *intf;
+       unsigned int i;
+
+       if (mdev == NULL)
+               return;
+
+       spin_lock(&mdev->lock);
+
+       /* Remove all interface links pointing to this entity */
+       list_for_each_entry(intf, &mdev->interfaces, graph_obj.list) {
+               list_for_each_entry_safe(link, tmp, &intf->links, list) {
+                       if (link->entity == entity)
+                               __media_remove_intf_link(link);
+               }
+       }
+
+       /* Remove all data links that belong to this entity */
+       __media_entity_remove_links(entity);
+
+       /* Remove all pads that belong to this entity */
+       for (i = 0; i < entity->num_pads; i++)
+               media_gobj_destroy(&entity->pads[i].graph_obj);
+
+       /* Remove the entity */
+       media_gobj_destroy(&entity->graph_obj);
+
+       spin_unlock(&mdev->lock);
+       entity->graph_obj.mdev = NULL;
+}
+EXPORT_SYMBOL_GPL(media_device_unregister_entity);
+
 /**
  * media_device_register - register a media device
  * @mdev:      The media device
@@ -611,86 +691,6 @@ void media_device_unregister(struct media_device *mdev)
 }
 EXPORT_SYMBOL_GPL(media_device_unregister);
 
-/**
- * media_device_register_entity - Register an entity with a media device
- * @mdev:      The media device
- * @entity:    The entity
- */
-int __must_check media_device_register_entity(struct media_device *mdev,
-                                             struct media_entity *entity)
-{
-       unsigned int i;
-
-       if (entity->function == MEDIA_ENT_F_V4L2_SUBDEV_UNKNOWN ||
-           entity->function == MEDIA_ENT_F_UNKNOWN)
-               dev_warn(mdev->dev,
-                        "Entity type for entity %s was not initialized!\n",
-                        entity->name);
-
-       /* Warn if we apparently re-register an entity */
-       WARN_ON(entity->graph_obj.mdev != NULL);
-       entity->graph_obj.mdev = mdev;
-       INIT_LIST_HEAD(&entity->links);
-       entity->num_links = 0;
-       entity->num_backlinks = 0;
-
-       spin_lock(&mdev->lock);
-       /* Initialize media_gobj embedded at the entity */
-       media_gobj_create(mdev, MEDIA_GRAPH_ENTITY, &entity->graph_obj);
-
-       /* Initialize objects at the pads */
-       for (i = 0; i < entity->num_pads; i++)
-               media_gobj_create(mdev, MEDIA_GRAPH_PAD,
-                              &entity->pads[i].graph_obj);
-
-       spin_unlock(&mdev->lock);
-
-       return 0;
-}
-EXPORT_SYMBOL_GPL(media_device_register_entity);
-
-/**
- * media_device_unregister_entity - Unregister an entity
- * @entity:    The entity
- *
- * If the entity has never been registered this function will return
- * immediately.
- */
-void media_device_unregister_entity(struct media_entity *entity)
-{
-       struct media_device *mdev = entity->graph_obj.mdev;
-       struct media_link *link, *tmp;
-       struct media_interface *intf;
-       unsigned int i;
-
-       if (mdev == NULL)
-               return;
-
-       spin_lock(&mdev->lock);
-
-       /* Remove all interface links pointing to this entity */
-       list_for_each_entry(intf, &mdev->interfaces, graph_obj.list) {
-               list_for_each_entry_safe(link, tmp, &intf->links, list) {
-                       if (link->entity == entity)
-                               __media_remove_intf_link(link);
-               }
-       }
-
-       /* Remove all data links that belong to this entity */
-       __media_entity_remove_links(entity);
-
-       /* Remove all pads that belong to this entity */
-       for (i = 0; i < entity->num_pads; i++)
-               media_gobj_destroy(&entity->pads[i].graph_obj);
-
-       /* Remove the entity */
-       media_gobj_destroy(&entity->graph_obj);
-
-       spin_unlock(&mdev->lock);
-       entity->graph_obj.mdev = NULL;
-}
-EXPORT_SYMBOL_GPL(media_device_unregister_entity);
-
 static void media_device_release_devres(struct device *dev, void *res)
 {
 }