This is an automatic generated email to let you know that the following patch 
were queued at the 
http://git.linuxtv.org/cgit.cgi/media_tree.git tree:

Subject: [media] v4l: exynos4-is: Drop unneeded check when setting up fimc-lite 
links
Author:  Laurent Pinchart <[email protected]>
Date:    Tue Mar 1 06:31:53 2016 -0300

The driver verifies that the type of the remote entity matches its
expectations when setting up fimc-lite links and returns an error if it
doesn't. Those checks can never fail as the links are created by the
driver in a way that always match its expectations (the SINK and
SOURCE_ISP pads are connected to subdevs only and the SOURCE_DMA pad is
connected to a video node only). Remove them.

Signed-off-by: Laurent Pinchart <[email protected]>
Signed-off-by: Mauro Carvalho Chehab <[email protected]>

 drivers/media/platform/exynos4-is/fimc-lite.c | 12 ++----------
 1 file changed, 2 insertions(+), 10 deletions(-)

---

http://git.linuxtv.org/cgit.cgi/media_tree.git/commit/?id=de08b5a8be0df1eb7c796b0fe6b30cf1d03d14a6
diff --git a/drivers/media/platform/exynos4-is/fimc-lite.c 
b/drivers/media/platform/exynos4-is/fimc-lite.c
index e85649147dc8..dc1b929f7a33 100644
--- a/drivers/media/platform/exynos4-is/fimc-lite.c
+++ b/drivers/media/platform/exynos4-is/fimc-lite.c
@@ -992,10 +992,6 @@ static int fimc_lite_link_setup(struct media_entity 
*entity,
 
        switch (local->index) {
        case FLITE_SD_PAD_SINK:
-               if (!is_media_entity_v4l2_subdev(remote->entity)) {
-                       ret = -EINVAL;
-                       break;
-               }
                if (flags & MEDIA_LNK_FL_ENABLED) {
                        if (fimc->source_subdev_grp_id == 0)
                                fimc->source_subdev_grp_id = sd->grp_id;
@@ -1010,19 +1006,15 @@ static int fimc_lite_link_setup(struct media_entity 
*entity,
        case FLITE_SD_PAD_SOURCE_DMA:
                if (!(flags & MEDIA_LNK_FL_ENABLED))
                        atomic_set(&fimc->out_path, FIMC_IO_NONE);
-               else if (is_media_entity_v4l2_io(remote->entity))
-                       atomic_set(&fimc->out_path, FIMC_IO_DMA);
                else
-                       ret = -EINVAL;
+                       atomic_set(&fimc->out_path, FIMC_IO_DMA);
                break;
 
        case FLITE_SD_PAD_SOURCE_ISP:
                if (!(flags & MEDIA_LNK_FL_ENABLED))
                        atomic_set(&fimc->out_path, FIMC_IO_NONE);
-               else if (is_media_entity_v4l2_subdev(remote->entity))
-                       atomic_set(&fimc->out_path, FIMC_IO_ISP);
                else
-                       ret = -EINVAL;
+                       atomic_set(&fimc->out_path, FIMC_IO_ISP);
                break;
 
        default:

_______________________________________________
linuxtv-commits mailing list
[email protected]
http://www.linuxtv.org/cgi-bin/mailman/listinfo/linuxtv-commits

Reply via email to