Take into account media_entity_pipeline_start() may fail. This patch is dependent on "media: Add link_validate() op to check links to the sink pad". Signed-off-by: Sakari Ailus <sakari.ailus@xxxxxx> --- The dependent patch is part of my pull req to Mauro here: <URL:http://www.spinics.net/lists/linux-media/msg46296.html> drivers/media/video/s5p-fimc/fimc-capture.c | 4 +++- 1 files changed, 3 insertions(+), 1 deletions(-) diff --git a/drivers/media/video/s5p-fimc/fimc-capture.c b/drivers/media/video/s5p-fimc/fimc-capture.c index dc18ba5..8fd8095 100644 --- a/drivers/media/video/s5p-fimc/fimc-capture.c +++ b/drivers/media/video/s5p-fimc/fimc-capture.c @@ -963,7 +963,9 @@ static int fimc_cap_streamon(struct file *file, void *priv, if (fimc_capture_active(fimc)) return -EBUSY; - media_entity_pipeline_start(&p->sensor->entity, p->pipe); + ret = media_entity_pipeline_start(&p->sensor->entity, p->pipe); + if (ret < 0) + return ret; if (fimc->vid_cap.user_subdev_api) { ret = fimc_pipeline_validate(fimc); -- 1.7.2.5 -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html