media: au0828 disable tuner to demod link in au0828_media_device_register()
authorShuah Khan <shuahkh@osg.samsung.com>
Thu, 10 Mar 2016 02:15:38 +0000 (19:15 -0700)
committerMauro Carvalho Chehab <mchehab@osg.samsung.com>
Thu, 10 Mar 2016 16:44:10 +0000 (13:44 -0300)
Disable tuner to demod link in au0828_media_device_register(). This step
should be done after dvb graph is created.

[mchehab@osg.samsung.com: Solve conflictst to apply it upstream]
Signed-off-by: Shuah Khan <shuahkh@osg.samsung.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com>
drivers/media/usb/au0828/au0828-core.c
drivers/media/v4l2-core/v4l2-mc.c

index 7fc3dba5e2b6e6e0b7a5050c9fc4971ffa8ca4e3..5dc82e8c8670f8ea299cf01151bf7293455410b7 100644 (file)
@@ -456,6 +456,7 @@ static int au0828_media_device_register(struct au0828_dev *dev,
 {
 #ifdef CONFIG_MEDIA_CONTROLLER
        int ret;
+       struct media_entity *entity, *demod = NULL, *tuner = NULL;
 
        if (!dev->media_dev)
                return 0;
@@ -479,6 +480,31 @@ static int au0828_media_device_register(struct au0828_dev *dev,
                */
                au0828_media_graph_notify(NULL, (void *) dev);
        }
+
+       /*
+        * Find tuner and demod to disable the link between
+        * the two to avoid disable step when tuner is requested
+        * by video or audio. Note that this step can't be done
+        * until dvb graph is created during dvb register.
+       */
+       media_device_for_each_entity(entity, dev->media_dev) {
+               if (entity->function == MEDIA_ENT_F_DTV_DEMOD)
+                       demod = entity;
+               else if (entity->function == MEDIA_ENT_F_TUNER)
+                       tuner = entity;
+       }
+       /* Disable link between tuner and demod */
+       if (tuner && demod) {
+               struct media_link *link;
+
+               list_for_each_entry(link, &demod->links, list) {
+                       if (link->sink->entity == demod &&
+                           link->source->entity == tuner) {
+                               media_entity_setup_link(link, 0);
+                       }
+               }
+       }
+
        /* register entity_notify callback */
        dev->entity_notify.notify_data = (void *) dev;
        dev->entity_notify.notify = (void *) au0828_media_graph_notify;
index f8b6e3be634f976bea1f9c426aae98302530f9ed..2a7b79bc90fdbb985a8ffacb84babb27f282aeb2 100644 (file)
@@ -57,9 +57,6 @@ int v4l2_mc_create_media_graph(struct media_device *mdev)
                case MEDIA_ENT_F_ATV_DECODER:
                        decoder = entity;
                        break;
-               case MEDIA_ENT_F_DTV_DEMOD:
-                       dtv_demod = entity;
-                       break;
                case MEDIA_ENT_F_IO_V4L:
                        io_v4l = entity;
                        break;
@@ -194,21 +191,6 @@ int v4l2_mc_create_media_graph(struct media_device *mdev)
                flags = 0;
        }
 
-       /*
-        * Disable tuner to demod link to avoid disable step
-        * when tuner is requested by video or audio
-        */
-       if (tuner && dtv_demod) {
-               struct media_link *link;
-
-               list_for_each_entry(link, &dtv_demod->links, list) {
-                       if (link->sink->entity == dtv_demod &&
-                           link->source->entity == tuner) {
-                               media_entity_setup_link(link, 0);
-                       }
-               }
-       }
-
        return 0;
 }
 EXPORT_SYMBOL_GPL(v4l2_mc_create_media_graph);