[media] media-entity: get rid of forward __media_entity_remove_link() declaration
authorMauro Carvalho Chehab <mchehab@osg.samsung.com>
Fri, 11 Dec 2015 13:19:38 +0000 (11:19 -0200)
committerMauro Carvalho Chehab <mchehab@osg.samsung.com>
Mon, 11 Jan 2016 14:19:05 +0000 (12:19 -0200)
Move this function to happen earlier, in order to avoid
a uneeded forward declaration.

Suggested-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com>
drivers/media/media-entity.c

index 5a5432524c102e96269cf453948125b7e30afee0..452af1d5a20d9cc9e27cbbbf9506b3d1a5785e20 100644 (file)
@@ -588,7 +588,38 @@ static struct media_link *media_add_link(struct list_head *head)
 }
 
 static void __media_entity_remove_link(struct media_entity *entity,
-                                      struct media_link *link);
+                                      struct media_link *link)
+{
+       struct media_link *rlink, *tmp;
+       struct media_entity *remote;
+       unsigned int r = 0;
+
+       if (link->source->entity == entity)
+               remote = link->sink->entity;
+       else
+               remote = link->source->entity;
+
+       list_for_each_entry_safe(rlink, tmp, &remote->links, list) {
+               if (rlink != link->reverse) {
+                       r++;
+                       continue;
+               }
+
+               if (link->source->entity == entity)
+                       remote->num_backlinks--;
+
+               /* Remove the remote link */
+               list_del(&rlink->list);
+               media_gobj_remove(&rlink->graph_obj);
+               kfree(rlink);
+
+               if (--remote->num_links == 0)
+                       break;
+       }
+       list_del(&link->list);
+       media_gobj_remove(&link->graph_obj);
+       kfree(link);
+}
 
 int
 media_create_pad_link(struct media_entity *source, u16 source_pad,
@@ -642,40 +673,6 @@ media_create_pad_link(struct media_entity *source, u16 source_pad,
 }
 EXPORT_SYMBOL_GPL(media_create_pad_link);
 
-static void __media_entity_remove_link(struct media_entity *entity,
-                                      struct media_link *link)
-{
-       struct media_link *rlink, *tmp;
-       struct media_entity *remote;
-       unsigned int r = 0;
-
-       if (link->source->entity == entity)
-               remote = link->sink->entity;
-       else
-               remote = link->source->entity;
-
-       list_for_each_entry_safe(rlink, tmp, &remote->links, list) {
-               if (rlink != link->reverse) {
-                       r++;
-                       continue;
-               }
-
-               if (link->source->entity == entity)
-                       remote->num_backlinks--;
-
-               /* Remove the remote link */
-               list_del(&rlink->list);
-               media_gobj_remove(&rlink->graph_obj);
-               kfree(rlink);
-
-               if (--remote->num_links == 0)
-                       break;
-       }
-       list_del(&link->list);
-       media_gobj_remove(&link->graph_obj);
-       kfree(link);
-}
-
 void __media_entity_remove_links(struct media_entity *entity)
 {
        struct media_link *link, *tmp;