[media] media: Change media device link_notify behaviour

Currently the media device link_notify callback is invoked before the
actual change of state of a link when the link is being enabled, and
after the actual change of state when the link is being disabled.
This doesn't allow a media device driver to perform any operations
on a full graph before a link is disabled, as well as performing
any tasks on a modified graph right after a link's state is changed.
This patch modifies signature of the link_notify callback. This
callback is now called always before and after a link's state change.
To distinguish the notifications a 'notification' argument is added
to the link_notify callback: MEDIA_DEV_NOTIFY_PRE_LINK_CH indicates
notification before link's state change and
MEDIA_DEV_NOTIFY_POST_LINK_CH corresponds to a notification after
link flags change.

[mchehab@redhat.com: whitespace cleanups]
Signed-off-by: Sylwester Nawrocki <s.nawrocki@samsung.com>
Signed-off-by: Kyungmin Park <kyungmin.park@samsung.com>
Acked-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Acked-by: Sakari Ailus <sakari.ailus@iki.fi>

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

1 files changed, 5 insertions, 13 deletions

diff --git a/drivers/media/media-entity.c b/drivers/media/media-entity.c
index 0438209d020a..df72f7d99d80 100644
--- a/drivers/media/media-entity.c
+++ b/drivers/media/media-entity.c
@@ -496,25 +496,17 @@ int __media_entity_setup_link(struct media_link *link, u32 flags)
 
         mdev = source->parent;
 
-        if ((flags & MEDIA_LNK_FL_ENABLED) && mdev->link_notify) {
-                ret = mdev->link_notify(link->source, link->sink,
-                                        MEDIA_LNK_FL_ENABLED);
+        if (mdev->link_notify) {
+                ret = mdev->link_notify(link, flags,
+                                        MEDIA_DEV_NOTIFY_PRE_LINK_CH);
                 if (ret < 0)
                         return ret;
         }
 
         ret = __media_entity_setup_link_notify(link, flags);
-        if (ret < 0)
-                goto err;
 
-        if (!(flags & MEDIA_LNK_FL_ENABLED) && mdev->link_notify)
-                mdev->link_notify(link->source, link->sink, 0);
-
-        return 0;
-
-err:
-        if ((flags & MEDIA_LNK_FL_ENABLED) && mdev->link_notify)
-                mdev->link_notify(link->source, link->sink, 0);
+        if (mdev->link_notify)
+                mdev->link_notify(link, flags, MEDIA_DEV_NOTIFY_POST_LINK_CH);
 
         return ret;
 }