[media] v4l: exynos4-is: Use the new media graph walk interface
authorSakari Ailus <sakari.ailus@iki.fi>
Wed, 16 Dec 2015 13:32:25 +0000 (15:32 +0200)
committerMauro Carvalho Chehab <mchehab@osg.samsung.com>
Mon, 11 Jan 2016 14:19:19 +0000 (12:19 -0200)
The media graph walk requires initialisation and cleanup soon. Update the
users to perform the soon necessary API calls.

Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>
Cc: Javier Martinez Canillas <javier@osg.samsung.com>
Cc: Kamil Debski <k.debski@samsung.com>
Cc: Sylwester Nawrocki <s.nawrocki@samsung.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com>
drivers/media/platform/exynos4-is/media-dev.c
drivers/media/platform/exynos4-is/media-dev.h

index 27663dd452949061eaeafda71386ec467dec98d6..f6b391e795c6541032b027a4496933b2227cb24d 100644 (file)
@@ -1046,10 +1046,10 @@ static int __fimc_md_modify_pipeline(struct media_entity *entity, bool enable)
 }
 
 /* Locking: called with entity->graph_obj.mdev->graph_mutex mutex held. */
-static int __fimc_md_modify_pipelines(struct media_entity *entity, bool enable)
+static int __fimc_md_modify_pipelines(struct media_entity *entity, bool enable,
+                                     struct media_entity_graph *graph)
 {
        struct media_entity *entity_err = entity;
-       struct media_entity_graph graph;
        int ret;
 
        /*
@@ -1058,9 +1058,9 @@ static int __fimc_md_modify_pipelines(struct media_entity *entity, bool enable)
         * through active links. This is needed as we cannot power on/off the
         * subdevs in random order.
         */
-       media_entity_graph_walk_start(&graph, entity);
+       media_entity_graph_walk_start(graph, entity);
 
-       while ((entity = media_entity_graph_walk_next(&graph))) {
+       while ((entity = media_entity_graph_walk_next(graph))) {
                if (!is_media_entity_v4l2_io(entity))
                        continue;
 
@@ -1071,10 +1071,11 @@ static int __fimc_md_modify_pipelines(struct media_entity *entity, bool enable)
        }
 
        return 0;
- err:
-       media_entity_graph_walk_start(&graph, entity_err);
 
-       while ((entity_err = media_entity_graph_walk_next(&graph))) {
+err:
+       media_entity_graph_walk_start(graph, entity_err);
+
+       while ((entity_err = media_entity_graph_walk_next(graph))) {
                if (!is_media_entity_v4l2_io(entity_err))
                        continue;
 
@@ -1090,21 +1091,29 @@ static int __fimc_md_modify_pipelines(struct media_entity *entity, bool enable)
 static int fimc_md_link_notify(struct media_link *link, unsigned int flags,
                                unsigned int notification)
 {
+       struct media_entity_graph *graph =
+               &container_of(link->graph_obj.mdev, struct fimc_md,
+                             media_dev)->link_setup_graph;
        struct media_entity *sink = link->sink->entity;
        int ret = 0;
 
        /* Before link disconnection */
        if (notification == MEDIA_DEV_NOTIFY_PRE_LINK_CH) {
+               ret = media_entity_graph_walk_init(graph,
+                                                  link->graph_obj.mdev);
+               if (ret)
+                       return ret;
                if (!(flags & MEDIA_LNK_FL_ENABLED))
-                       ret = __fimc_md_modify_pipelines(sink, false);
+                       ret = __fimc_md_modify_pipelines(sink, false, graph);
 #if 0
                else
                        /* TODO: Link state change validation */
 #endif
        /* After link activation */
-       } else if (notification == MEDIA_DEV_NOTIFY_POST_LINK_CH &&
-                  (link->flags & MEDIA_LNK_FL_ENABLED)) {
-               ret = __fimc_md_modify_pipelines(sink, true);
+       } else if (notification == MEDIA_DEV_NOTIFY_POST_LINK_CH) {
+               if (link->flags & MEDIA_LNK_FL_ENABLED)
+                       ret = __fimc_md_modify_pipelines(sink, true, graph);
+               media_entity_graph_walk_cleanup(graph);
        }
 
        return ret ? -EPIPE : 0;
index e8845e1f5aabedfee8d28eaa90264a5871b268db..ed122cb2dd7416bbc8fd3a6dbda4e7b2e0341b17 100644 (file)
@@ -154,6 +154,7 @@ struct fimc_md {
        bool user_subdev_api;
        spinlock_t slock;
        struct list_head pipelines;
+       struct media_entity_graph link_setup_graph;
 };
 
 static inline