OSDN Git Service

[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 856fdf5..313fd13 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)))