[media] omap3isp: Don't try to locate external subdev for mem-to-mem pipelines
authorLaurent Pinchart <laurent.pinchart@ideasonboard.com>
Sat, 15 Feb 2014 00:40:48 +0000 (21:40 -0300)
committerMauro Carvalho Chehab <m.chehab@samsung.com>
Mon, 24 Feb 2014 15:53:11 +0000 (12:53 -0300)
Memory-to-memory pipelines have no external subdev, we shouldn't try to
locate one and validate its configuration. The driver currently works by
chance due to another bug that results in failure to locate the external
subdev being ignored.

This gets rid of the "omap3isp omap3isp: can't find source, failing now"
error message in the kernel log when operating on a memory-to-memory
pipeline.

Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Acked-by: Sakari Ailus <sakari.ailus@iki.fi>
Tested-by: Peter Meerwald <pmeerw@pmeerw.net>
Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>
drivers/media/platform/omap3isp/ispvideo.c

index 856fdf55403580c996493b03b811b3c4ef6ac5b0..313fd13994a10cdadffbe94a06722a6f2268d440 100644 (file)
@@ -888,6 +888,10 @@ static int isp_video_check_external_subdevs(struct isp_video *video,
        unsigned int i;
        int ret = 0;
 
+       /* Memory-to-memory pipelines have no external subdev. */
+       if (pipe->input != NULL)
+               return 0;
+
        for (i = 0; i < ARRAY_SIZE(ents); i++) {
                /* Is the entity part of the pipeline? */
                if (!(pipe->entities & (1 << ents[i]->id)))