Hi Laurent, Laurent Pinchart wrote: > Hi Sakari, > > Thanks for the patch. > > On Monday 20 February 2012 03:57:09 Sakari Ailus wrote: >> Signed-off-by: Sakari Ailus <sakari.ailus@xxxxxx> >> --- >> drivers/media/video/omap3isp/ispresizer.c | 4 ++++ >> 1 files changed, 4 insertions(+), 0 deletions(-) >> >> diff --git a/drivers/media/video/omap3isp/ispresizer.c >> b/drivers/media/video/omap3isp/ispresizer.c index 6ce2349..81e1bc4 100644 >> --- a/drivers/media/video/omap3isp/ispresizer.c >> +++ b/drivers/media/video/omap3isp/ispresizer.c >> @@ -1147,9 +1147,13 @@ static int resizer_set_stream(struct v4l2_subdev *sd, >> int enable) struct device *dev = to_device(res); >> >> if (res->state == ISP_PIPELINE_STREAM_STOPPED) { >> + struct isp_pipeline *pipe = to_isp_pipeline(&sd->entity); >> + >> if (enable == ISP_PIPELINE_STREAM_STOPPED) >> return 0; >> >> + omap3isp_resizer_max_rate(res, &pipe->max_rate); >> + >> omap3isp_subclk_enable(isp, OMAP3_ISP_SUBCLK_RESIZER); >> resizer_configure(res); >> resizer_print_status(res); > > What about moving this to link validation ? Done. I hope I can resend the patchset tomorrow. The SMIA++ driver is unchanged just in case you happen to have time to review that one. ;-) Cheers, -- Sakari Ailus sakari.ailus@xxxxxx -- 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