On 07/11/11 11:57, Laurent Pinchart wrote: > On Monday 11 July 2011 12:54:42 Laurent Pinchart wrote: >> On Monday 11 July 2011 12:41:49 Jonathan Cameron wrote: > > [snip] > >> I think we should try to fix it in ispvideo.c instead. You could add a >> check to isp_video_validate_pipeline() to make sure that the pipeline has >> a video source. > > And I forgot to mention, I can send a patch if you don't want to write it. > Given I can't quite see why the validate_pipeline code would ever want to break on source pad being null (which I think is what it is currently doing), I'll leave it to you. Really don't know this code well enough! Thanks. Jonathan -- 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